fix: visibility for impl methods #6261

asterite · 2024-10-09T15:32:22Z

Description

Problem

Part of #4515

Apparently in #6179 I only made it work for calls like foo::Bar::baz() but no checks were done for method calls (where self is involved).

Summary

This PR now warns when calling private or pub(crate) methods, either on structs or primitive types, when they are not accessible from the current module.

We also now don't suggest non-accessible methods from LSP.

Additional Context

I also removed an unused global that was introduced some time ago.

Documentation

Check one:

No documentation needed.
Documentation included in this PR.
[For Experimental Features] Documentation to be submitted in a separate PR.

PR Checklist

I have tested the changes locally.
I have formatted the changes with Prettier and/or cargo fmt on default settings.

jfecher

One thing, otherwise LGTM

compiler/noirc_frontend/src/hir_def/types.rs

feat: Add `checked_transmute` (noir-lang/noir#6262) feat!: kind size checks (noir-lang/noir#6137) feat: don't crash LSP when there are errors resolving the workspace (noir-lang/noir#6257) fix: don't warn on unuse global if it has an abi annotation (noir-lang/noir#6258) fix: don't warn on unused struct that has an abi annotation (noir-lang/noir#6254) feat: don't suggest private struct fields in LSP (noir-lang/noir#6256) feat: visibility for struct fields (noir-lang/noir#6221) fix: handle dfg databus in SSA normalization (noir-lang/noir#6249) fix: homogeneous input points for EC ADD (noir-lang/noir#6241) chore: add regression test for #5756 (noir-lang/noir#5770) feat: allow `unconstrained` after visibility (noir-lang/noir#6246) feat: optimize `Quoted::as_expr` by parsing just once (noir-lang/noir#6237) fix(frontend): Do not warn when a nested struct is provided as input to main (noir-lang/noir#6239) fix!: Change tag attributes to require a ' prefix (noir-lang/noir#6235) feat: recover from '=' instead of ':' in struct constructor/pattern (noir-lang/noir#6236)

…ry (noir-lang/noir#6273) feat: remove byte decomposition in `compute_decomposition` (noir-lang/noir#6159) fix: address inactive public key check in `verify_signature_noir` (noir-lang/noir#6270) feat(test): Fuzz test poseidon2 hash equivalence (noir-lang/noir#6265) fix!: Integer division is not the inverse of integer multiplication (noir-lang/noir#6243) feat(perf): Flamegraphs for test program execution benchmarks (noir-lang/noir#6253) fix: visibility for impl methods (noir-lang/noir#6261) feat: Add `checked_transmute` (noir-lang/noir#6262) feat!: kind size checks (noir-lang/noir#6137) feat: don't crash LSP when there are errors resolving the workspace (noir-lang/noir#6257) fix: don't warn on unuse global if it has an abi annotation (noir-lang/noir#6258) fix: don't warn on unused struct that has an abi annotation (noir-lang/noir#6254) feat: don't suggest private struct fields in LSP (noir-lang/noir#6256) feat: visibility for struct fields (noir-lang/noir#6221) fix: handle dfg databus in SSA normalization (noir-lang/noir#6249) fix: homogeneous input points for EC ADD (noir-lang/noir#6241) chore: add regression test for #5756 (noir-lang/noir#5770) feat: allow `unconstrained` after visibility (noir-lang/noir#6246) feat: optimize `Quoted::as_expr` by parsing just once (noir-lang/noir#6237) fix(frontend): Do not warn when a nested struct is provided as input to main (noir-lang/noir#6239) fix!: Change tag attributes to require a ' prefix (noir-lang/noir#6235) feat: recover from '=' instead of ':' in struct constructor/pattern (noir-lang/noir#6236)

…g/noir#6273) feat: remove byte decomposition in `compute_decomposition` (noir-lang/noir#6159) fix: address inactive public key check in `verify_signature_noir` (noir-lang/noir#6270) feat(test): Fuzz test poseidon2 hash equivalence (noir-lang/noir#6265) fix!: Integer division is not the inverse of integer multiplication (noir-lang/noir#6243) feat(perf): Flamegraphs for test program execution benchmarks (noir-lang/noir#6253) fix: visibility for impl methods (noir-lang/noir#6261) feat: Add `checked_transmute` (noir-lang/noir#6262) feat!: kind size checks (noir-lang/noir#6137) feat: don't crash LSP when there are errors resolving the workspace (noir-lang/noir#6257) fix: don't warn on unuse global if it has an abi annotation (noir-lang/noir#6258) fix: don't warn on unused struct that has an abi annotation (noir-lang/noir#6254) feat: don't suggest private struct fields in LSP (noir-lang/noir#6256) feat: visibility for struct fields (noir-lang/noir#6221) fix: handle dfg databus in SSA normalization (noir-lang/noir#6249) fix: homogeneous input points for EC ADD (noir-lang/noir#6241) chore: add regression test for #5756 (noir-lang/noir#5770) feat: allow `unconstrained` after visibility (noir-lang/noir#6246) feat: optimize `Quoted::as_expr` by parsing just once (noir-lang/noir#6237) fix(frontend): Do not warn when a nested struct is provided as input to main (noir-lang/noir#6239) fix!: Change tag attributes to require a ' prefix (noir-lang/noir#6235) feat: recover from '=' instead of ':' in struct constructor/pattern (noir-lang/noir#6236)

…oir-lang/noir#6279) feat(test): Fuzz poseidon hases against an external library (noir-lang/noir#6273) feat: remove byte decomposition in `compute_decomposition` (noir-lang/noir#6159) fix: address inactive public key check in `verify_signature_noir` (noir-lang/noir#6270) feat(test): Fuzz test poseidon2 hash equivalence (noir-lang/noir#6265) fix!: Integer division is not the inverse of integer multiplication (noir-lang/noir#6243) feat(perf): Flamegraphs for test program execution benchmarks (noir-lang/noir#6253) fix: visibility for impl methods (noir-lang/noir#6261) feat: Add `checked_transmute` (noir-lang/noir#6262) feat!: kind size checks (noir-lang/noir#6137) feat: don't crash LSP when there are errors resolving the workspace (noir-lang/noir#6257) fix: don't warn on unuse global if it has an abi annotation (noir-lang/noir#6258) fix: don't warn on unused struct that has an abi annotation (noir-lang/noir#6254) feat: don't suggest private struct fields in LSP (noir-lang/noir#6256) feat: visibility for struct fields (noir-lang/noir#6221) fix: handle dfg databus in SSA normalization (noir-lang/noir#6249) fix: homogeneous input points for EC ADD (noir-lang/noir#6241) chore: add regression test for #5756 (noir-lang/noir#5770) feat: allow `unconstrained` after visibility (noir-lang/noir#6246) feat: optimize `Quoted::as_expr` by parsing just once (noir-lang/noir#6237) fix(frontend): Do not warn when a nested struct is provided as input to main (noir-lang/noir#6239) fix!: Change tag attributes to require a ' prefix (noir-lang/noir#6235) feat: recover from '=' instead of ':' in struct constructor/pattern (noir-lang/noir#6236)

…ir#6279) feat(test): Fuzz poseidon hases against an external library (noir-lang/noir#6273) feat: remove byte decomposition in `compute_decomposition` (noir-lang/noir#6159) fix: address inactive public key check in `verify_signature_noir` (noir-lang/noir#6270) feat(test): Fuzz test poseidon2 hash equivalence (noir-lang/noir#6265) fix!: Integer division is not the inverse of integer multiplication (noir-lang/noir#6243) feat(perf): Flamegraphs for test program execution benchmarks (noir-lang/noir#6253) fix: visibility for impl methods (noir-lang/noir#6261) feat: Add `checked_transmute` (noir-lang/noir#6262) feat!: kind size checks (noir-lang/noir#6137) feat: don't crash LSP when there are errors resolving the workspace (noir-lang/noir#6257) fix: don't warn on unuse global if it has an abi annotation (noir-lang/noir#6258) fix: don't warn on unused struct that has an abi annotation (noir-lang/noir#6254) feat: don't suggest private struct fields in LSP (noir-lang/noir#6256) feat: visibility for struct fields (noir-lang/noir#6221) fix: handle dfg databus in SSA normalization (noir-lang/noir#6249) fix: homogeneous input points for EC ADD (noir-lang/noir#6241) chore: add regression test for #5756 (noir-lang/noir#5770) feat: allow `unconstrained` after visibility (noir-lang/noir#6246) feat: optimize `Quoted::as_expr` by parsing just once (noir-lang/noir#6237) fix(frontend): Do not warn when a nested struct is provided as input to main (noir-lang/noir#6239) fix!: Change tag attributes to require a ' prefix (noir-lang/noir#6235) feat: recover from '=' instead of ':' in struct constructor/pattern (noir-lang/noir#6236)

feat(improve): Remove scan through globals (noir-lang/noir#6282) feat: show LSP diagnostic related information (noir-lang/noir#6277) feat: trait inheritance (noir-lang/noir#6252) feat: optimize reading a workspace's files (noir-lang/noir#6281) fix: prevent compiler panic when popping from empty slices (noir-lang/noir#6274) feat(test): Include the PoseidonHasher in the fuzzing (noir-lang/noir#6280) feat: slightly improve "unexpected token" error message (noir-lang/noir#6279) feat(test): Fuzz poseidon hases against an external library (noir-lang/noir#6273) feat: remove byte decomposition in `compute_decomposition` (noir-lang/noir#6159) fix: address inactive public key check in `verify_signature_noir` (noir-lang/noir#6270) feat(test): Fuzz test poseidon2 hash equivalence (noir-lang/noir#6265) fix!: Integer division is not the inverse of integer multiplication (noir-lang/noir#6243) feat(perf): Flamegraphs for test program execution benchmarks (noir-lang/noir#6253) fix: visibility for impl methods (noir-lang/noir#6261) feat: Add `checked_transmute` (noir-lang/noir#6262) feat!: kind size checks (noir-lang/noir#6137) feat: don't crash LSP when there are errors resolving the workspace (noir-lang/noir#6257) fix: don't warn on unuse global if it has an abi annotation (noir-lang/noir#6258) fix: don't warn on unused struct that has an abi annotation (noir-lang/noir#6254) feat: don't suggest private struct fields in LSP (noir-lang/noir#6256) feat: visibility for struct fields (noir-lang/noir#6221) fix: handle dfg databus in SSA normalization (noir-lang/noir#6249) fix: homogeneous input points for EC ADD (noir-lang/noir#6241) chore: add regression test for #5756 (noir-lang/noir#5770) feat: allow `unconstrained` after visibility (noir-lang/noir#6246) feat: optimize `Quoted::as_expr` by parsing just once (noir-lang/noir#6237) fix(frontend): Do not warn when a nested struct is provided as input to main (noir-lang/noir#6239) fix!: Change tag attributes to require a ' prefix (noir-lang/noir#6235) feat: recover from '=' instead of ':' in struct constructor/pattern (noir-lang/noir#6236)

Automated pull of development from the [noir](https://github.com/noir-lang/noir) programming language, a dependency of Aztec. BEGIN_COMMIT_OVERRIDE feat(test): Fuzz poseidon hases against an external library (noir-lang/noir#6273) feat: remove byte decomposition in `compute_decomposition` (noir-lang/noir#6159) fix: address inactive public key check in `verify_signature_noir` (noir-lang/noir#6270) feat(test): Fuzz test poseidon2 hash equivalence (noir-lang/noir#6265) fix!: Integer division is not the inverse of integer multiplication (noir-lang/noir#6243) feat(perf): Flamegraphs for test program execution benchmarks (noir-lang/noir#6253) fix: visibility for impl methods (noir-lang/noir#6261) feat: Add `checked_transmute` (noir-lang/noir#6262) feat!: kind size checks (noir-lang/noir#6137) feat: don't crash LSP when there are errors resolving the workspace (noir-lang/noir#6257) fix: don't warn on unuse global if it has an abi annotation (noir-lang/noir#6258) fix: don't warn on unused struct that has an abi annotation (noir-lang/noir#6254) feat: don't suggest private struct fields in LSP (noir-lang/noir#6256) feat: visibility for struct fields (noir-lang/noir#6221) fix: handle dfg databus in SSA normalization (noir-lang/noir#6249) fix: homogeneous input points for EC ADD (noir-lang/noir#6241) chore: add regression test for #5756 (noir-lang/noir#5770) feat: allow `unconstrained` after visibility (noir-lang/noir#6246) feat: optimize `Quoted::as_expr` by parsing just once (noir-lang/noir#6237) fix(frontend): Do not warn when a nested struct is provided as input to main (noir-lang/noir#6239) fix!: Change tag attributes to require a ' prefix (noir-lang/noir#6235) feat: recover from '=' instead of ':' in struct constructor/pattern (noir-lang/noir#6236) END_COMMIT_OVERRIDE --------- Co-authored-by: Tom French <[email protected]>

asterite added 7 commits October 9, 2024 11:45

Warn when calling private method

0d6b9d7

LSP: don't suggest private methods

875231e

Error if calling private primitive method outside of the stdlib

ec7f08e

Fixes, and hide private primitive methods from LSP too

baa8de0

Merge branch 'master' into ab/impl-fn-visibility

adede47

Make a bunch of stdlib impl functions pub(crate)

da00014

Remove unused global

04366d4

asterite changed the title ~~Ab/impl fn visibility~~ feat: visibility for impl methods Oct 9, 2024

asterite changed the title ~~feat: visibility for impl methods~~ fix: visibility for impl methods Oct 9, 2024

asterite added 3 commits October 9, 2024 12:39

cargo fmt

3986401

The functions seem to be public

6fa1e6b

One more pub

bbcdf41

asterite requested a review from a team October 9, 2024 15:56

jfecher reviewed Oct 9, 2024

View reviewed changes

compiler/noirc_frontend/src/hir_def/types.rs Show resolved Hide resolved

jfecher approved these changes Oct 9, 2024

View reviewed changes

asterite added this pull request to the merge queue Oct 9, 2024

Merged via the queue into master with commit 70cbeb4 Oct 9, 2024
49 checks passed

asterite deleted the ab/impl-fn-visibility branch October 9, 2024 18:16

noirwhal mentioned this pull request Oct 9, 2024

chore: Release Noir(0.36.0) #6213

Open

AztecBot mentioned this pull request Oct 10, 2024

feat: Sync from noir AztecProtocol/aztec-packages#9099

Merged

AztecBot mentioned this pull request Oct 14, 2024

chore(master): Release 0.58.0 AztecProtocol/aztec-packages#9068

Open

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

fix: visibility for impl methods #6261

fix: visibility for impl methods #6261

asterite commented Oct 9, 2024 •

edited

Loading

jfecher left a comment

fix: visibility for impl methods #6261

fix: visibility for impl methods #6261

Conversation

asterite commented Oct 9, 2024 • edited Loading

Description

Problem

Summary

Additional Context

Documentation

PR Checklist

jfecher left a comment

Choose a reason for hiding this comment

asterite commented Oct 9, 2024 •

edited

Loading