Skip to content
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

Track more mut locations #123

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Track more mut locations #123

wants to merge 2 commits into from

Commits on Apr 12, 2020

  1. Move location so macro can be used in more places 

    Rust only allows macros to be used strictly after they have been
defined. It's weird, and not needed for this PR, but it will avoid later
developers running into the issues I did while developing this.
    @jonhoo
    jonhoo committed Apr 12, 2020
    Configuration menu
    Copy the full SHA
    adc8fab View commit details
    Browse the repository at this point in the history

  2. Track location for atomic new 

    This doesn't _really_ add any information, since the mut location is the
same as the "crated" location. However, when presented with an error
with no `mut` location, the user may erroneously believe that there's
some mutation they haven't been told about, when the race is really
on the construction of the item.
    @jonhoo
    jonhoo committed Apr 12, 2020
    Configuration menu
    Copy the full SHA
    7d96e7d View commit details
    Browse the repository at this point in the history