On Mon, Oct 17, 2011 at 12:09 PM, Smith, Stan <[email protected]> wrote:
> Hello,**** > > I too noticed this change, no assert() fire on win7; not surprising. > Seems the discussion Alex, Hal and others had w.r.t. Windows setvbuf(BUFSIZ) > did not make it into the mainline.**** > > I’d prefer BUFSIZ as it’s a well-recognized system wide entity, although > 1024 would work equally well. > > > Although this was agreed to back on May 27, no patch was submitted :-( I'll take care of that shortly. -- Hal > > > I’ll update, thanks for pointing out the omission.**** > > ** ** > > Stan.**** > > ** ** > > ** ** > > ** ** > > *From:* Leonid Keller [mailto:[email protected]] > *Sent:* Saturday, October 15, 2011 11:23 AM > *To:* Smith, Stan > *Cc:* ofw_list > *Subject:* [ofw][bug] Assert in opensm 3.3.11**** > > ** ** > > Hi Stan,**** > > ** ** > > I got an assert in Win8 on setvbuf (see below).**** > > It states that *size* should be at least 2.**** > > Also MSDN agrees with that:**** > > *size * > > Buffer size in bytes. Allowable range: 2 <= size <= INT_MAX (2147483647)** > ** > > Here is the suggested patch.**** > > ** ** > > Index: main.c**** > > ===================================================================**** > > --- main.c (revision 8962)**** > > +++ main.c (revision 8963)**** > > @@ -732,7 +732,7 @@**** > > };**** > > **** > > /* force stdout to be line-buffered */**** > > - setvbuf(stdout, NULL, _IOLBF, 0);**** > > + setvbuf(stdout, NULL, _IOLBF, 1024);**** > > **** > > /* Make sure that the opensm and complib were compiled > using**** > > same modes (debug/free) */**** > > _______________________________________________ > ofw mailing list > [email protected] > http://lists.openfabrics.org/cgi-bin/mailman/listinfo/ofw >
_______________________________________________ ofw mailing list [email protected] http://lists.openfabrics.org/cgi-bin/mailman/listinfo/ofw
