Skip to content
By using Twitter’s services you agree to our Cookies Use. We and our partners operate globally and use cookies, including for analytics, personalisation, and ads.

This is the legacy version of twitter.com. We will be shutting it down on June 1, 2020. Please switch to a supported browser, or disable the extension which masks your browser. You can see a list of supported browsers in our Help Center.

  • Home Home Home, current page.
  • About

Saved searches

  • Remove
  • In this conversation
    Verified accountProtected Tweets @
Suggested users
  • Verified accountProtected Tweets @
  • Verified accountProtected Tweets @
  • Language: English
    • Bahasa Indonesia
    • Bahasa Melayu
    • Català
    • Čeština
    • Dansk
    • Deutsch
    • English UK
    • Español
    • Filipino
    • Français
    • Hrvatski
    • Italiano
    • Magyar
    • Nederlands
    • Norsk
    • Polski
    • Português
    • Română
    • Slovenčina
    • Suomi
    • Svenska
    • Tiếng Việt
    • Türkçe
    • Ελληνικά
    • Български език
    • Русский
    • Српски
    • Українська мова
    • עִבְרִית
    • العربية
    • فارسی
    • मराठी
    • हिन्दी
    • বাংলা
    • ગુજરાતી
    • தமிழ்
    • ಕನ್ನಡ
    • ภาษาไทย
    • 한국어
    • 日本語
    • 简体中文
    • 繁體中文
  • Have an account? Log in
    Have an account?
    · Forgot password?

    New to Twitter?
    Sign up
pcwalton's profile
Patrick Walton
Patrick Walton
Patrick Walton
@pcwalton

Tweets

Patrick Walton

@pcwalton

Research engineer at Mozilla

San Francisco, CA
pcwalton.github.io
Joined November 2009

Tweets

  • © 2020 Twitter
  • About
  • Help Center
  • Terms
  • Privacy policy
  • Imprint
  • Cookies
  • Ads info
Dismiss
Previous
Next

Go to a person's profile

Saved searches

  • Remove
  • In this conversation
    Verified accountProtected Tweets @
Suggested users
  • Verified accountProtected Tweets @
  • Verified accountProtected Tweets @

Promote this Tweet

