Turn off verbose logging but keep logic for future reference for debug logging.

git-svn-id: http://google-breakpad.googlecode.com/svn/trunk@1144 4c0a9323-5329-0410-9bdc-e9ce6186880e
This commit is contained in:
jessicag.feedback 2013-04-05 00:53:45 +00:00
parent b6e66b294f
commit 2652675ec2

View file

@ -72,12 +72,18 @@ bool RangeMap<AddressType, EntryType>::StoreRange(const AddressType &base,
// Some other range begins in the space used by this range. It may be // Some other range begins in the space used by this range. It may be
// contained within the space used by this range, or it may extend lower. // contained within the space used by this range, or it may extend lower.
// Regardless, it is an error. // Regardless, it is an error.
AddressType other_base = iterator_base->second.base(); // The processor hits this case too frequently with common symbol files.
AddressType other_size = iterator_base->first - other_base + 1; // This is most appropriate for a DEBUG channel, but since none exists now
BPLOG(INFO) << "StoreRange failed, an existing range is contained by or " // simply comment out this logging.
"extends lower than the new range: new " << //
HexString(base) << "+" << HexString(size) << ", existing " << // AddressType other_base = iterator_base->second.base();
HexString(other_base) << "+" << HexString(other_size); // AddressType other_size = iterator_base->first - other_base + 1;
// BPLOG(INFO) << "StoreRange failed, an existing range is contained by or "
// "extends lower than the new range: new " <<
// HexString(base) << "+" << HexString(size) <<
// ", existing " << HexString(other_base) << "+" <<
// HexString(other_size);
return false; return false;
} }
@ -86,13 +92,17 @@ bool RangeMap<AddressType, EntryType>::StoreRange(const AddressType &base,
// The range above this one overlaps with this one. It may fully // The range above this one overlaps with this one. It may fully
// contain this range, or it may begin within this range and extend // contain this range, or it may begin within this range and extend
// higher. Regardless, it's an error. // higher. Regardless, it's an error.
AddressType other_base = iterator_high->second.base(); // The processor hits this case too frequently with common symbol files.
AddressType other_size = iterator_high->first - other_base + 1; // This is most appropriate for a DEBUG channel, but since none exists now
BPLOG(INFO) << "StoreRange failed, an existing range contains or " // simply comment out this logging.
"extends higher than the new range: new " << //
HexString(base) << "+" << HexString(size) << // AddressType other_base = iterator_high->second.base();
", existing " << // AddressType other_size = iterator_high->first - other_base + 1;
HexString(other_base) << "+" << HexString(other_size); // BPLOG(INFO) << "StoreRange failed, an existing range contains or "
// "extends higher than the new range: new " <<
// HexString(base) << "+" << HexString(size) <<
// ", existing " << HexString(other_base) << "+" <<
// HexString(other_size);
return false; return false;
} }
} }