This is an archive of the discontinued LLVM Phabricator instance.

Modify Mozilla - break before binary operator
AbandonedPublic

Authored by Abpostelnicu on Feb 15 2017, 6:45 AM.

Details

Reviewers
sylvestre.ledru
Summary

We at Mozilla want to update the coding style that is used for the Mozilla format in order to make it compliant with our coding style guidelines: https://developer.mozilla.org/en-US/docs/Mozilla/Developer_guide/Coding_Style#Operators

This revision is addressed to fix the following situations when the correct format should be changed from:

MOZ_ASSERT(!mChild && !(mBits & nsCachedStyleData::GetBitForSID(aSID)) &&
           !GetCachedStyleData(aSID),
           "bar");

MOZ_ASSERT(!mChild && !(mBits & nsCachedStyleData::GetBitForSID(aSID))
             && !GetCachedStyleData(aSID),
           "bar");

Diff Detail

Repository
rL LLVM

Event Timeline

Abpostelnicu created this revision.Feb 15 2017, 6:45 AM
Abpostelnicu edited the summary of this revision. (Show Details)Feb 15 2017, 7:26 AM

Please don't accept it yet. Seems that there is no consensus yet

Abpostelnicu abandoned this revision.Jan 16 2019, 11:56 PM