Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

Update variable scope section in ReadMe instructing to use a let block #67

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Update variable scope section in ReadMe instructing to use a let block #67

wants to merge 1 commit into from

Conversation

Tortar
Copy link

@Tortar Tortar commented May 11, 2024 •
edited
Loading

Should superseed #61 and fix #66

both of this work with the new suggestion:

using Suppressor
g = let
        g = 1
        @suppress g += 1
        g
    end

module M
    using Suppressor
    g = let
            g = 1
            @suppress g += 1
            g
        end
end

since now

julia> g
2

julia> Main.M.g
2

@Tortar Tortar mentioned this pull request May 11, 2024
Open
@Tortar
Copy link
Author

Tortar commented May 11, 2024 •
edited
Loading

Actually another possibility is to say that @supress should be used only one expression at a time with rebinding e.g.

julia> using Suppressor

julia> f = @suppress function f end; f
f (generic function with 0 methods)

julia> f
f (generic function with 0 methods)

# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Suppressor.jl doesn't work when used in a module
1 participant
@Tortar