On Tue, Nov 6, 2012 at 11:54 AM, Zdenek Styblik
<zdenek.styb...@gmail.com> wrote:
> Hi,
>
> attached is my take on mending of ipmi_fru_main(). Main objectives were:
> * printf() -> lprintf() where appropriate
> * return codes
> * clean up ipmi_fru_main() a bit
>
[...]

Hello,

attached is a second version of proposed patch. It actually got a bit
beyond original scope.
Despite ipmi_fru_main() is still quite massive(~ 200+ lines) I hope
it's more readable than it used to be.

I do plan to commit this soon and move on.

Z.

Attachment: ipmi_fru.c-main_rework.diff
Description: Binary data

------------------------------------------------------------------------------
Everyone hates slow websites. So do we.
Make your web apps faster with AppDynamics
Download AppDynamics Lite for free today:
http://p.sf.net/sfu/appdyn_d2d_nov
_______________________________________________
Ipmitool-devel mailing list
Ipmitool-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/ipmitool-devel

Reply via email to