[illumos-Developer] [REVIEW] 308 ld may misalign sections only preceded by empty sections

Garrett D'Amore garrett at nexenta.com
Fri Feb 25 22:31:43 PST 2011


I've gone over the code in detail, and it looks ok to me.  I'm not too thrilled about keeping the history in a readme, but there is precedent.  I'd like to see that pruned in the future.

  -- Garrett

Gordon Ross <gordon.w.ross at gmail.com> wrote:

>I think Rich is being more than thorough here.
>Looks fine to me.
>
>On Fri, Feb 25, 2011 at 2:34 PM, Richard Lowe <richlowe at richlowe.net> wrote:
>>
>> Hey all, I'd like cautious code review for:
>>    308 ld may misalign sections only preceded by empty sections
>>
>> webrev:
>>    http://richlowe.net/webrevs/il_308/
>>
>[...]
>
>_______________________________________________
>Developer mailing list
>Developer at lists.illumos.org
>http://lists.illumos.org/m/listinfo/developer


More information about the Developer mailing list