Closed Bug 1478896 Opened 2 years ago Closed 2 years ago

Fix HashTable.h style

Categories

(Core :: MFBT, enhancement)

enhancement
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla63
Tracking Status
firefox63 --- fixed

People

(Reporter: njn, Assigned: njn)

References

Details

Attachments

(2 files)

It's currently in SpiderMonkey style, and needs converting to Gecko style. I'll do this in two steps:

- Run clang-format to fix up whitespace.

- Manually do other stuff, such as fixing brace style, using mFoo/aFoo naming, etc.
froydnj: I will combine the two patches before landing in order to minimize blame churn. And apologies for the tedious reviews.
Comment on attachment 8996221 [details]
Bug 1478896 - Fix HashTable.h style.

https://reviewboard.mozilla.org/r/260422/#review267534

rs=me.  I skimmed this, and nothing looks objectionable.
Attachment #8996221 - Flags: review?(nfroyd) → review+
Comment on attachment 8996222 [details]
Bug 1478896 - Fix HashTable.h style.

https://reviewboard.mozilla.org/r/260424/#review267536
Attachment #8996222 - Flags: review?(nfroyd) → review+
https://hg.mozilla.org/mozilla-central/rev/a8d92bdf7d0e
Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla63
You need to log in before you can comment on or make changes to this bug.