Block

  • Tweet with a location

    You can add location information to your Tweets, such as your city or precise location, from the web and via third-party applications. You always have the option to delete your Tweet location history. Learn more

    Your lists

    Create a new list


    Under 100 characters, optional

    Privacy

    Copy link to Tweet

    Embed this Tweet

    Embed this Video

    Add this Tweet to your website by copying the code below. Learn more

    Add this video to your website by copying the code below. Learn more

    Hmm, there was a problem reaching the server.

    By embedding Twitter content in your website or app, you are agreeing to the Twitter Developer Agreement and Developer Policy.

    Preview

    Why you're seeing this ad

    Log in to Twitter

    · Forgot password?
    Don't have an account? Sign up »

    Sign up for Twitter

    Not on Twitter? Sign up, tune into the things you care about, and get updates as they happen.

    Sign up
    Have an account? Log in »

    Two-way (sending and receiving) short codes:

    Country Code For customers of
    United States 40404 (any)
    Canada 21212 (any)
    United Kingdom 86444 Vodafone, Orange, 3, O2
    Brazil 40404 Nextel, TIM
    Haiti 40404 Digicel, Voila
    Ireland 51210 Vodafone, O2
    India 53000 Bharti Airtel, Videocon, Reliance
    Indonesia 89887 AXIS, 3, Telkomsel, Indosat, XL Axiata
    Italy 4880804 Wind
    3424486444 Vodafone
    » See SMS short codes for other countries

    Confirmation

     

    Welcome home!

    This timeline is where you’ll spend most of your time, getting instant updates about what matters to you.

    Tweets not working for you?

    Hover over the profile pic and click the Following button to unfollow any account.

    Say a lot with a little

    When you see a Tweet you love, tap the heart — it lets the person who wrote it know you shared the love.

    Spread the word

    The fastest way to share someone else’s Tweet with your followers is with a Retweet. Tap the icon to send it instantly.

    Join the conversation

    Add your thoughts about any Tweet with a Reply. Find a topic you’re passionate about, and jump right in.

    Learn the latest

    Get instant insight into what people are talking about now.

    Get more of what you love

    Follow more accounts to get instant updates about topics you care about.

    Find what's happening

    See the latest conversations about any topic instantly.

    Never miss a Moment

    Catch up instantly on the best stories happening as they unfold.

    1. Graydon Hoare‏ @graydon_pub 11 Jan 2019
      • Report Tweet
      • Report NetzDG Violation

      For those of you who are all like "hey why did you initially try to make it so rust didn't allow variable shadowing?" I present the past 3 hours of hunting down a variable shadowing bug I wrote, that was only caught by a very-downstream integrity assertion.

      6 replies 3 retweets 74 likes
    2. corvus frugilegus‏ @glaebhoerl 12 Jan 2019
      • Report Tweet
      • Report NetzDG Violation
      Replying to @graydon_pub

      Could you briefly share relevant details (or a link, if public)? I've been on a hunt w.r.t. "shadowing warnings are annoying as _heck_, I wonder what sort of bugs they actually prevent" for some time now.

      1 reply 0 retweets 3 likes
    3. Graydon Hoare‏ @graydon_pub 12 Jan 2019
      • Report Tweet
      • Report NetzDG Violation
      Replying to @glaebhoerl

      In this case, moved some “self contained” lines of code (that happen to bind and use a local temporary foo) into a function that had foo as a parameter name. Parameter now silently ignored in favor of local.

      3 replies 0 retweets 5 likes
    4. Graydon Hoare‏ @graydon_pub 12 Jan 2019
      • Report Tweet
      • Report NetzDG Violation
      Replying to @graydon_pub @glaebhoerl

      (in old lines of function, following the new lines)

      1 reply 0 retweets 0 likes
    5. Patrick Walton‏ @pcwalton 12 Jan 2019
      • Report Tweet
      • Report NetzDG Violation
      Replying to @graydon_pub @glaebhoerl

      In Rust, or some other language?

      1 reply 0 retweets 1 like
    6. Graydon Hoare‏ @graydon_pub 12 Jan 2019
      • Report Tweet
      • Report NetzDG Violation
      Replying to @pcwalton @glaebhoerl

      C++. I think Rust at least warns by default. There's an extended warning (-Wshadow) but of course it wasn't turned on on this codebase (-Wall doesn't include it). Representative example:https://gist.github.com/graydon/eec72ed4da23ddd156d2642b49d53f26 …

      2 replies 0 retweets 1 like
    7. Patrick Walton‏ @pcwalton 12 Jan 2019
      • Report Tweet
      • Report NetzDG Violation
      Replying to @graydon_pub @glaebhoerl

      Yeah, the unused parameter/unused var/variable never read warnings usually catch this in Rust by default. It is possible to have shadowing bugs, but they’re rarer. There’s a clippy lint for the rest of them: https://github.com/rust-lang/rust-clippy/blob/master/clippy_lints/src/shadow.rs …

      1 reply 0 retweets 2 likes
      Patrick Walton‏ @pcwalton 12 Jan 2019
      • Report Tweet
      • Report NetzDG Violation
      Replying to @pcwalton @graydon_pub @glaebhoerl

      Sadly unused parameters are so common in C++ that warning on them would be too noisy to turn by default :(

      8:30 PM - 12 Jan 2019
      • 1 Like
      • Valentin Goșu 🇷🇴🤓🦖🦊
      2 replies 0 retweets 1 like
        1. Graydon Hoare‏ @graydon_pub 12 Jan 2019
          • Report Tweet
          • Report NetzDG Violation
          Replying to @pcwalton @glaebhoerl

          Yeah. Existing codebases are tough; the C++ song has been playing long enough that the constant record skips are by now all part of the melody.

          0 replies 1 retweet 7 likes
          Thanks. Twitter will use this to make your timeline better. Undo
          Undo
        1. Nicholas Nethercote‏ @nnethercote 12 Jan 2019
          • Report Tweet
          • Report NetzDG Violation
          Replying to @pcwalton @graydon_pub @glaebhoerl

          I looked into it for Firefox a while back: https://bugzilla.mozilla.org/show_bug.cgi?id=1232547 … If you exclude or comment out the argument's name then you avoid the warning. It's a heavy lift for a big C++ codebase, but would be very doable for a new or small codebase.

          0 replies 0 retweets 3 likes
          Thanks. Twitter will use this to make your timeline better. Undo
          Undo

      Loading seems to be taking a while.

      Twitter may be over capacity or experiencing a momentary hiccup. Try again or visit Twitter Status for more information.

        Promoted Tweet

        false

        • © 2020 Twitter
        • About
        • Help Center
        • Terms
        • Privacy policy
        • Imprint
        • Cookies
        • Ads info