[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