[illumos-Developer] [REVIEW] issue 1126 - cosmetic change in date.c
Albert Lee
trisk at opensolaris.org
Mon Jun 20 10:56:08 PDT 2011
On Mon, Jun 20, 2011 at 10:23 AM, Garrett D'Amore <garrett at nexenta.com> wrote:
> code lgtm.
>
> -- Garrett D'Amore
>
> On Jun 20, 2011, at 6:42 AM, "Gouverneur Thomas" <tgo at ians.be> wrote:
>
>> Hi all,
>>
>> As this is my first contrib, please notice me of any mistake in the process ;-)
>>
>> CR: https://www.illumos.org/issues/1126
>> Webkit: http://wildcat.espix.org/cr/illumos-1126-webrev/
>> Comment: Correct by inspection, will run once build is complete to verify
>>
Looks good to me. (It's curious that -R is not allowed with the
mmddHHMM... date specification but it seems to be that way by design).
>> Also, How could I notice everyone on the bug tracker that this issue is currently under review
>> and that no other developer should spend time on this ?
>>
I've assigned the issue to you in the tracker.
-Albert
More information about the Developer
mailing list