[go: up one dir, main page]

Page MenuHomePhabricator

Bug 2040286 - Treat using bindings as non-writable like const. r=arai
ClosedPublic

Authored by debadree333 on Aug 1 2026, 5:56 PM.
Referenced Files
Unknown Object (File)
Mon, Oct 12, 8:16 PM
Unknown Object (File)
Sun, Oct 11, 11:09 PM
Unknown Object (File)
Sun, Oct 11, 9:46 PM
Unknown Object (File)
Sun, Oct 11, 8:13 PM
Unknown Object (File)
Fri, Oct 2, 10:49 PM
Unknown Object (File)
Wed, Sep 30, 8:11 PM
Unknown Object (File)
Tue, Sep 29, 12:18 AM
Unknown Object (File)
Sep 11 2026, 3:05 PM
Subscribers
None

Diff Detail

Event Timeline

debadree333 created this revision.
phab-bot changed the visibility from "Custom Policy" to "Public (No Login Required)".Aug 1 2026, 5:56 PM
phab-bot changed the edit policy from "Custom Policy" to "Restricted Project (Project)".
phab-bot removed a project: secure-revision.
debadree333 edited the summary of this revision. (Show Details)

Shorten summary and simplify tests

debadree333 updated this revision to Diff 1349036.
debadree333 retitled this revision from WIP: Bug 2040286 - Treat using bindings as non-writable like const. to Bug 2040286 - Treat using bindings as non-writable like const. r=arai.
debadree333 added a reviewer: arai.
This comment was removed by debadree333.

Hello arai!

Hope you are doing well, I apologise for being away for so long and leaving this bugs, this patch solves 2040286 almost same according to how the bug's description states + added tests, I was thinking if for checks like

bi.kind() == BindingKind::Const || bi.kind() == BindingKind::Using)

we should have some helper function like isConstLike() or maybe that would be a overkill?

Thank you!

arai added a project: testing-approved.

Looks good!

I was thinking if for checks like

bi.kind() == BindingKind::Const || bi.kind() == BindingKind::Using)

we should have some helper function like isConstLike() or maybe that would be a overkill?

For the current consumers, it's not necessary, but if we have more similar consumers, it would be nice to add some helper method.

Have you already pushed the patch to try? If so, please post the URL of the try.
I'll land the patch once the green try run is confirmed.

Thanks!

This revision is now accepted and ready to land.Aug 18 2026, 12:04 AM