mirror of
https://github.com/mozilla/gecko-dev.git
synced 2024-11-25 13:51:41 +00:00
bug 298264: Change (to if-returns) and remove unnecessary NS_ENSURE_SUCCESS()s to reduce debug clutter. r=jst sr+a=brendan
This commit is contained in:
parent
53259eb945
commit
639c3faaa3
@ -807,9 +807,7 @@ nsresult nsHTMLTokenizer::ConsumeStartTag(PRUnichar aChar,
|
||||
result= aToken->Consume(aChar,aScanner,mFlags);
|
||||
|
||||
if(NS_SUCCEEDED(result)) {
|
||||
|
||||
AddToken(aToken,result,&mTokenDeque,theAllocator);
|
||||
NS_ENSURE_SUCCESS(result, result);
|
||||
|
||||
eHTMLTags theTag=(eHTMLTags)aToken->GetTypeID();
|
||||
|
||||
@ -980,7 +978,12 @@ nsresult nsHTMLTokenizer::ConsumeEndTag(PRUnichar aChar,
|
||||
// Tell the new token to finish consuming text...
|
||||
result= aToken->Consume(aChar,aScanner,mFlags);
|
||||
AddToken(aToken,result,&mTokenDeque,theAllocator);
|
||||
NS_ENSURE_SUCCESS(result, result);
|
||||
if (NS_FAILED(result)) {
|
||||
// Note that this early-return here is safe because we have not yet
|
||||
// added any of our tokens to the queue (AddToken only adds the token if
|
||||
// result is a success), so we don't need to fall through.
|
||||
return result;
|
||||
}
|
||||
|
||||
result = aScanner.Peek(aChar);
|
||||
if (NS_FAILED(result)) {
|
||||
|
Loading…
Reference in New Issue
Block a user