tim: Tim with short hair, smiling, wearing a black jacket over a white T-shirt (Default)
[personal profile] tim
Today:

* Got my massive pull request for issue 2633 (removing last-use analysis) ready to submit. Yes, I said "pull request", even though normally I'd be committing this directly. This week we decided to switch to a code-review model, even for core Rust contributors. So all of us will have to file a pull request and request a code review for any non-trivial change (we're being trusted as to deciding what's "trivial"). I think that's a good thing -- should foster more of us understanding more of the code in the compiler.

* Worked on issue 3521 and related. I had a minor "aha!" of realizing that there was already a convenient place in resolve to attach the check I needed to attach (that constants don't refer to names that aren't constant, such as local vars or arguments): in upvarify. A bit of resolve hacking later, I got the test case that tests that a constant declaration referring to a local variable is rejected working. Still working on getting discriminant exprs in enums to be treated the same way.

(no subject)

Date: 2012-10-13 08:30 pm (UTC)
ext_17921: (Default)
From: [identity profile] lindseykuper.livejournal.com
This week we decided to switch to a code-review model

I'm really glad to hear this -- I think it'll be good for the project. Someone should do a rust-dev post about it once the process is official.

Profile

tim: Tim with short hair, smiling, wearing a black jacket over a white T-shirt (Default)
Tim Chevalier

October 2014

S M T W T F S
    1234
56 7891011
1213 1415161718
19202122232425
262728293031 

Most Popular Tags

Style Credit

Expand Cut Tags

No cut tags