[PATCH] Store pointers to seen formatting states.

[David Blaikie]

On Thu, May 22, 2014 at 8:18 AM, Daniel Jasper <djasper at google.com> wrote:
>
>
>
> On Thu, May 22, 2014 at 5:15 PM, David Blaikie <dblaikie at gmail.com> wrote:
>>
>> On Thu, May 22, 2014 at 4:54 AM, Daniel Jasper <djasper at google.com> wrote:
>> > As there was no objection by now, I assume this is acceptable.
>>
>> It's generally considered poor form to submit code you've sent review
>> when you haven't received that review... it creates a difficult
>> situation where people may feel it acceptable to submit due to long
>> review turnaround which isn't something we want to encourage in the
>> community.
>
>
> And you think me and the reviewer sitting right next to me can't figure this
> out ...

That's not really visible to the community - what it looks like is
code that was sent for review, never reviewed, and committed.

It's just be helpful for the reviewer sitting next to you have replied
on the review thread with an acceptance of some kind.

- David

>
>> If you send something for review, it's presumed to be because you
>> don't feel confident that it's the right change and need a second set
>> of eyes to confirm/discuss that with - time shouldn't change that.
>>
>> - David
>>
>> >
>> > http://reviews.llvm.org/D418
>> >
>> >
>> >
>> > _______________________________________________
>> > cfe-commits mailing list
>> > cfe-commits at cs.uiuc.edu
>> > http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
>
>