Closed Bug 1486547 Opened Last year Closed Last year
In a protcol's class heirarchy there are two instances of a field named m
State, this is confusing
46 bytes, text/x-phabricator-request
|Details | Review|
I'm going to rename one of them to make reading the code easier.
There's also a field named mState on IProtocol, and this reduces confusion.
Comment on attachment 9004298 [details] Bug 1486547 - renamed the mState field on generated protocol classes; r?froydnj Nathan Froyd [:froydnj] has approved the revision.
Attachment #9004298 - Flags: review+
Pushed by firstname.lastname@example.org: https://hg.mozilla.org/integration/autoland/rev/8a38246f1a70 renamed the mState field on generated protocol classes; r=froydnj
You need to log in before you can comment on or make changes to this bug.