<div dir="ltr"><br><br><div class="gmail_quote"><div dir="ltr">On Thu, Jun 8, 2017 at 12:59 PM Christophe Fergeau <<a href="mailto:cfergeau@redhat.com">cfergeau@redhat.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Thu, Jun 08, 2017 at 04:41:55AM -0400, Marc-André Lureau wrote:<br>
> Hi<br>
><br>
> ----- Original Message -----<br>
> > On Thu, Jun 08, 2017 at 04:31:20AM -0400, Marc-André Lureau wrote:<br>
> > > Hi<br>
> > ><br>
> > > ----- Original Message -----<br>
> > > > On Thu, Jun 08, 2017 at 12:55:38AM +0400, <a href="mailto:marcandre.lureau@redhat.com" target="_blank">marcandre.lureau@redhat.com</a><br>
> > > > wrote:<br>
> > > > > From: Marc-André Lureau <<a href="mailto:marcandre.lureau@redhat.com" target="_blank">marcandre.lureau@redhat.com</a>><br>
> > > > ><br>
> > > > > Use shorter line, use the common "d" variable for private data access,<br>
> > > > > add brackets to ease reading the inner block vs the condition, remove<br>
> > > > > needless != NULL.<br>
> > > ><br>
> > > > I'd lean towards NACK for this one, one letter variable names is imo<br>
> > > > very bad for readability. I know this is widespread in the spice-gtk<br>
> > > > codebase, but I'd at least rather not expand that usage.<br>
> > > ><br>
> > ><br>
> > > You may rename it "priv", but then you lost the benefit of being really<br>
> > > short.<br>
> ><br>
> > Being really short is a benefit? This is where we are going to disagree :)<br>
><br>
> Well, in an proper OO language, you wouldn't even have it, it would be like magic!<br>
<br>
Maybe, maybe not, 'priv' members in C++ classes are not that unusual<br>
( <a href="https://en.wikipedia.org/wiki/Opaque_pointer#C.2B.2B" rel="noreferrer" target="_blank">https://en.wikipedia.org/wiki/Opaque_pointer#C.2B.2B</a> )<br>
<br>
> So yes, I like private member being accessed with a very short<br>
> variable in C. If it's use consistently, there is very little<br>
> confusion possible imho.<br>
<br>
As I said, we are not going to agree there :) "If it's used<br>
consistently, it does not cause confusion", does not mean that's a good<br>
thing :) This just makes things harder to read for someone not knowing<br>
the convention, and for no great reason (saving at most a few seconds of<br>
typing?)<br></blockquote><div><br></div><div>Ok you don't like that single letter variable, but as maintainer I prefer consistency, even though I don't have written rules, I am not strict either. If you feel strongly about it, I can drop the patch. I suggest you send a patch to change it if you think it will improve readability anyway.<br><br></div><div>thanks<br></div><div><br></div></div></div><div dir="ltr">-- <br></div><div data-smartmail="gmail_signature"><div dir="ltr">Marc-André Lureau<br></div></div>