<div class="socmaildefaultfont" dir="ltr" style="font-family:Arial;font-size:10.5pt" ><div dir="ltr" >Hey Jeremy,</div>
<div dir="ltr" > </div>
<div dir="ltr" >For the signature itself, I believe the current way I'm using it is what exactly the sd_bus API requires (except for converting those inline strings to static const global variables). I cannot figure out a way other than that. Any suggestions on how to handle the signatures in a better way?</div>
<div dir="ltr" > </div>
<div dir="ltr" >Again, currently those signatures are exactly what those methods/property calls required, and they are working fine according to my test. Please let me know if this community has a better way or has any convention of doing that.</div>
<div dir="ltr" > </div>
<div class="socmaildefaultfont" dir="ltr" style="font-family:Arial;font-size:10.5pt" ><div class="socmaildefaultfont" dir="ltr" style="font-family:Arial;font-size:10.5pt" ><div dir="ltr" >GOU, Peng Fei (苟鹏飞), Ph.D.</div>
<div dir="ltr" >OpenPower Team.</div>
<div dir="ltr" >+86-21-609-28631</div></div></div>
<div dir="ltr" > </div>
<div dir="ltr" > </div>
<blockquote data-history-content-modified="1" dir="ltr" style="border-left:solid #aaaaaa 2px; margin-left:5px; padding-left:5px; direction:ltr; margin-right:0px" >----- Original message -----<br>From: Jeremy Kerr <jk@ozlabs.org><br>To: Cyril Bur <cyrilbur@gmail.com>, Peng Fei BG Gou/China/IBM@IBMCN<br>Cc: openbmc@lists.ozlabs.org, openbmc-patches@stwcx.xyz<br>Subject: Re: [PATCH phosphor-host-ipmid v4] Add get/set boot option ipmid command support with correct DBUS property handling.<br>Date: Thu, Jan 7, 2016 9:42 AM<br> 
<div><font face="Default Monospace,Courier New,Courier,monospace" size="2" >Hi all,<br><br>> Again, I'm not saying it won't work but I'm trying to understand why the<br>> complexity in these signatures, it doesn't seem like it's being used and<br>> pointless complexity doesn't help anyone.<br><br>Yes - I'd prefer to get these method signatures correct before other<br>code starts relying on it.<br><br>[the same philosophy applies to the contributions in general - we should<br>be aiming to get the patch as correct as possible, rather than trying to<br>fix up later]<br><br>Regards,<br><br><br>Jeremy</font><br> </div></blockquote>
<div dir="ltr" > </div></div><BR>