[prev in list] [next in list] [prev in thread] [next in thread]
List: illumos-developer
Subject: Re: [developer] [REVIEW] 7126 NSS_XbyY_FINI has too much logic for a function-like macro
From: "Igor Kozhukhov" <ikozhukhov () gmail ! com>
Date: 2016-06-23 15:36:36
Message-ID: 62BB0867-85C4-45B3-8BD5-5A39AE73E354 () gmail ! com
[Download RAW message or body]
LGTM.
i like your description about it - it annoys not only you :)
-Igor
> On Jun 23, 2016, at 6:21 PM, Richard Lowe <richlowe@richlowe.net> wrote:
>
> webrev: http://cr.illumos.org/~webrev/richlowe/il_7126/ \
> <http://cr.illumos.org/~webrev/richlowe/il_7126/>
> This pacifies the compiler, and also all right-thinking people.
>
> -- Rich
> illumos-developer | Archives <https://www.listbox.com/member/archive/182179/=now> \
> <https://www.listbox.com/member/archive/rss/182179/21175093-eccedb2d> | Modify \
> <https://www.listbox.com/member/?&> Your Subscription <http://www.listbox.com/>
-------------------------------------------
illumos-developer
Archives: https://www.listbox.com/member/archive/182179/=now
RSS Feed: https://www.listbox.com/member/archive/rss/182179/25758058-4e9228dc
Modify Your Subscription: \
https://www.listbox.com/member/?member_id=25758058&id_secret=25758058-c19b436a \
Powered by Listbox: http://www.listbox.com
[Attachment #3 (unknown)]
<html><html><html><head><meta content="text/html charset=us-ascii" \
http-equiv="Content-Type" /></head><body class="" style="word-wrap: break-word; \
-webkit-nbsp-mode: space; -webkit-line-break: after-white-space;"><div \
class="">LGTM.</div><div class="">i like your description about it - it annoys not \
only you :)</div><div class="">-Igor </div><br class="" /><div><blockquote \
class="" type="cite"><div class="">On Jun 23, 2016, at 6:21 PM, Richard Lowe <<a \
class="" href="mailto:richlowe@richlowe.net">richlowe@richlowe.net</a>> \
wrote:</div><br class="Apple-interchange-newline" /><div class=""><div class="" \
dir="ltr">webrev: <a class="" \
href="http://cr.illumos.org/~webrev/richlowe/il_7126/">http://cr.illumos.org/~webrev/richlowe/il_7126/</a><div \
class=""><br class="" /></div><div class="">This pacifies the compiler, and also all \
right-thinking people.</div><div class=""><br class="" /></div><div class="">-- \
Rich</div></div></div></blockquote></div><br class="" /><div bgcolor="#ffffff" \
id="listbox-footer" style="width:auto;margin:0;padding:5px;background-color:#fff;clear:both;border-top: \
1px solid #ccc;"><table bgcolor="#ffffff" border="0" cellpadding="0" cellspacing="0" \
style="background-color:#fff" width="100%"><tr><td padding="4px"><font \
color="#333333" size="1" style="font-family:helvetica, sans-serif;"> \
<strong>illumos-developer</strong> | <a \
href="https://www.listbox.com/member/archive/182179/=now" \
style="text-decoration:none;color:#669933;border-bottom: 1px solid #444444" title="Go \
to archives for illumos-developer">Archives</a> <a border="0" \
href="https://www.listbox.com/member/archive/rss/182179/25758058-4e9228dc" \
style="text-decoration:none;color:#669933" title="RSS feed for \
illumos-developer"><img border="0" \
src="http://postlink.www.listbox.com/2148198/833487e62783d55fe81f119fb93ef644/25758058 \
/bb3fe179.jpg?uri=aHR0cHM6Ly93d3cubGlzdGJveC5jb20vaW1hZ2VzL2ZlZWQtaWNvbi0xMHgxMC5qcGc" \
/></a> | <a href="https://www.listbox.com/member/?member_id=25758058&id_secret=25758058-c19b436a" \
style="text-decoration:none;color:#669933;border-bottom: 1px solid #444444" \
title="">Modify</a> Your Subscription<td align="right" valign="top"><a \
href="http://www.listbox.com" style="border-bottom:none;"> <img border="0" \
src="http://postlink.www.listbox.com/2148199/3379085af0f1cf7fc3708f04b4471ae2/25758058 \
/bb3fe179.png?uri=aHR0cHM6Ly93d3cubGlzdGJveC5jb20vaW1hZ2VzL2xpc3Rib3gtbG9nby1zbWFsbC5wbmc" \
title="Powered by Listbox" \
/></a></td></font></td></tr></table></div></body></html></html></html>
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic