Bugzilla – Full Text Bug Listing |
Summary: | ConfigStore RawTextConfigLoad doesn't reset stream error state | ||
---|---|---|---|
Product: | ns-3 | Reporter: | Peter Barnes <pdbarnes> |
Component: | config-store | Assignee: | Peter Barnes <pdbarnes> |
Status: | RESOLVED FIXED | ||
Severity: | major | CC: | tommaso.pecorella |
Priority: | P5 | ||
Version: | ns-3-dev | ||
Hardware: | All | ||
OS: | All | ||
See Also: | https://www.nsnam.org/bugzilla/show_bug.cgi?id=2508 | ||
Attachments: |
Proposed patch
test program input file for test program Revised test program |
Description
Peter Barnes
2016-09-21 14:13:21 EDT
Created attachment 2586 [details]
Proposed patch
Created attachment 2587 [details]
test program
Created attachment 2588 [details]
input file for test program
Created attachment 2589 [details]
Revised test program
Revised test program to simplify invocation:
- set ConfigStore defaults
- turn on logging for UdpEchoServerApplication
./waf --run config-test
*** Bug 2508 has been marked as a duplicate of this bug. *** (In reply to Tommaso Pecorella from comment #5) > *** Bug 2508 has been marked as a duplicate of this bug. *** In my opinion this could be safely pushed. I'd also say to indicate Robert Schmidt <rschmi@gmx.net> as the patch author. Wrt
> Since seekg(0) doesn't clear the eof bit (in c++99, in c++11 it does)
Since we have --std=c++11 I'm puzzled why this cropped up at all?
Well, according to http://www.cplusplus.com/reference/istream/istream/seekg/ it should. Perhaps some compilers aren't yet very pedantic about these small details. (In reply to Peter Barnes from comment #7) > Wrt > > > Since seekg(0) doesn't clear the eof bit (in c++99, in c++11 it does) > > Since we have --std=c++11 I'm puzzled why this cropped up at all? Pushed in 12337 e37de7f82ed2 |