Details
Details
- Reviewers
arai - Commits
- rFIREFOXAUTOLANDfc9d369df353: Bug 2040286 - Treat using bindings as non-writable like const. r=arai
- Bugzilla Bug ID
- 2040286
Diff Detail
Diff Detail
- Repository
- rFIREFOXAUTOLAND firefox-autoland
Event Timeline
This comment was removed by debadree333.
Comment Actions
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!
Comment Actions
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!