Re: [Ipmitool-devel] missing break at lib/ipmi_sdr.c

2013-02-27 Thread Zdenek Styblik
On Wed, Feb 27, 2013 at 9:37 AM, Zdenek Styblik wrote: [...] > > I've logged SF.net ticket for this. Link: > https://sourceforge.net/tracker/?group_id=95200&atid=610550 > > Z. Grh! Correct link: https://sourceforge.net/tracker/?func=detail&aid=3606181&group_id=95200&atid=610550 Sorry about that,

Re: [Ipmitool-devel] missing break at lib/ipmi_sdr.c

2013-02-27 Thread Zdenek Styblik
On Tue, Jan 8, 2013 at 12:57 PM, Zdenek Styblik wrote: > On Tue, Jan 8, 2013 at 12:42 PM, Liebig, Holger > wrote: >>> > >>> > Hi, >>> > >>> > The lib/ipmi_sdr.c above comment "/* SR2300, non-redundant, PSx present >>> */" >>> > The missing break is intentional due to redundancy, right? >>> >>> [L

Re: [Ipmitool-devel] missing break at lib/ipmi_sdr.c

2013-01-08 Thread Zdenek Styblik
On Tue, Jan 8, 2013 at 12:42 PM, Liebig, Holger wrote: >> > >> > Hi, >> > >> > The lib/ipmi_sdr.c above comment "/* SR2300, non-redundant, PSx present >> */" >> > The missing break is intentional due to redundancy, right? >> >> [Liebig, Holger] >> I would say no, but who knows for sure without Int

Re: [Ipmitool-devel] missing break at lib/ipmi_sdr.c

2013-01-08 Thread Liebig, Holger
> > > > Hi, > > > > The lib/ipmi_sdr.c above comment "/* SR2300, non-redundant, PSx present > */" > > The missing break is intentional due to redundancy, right? > > [Liebig, Holger] > I would say no, but who knows for sure without Intel documentation about > the OEM SDR's in this particular quite

Re: [Ipmitool-devel] missing break at lib/ipmi_sdr.c

2013-01-07 Thread Liebig, Holger
> > Hi, > > The lib/ipmi_sdr.c above comment "/* SR2300, non-redundant, PSx present */" > The missing break is intentional due to redundancy, right? [Liebig, Holger] I would say no, but who knows for sure without Intel documentation about the OEM SDR's in this particular quite old system from

Re: [Ipmitool-devel] missing break at lib/ipmi_sdr.c

2013-01-07 Thread Zdenek Styblik
On Mon, Jan 7, 2013 at 3:02 PM, Ales Ledvinka wrote: > Hi, > > The lib/ipmi_sdr.c above comment "/* SR2300, non-redundant, PSx present */" > The missing break is intentional due to redundancy, right? > Hello, yes, it seems like it(= it is not a bug). One line of comment wouldn't hurt, I'd say(I

[Ipmitool-devel] missing break at lib/ipmi_sdr.c

2013-01-07 Thread Ales Ledvinka
Hi, The lib/ipmi_sdr.c above comment "/* SR2300, non-redundant, PSx present */" The missing break is intentional due to redundancy, right? -- Master Visual Studio, SharePoint, SQL, ASP.NET, C# 2012, HTML5, CSS, MVC, Windo