Re: [Ipmitool-devel] Code Review - ID: 3608261 - 'lib/ipmi_delloem.c' - code clean-up

2013-03-20 Thread Zdenek Styblik
Hello G, I don't have access to Dell hardware, so I haven't done any testing beyond compilation. I'll check some(= not all of them) help outputs later, but that's the most I can do right now. I have some changes in the code planned, but I don't intend to change functionality per se. To be more

Re: [Ipmitool-devel] Code Review - ID: 3608261 - 'lib/ipmi_delloem.c' - code clean-up

2013-03-20 Thread Srinivas_G_Gowda
Z, Thanks for the commit. Ideally we would want smaller patches. Helps in getting it reviewed better... ;) since we are not doing anything major in terms of features or change in functionality, we should be able to go ahead with it unless there are no issues checking in the code(I don't have