[go: up one dir, main page]

Page MenuHomePhabricator

Bug 2080792 - Move EvalOptions::setUseInnerBindings() to match getter/setter order. r=arai
ClosedPublic

Authored by cemase on Mon, Oct 12, 10:49 AM.
Referenced Files
Unknown Object (File)
Thu, Oct 15, 10:49 AM
Unknown Object (File)
Thu, Oct 15, 3:14 AM
Unknown Object (File)
Thu, Oct 15, 3:03 AM
Unknown Object (File)
Thu, Oct 15, 3:01 AM
Subscribers
None

Diff Detail

Event Timeline

phab-bot changed the visibility from "Custom Policy" to "Public (No Login Required)".
phab-bot changed the edit policy from "Custom Policy" to "Restricted Project (Project)".
phab-bot removed a project: secure-revision.
arai requested changes to this revision.Mon, Oct 12, 12:56 PM
arai added inline comments.
js/src/debugger/Debugger.h
436

While you're here, can you put an empty line here, so that the distinction between getter, setter, others are clearer?

439–442

Please flip them, given that kind_ is the last field.

449–455

Here too.

This revision now requires changes to proceed.Mon, Oct 12, 12:56 PM
cemase updated this revision to Diff 1403206.
cemase marked 3 inline comments as done.
This revision is now accepted and ready to land.Tue, Oct 13, 2:06 AM