[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: Issue 1650: merge multiple header specifications. (issue 6445053)
From: |
reinhold . kainhofer |
Subject: |
Re: Issue 1650: merge multiple header specifications. (issue 6445053) |
Date: |
Mon, 30 Jul 2012 15:06:02 +0000 |
LGTM, seems to work correctly on all my (reg)tests.
I actually like David's idea of changing the header field values to
include correctness information. Still I like comments inside sample
code to make the reasons for a particular block clearer.
http://codereview.appspot.com/6445053/