Skip to content

Closing over an outer function arg is not caught by capture pass #364

New issue

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

Sign up for GitHub

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

Already on GitHub? Sign in to your account

Jump to bottom
Closed
marijnh opened this issue May 12, 2011 · 1 comment
Closed

Closing over an outer function arg is not caught by capture pass #364

marijnh opened this issue May 12, 2011 · 1 comment

Comments

@marijnh
Copy link
Contributor

marijnh commented May 12, 2011

This currently crashes the compiler (Map.get miss in ty_param_count_and_ty_for_def in typeck.rs when looking up outer_arg):

fn foo(int outer_arg) {
    fn local() -> int {
        ret outer_arg;
    }
}
@marijnh
Copy link
Contributor Author

marijnh commented May 13, 2011

This is handled by 58ec5d1

@marijnh marijnh closed this as completed May 13, 2011
keeperofdakeys pushed a commit to keeperofdakeys/rust that referenced this issue Dec 12, 2017
@alexcrichton
Merge pull request rust-lang#364 from nbaksalyar/master
85ac087 
Add missing consts on Solaris
dlrobertson pushed a commit to dlrobertson/rust that referenced this issue Nov 29, 2018
@Havvy
Document interaction of | patterns and pattern guards
249bc2f 
Fixes rust-lang#364
Aaron1011 pushed a commit to Aaron1011/rust that referenced this issue Oct 26, 2020
@dependabot
Merge pull request rust-lang#364 from bjorn3/dependabot/cargo/craneli…
4cb50fa 
…ft-2f61600
antoyo added a commit to antoyo/rust that referenced this issue Oct 26, 2023
@antoyo
Merge pull request rust-lang#364 from rust-lang/doc/target-spec
46887f2 
Fix to be able to use a target specification JSON file and document the process
carolynzech pushed a commit to carolynzech/rust that referenced this issue Jun 13, 2025
@github-actions @Natural-selection1
@Urgau @matthiaskrgr @bend-n @RalfJung @bors @tgross35 @ismailarilik @t5kd @ethanwu10 @Ayush1325 @shepmaster @notriddle @CAD97 @GrigorenkoPV @DiuDiu777 @SabrinaJewson @nnethercote @ChrisDenton @joboet @azhogin @DaniPopes @ShE3py @xobs @baumanj @lolbinarycat @m-ou-se @mathisbot @zachs18 @thaliaarchi @Amanieu @sayantn @ktnlvr @ebkalderon @GuillaumeGomez @Lynnesbian @cberner @Zalathar @paolobarbolini @the8472 @nk9 @vkrivopalov @viliml @dtolnay @hkBst @jhpratt @Berrysoft @ivmarkov @mkroening @dpaoliello @veluca93 @xizheyin @saethlin @Dietr1ch @fmease @pietroalbini @JulianKnodt @compiler-errors @mlowicki @samueltardieu @biabbas @federico-terzi @est31 @fluiderson @speedy-lex @melrief @DJMcNab @joshtriplett @tautschnig
Merge subtree update for toolchain nightly-2025-05-22 (rust-lang#364)
b4f59e8 
This is an automated PR to merge library subtree updates from 2025-05-20
(rust-lang/rust@60dabef) to 2025-05-22
(rust-lang/rust@bf64d66) (inclusive)
into main. `git merge` resulted in conflicts, which require manual
resolution. Files were commited with merge conflict markers. **Do not
remove or edit the following annotations:**
git-subtree-dir: library
git-subtree-split: 02d9329

---------

Signed-off-by: Ayush Singh <[email protected]>
Signed-off-by: Sean Cross <[email protected]>
Signed-off-by: Vladimir Krivopalov <[email protected]>
Signed-off-by: xizheyin <[email protected]>
Co-authored-by: Hegui Dai <[email protected]>
Co-authored-by: Urgau <[email protected]>
Co-authored-by: Matthias Krüger <[email protected]>
Co-authored-by: bendn <[email protected]>
Co-authored-by: Ralf Jung <[email protected]>
Co-authored-by: bors <[email protected]>
Co-authored-by: Trevor Gross <[email protected]>
Co-authored-by: The Miri Cronjob Bot <[email protected]>
Co-authored-by: ismailarilik <[email protected]>
Co-authored-by: Tobias <[email protected]>
Co-authored-by: Ethan Wu <[email protected]>
Co-authored-by: Ayush Singh <[email protected]>
Co-authored-by: Jake Goulding <[email protected]>
Co-authored-by: Michael Howell <[email protected]>
Co-authored-by: Christopher Durham <[email protected]>
Co-authored-by: Pavel Grigorenko <[email protected]>
Co-authored-by: LemonJ <[email protected]>
Co-authored-by: SabrinaJewson <[email protected]>
Co-authored-by: Nicholas Nethercote <[email protected]>
Co-authored-by: Chris Denton <[email protected]>
Co-authored-by: joboet <[email protected]>
Co-authored-by: Andrew Zhogin <[email protected]>
Co-authored-by: DaniPopes <[email protected]>
Co-authored-by: Lieselotte <[email protected]>
Co-authored-by: Sean Cross <[email protected]>
Co-authored-by: Trevor Gross <[email protected]>
Co-authored-by: Jon Bauman <[email protected]>
Co-authored-by: binarycat <[email protected]>
Co-authored-by: Mara Bos <[email protected]>
Co-authored-by: Mathis B <[email protected]>
Co-authored-by: Zachary S <[email protected]>
Co-authored-by: Thalia Archibald <[email protected]>
Co-authored-by: Amanieu d'Antras <[email protected]>
Co-authored-by: sayantn <[email protected]>
Co-authored-by: Artur Roos <[email protected]>
Co-authored-by: Eyal Kalderon <[email protected]>
Co-authored-by: Guillaume Gomez <[email protected]>
Co-authored-by: Lynnesbian <[email protected]>
Co-authored-by: Christopher Berner <[email protected]>
Co-authored-by: Stuart Cook <[email protected]>
Co-authored-by: Paolo Barbolini <[email protected]>
Co-authored-by: github-actions <[email protected]>
Co-authored-by: The 8472 <[email protected]>
Co-authored-by: Nick Kocharhook <[email protected]>
Co-authored-by: Vladimir Krivopalov <[email protected]>
Co-authored-by: Vilim Lendvaj <[email protected]>
Co-authored-by: David Tolnay <[email protected]>
Co-authored-by: Marijn Schouten <[email protected]>
Co-authored-by: Jacob Pratt <[email protected]>
Co-authored-by: 王宇逸 <[email protected]>
Co-authored-by: ivmarkov <[email protected]>
Co-authored-by: Martin Kröning <[email protected]>
Co-authored-by: Daniel Paoliello <[email protected]>
Co-authored-by: Luca Versari <[email protected]>
Co-authored-by: xizheyin <[email protected]>
Co-authored-by: Ben Kimock <[email protected]>
Co-authored-by: Dietrich Daroch <[email protected]>
Co-authored-by: León Orell Valerian Liehr <[email protected]>
Co-authored-by: Pietro Albini <[email protected]>
Co-authored-by: Julian Knodt <[email protected]>
Co-authored-by: Michael Goulet <[email protected]>
Co-authored-by: Michał Łowicki <[email protected]>
Co-authored-by: Samuel Tardieu <[email protected]>
Co-authored-by: B I Mohammed Abbas <[email protected]>
Co-authored-by: Federico Terzi <[email protected]>
Co-authored-by: sam skeoch <[email protected]>
Co-authored-by: est31 <[email protected]>
Co-authored-by: Fluid <[email protected]>
Co-authored-by: Speedy_Lex <[email protected]>
Co-authored-by: Mario Pastorelli <[email protected]>
Co-authored-by: Daniel McNab <[email protected]>
Co-authored-by: Josh Triplett <[email protected]>
Co-authored-by: gitbot <git@bot>
Co-authored-by: Michael Tautschnig <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@marijnh