On Tue, Nov 11, 2014 at 6:00 PM, Dirk Hohndel <[email protected]> wrote:

> On Tue, Nov 11, 2014 at 05:52:55PM +0800, Miika Turkia wrote:
> > On Tue, Nov 11, 2014 at 5:30 PM, Willem Ferguson <
> > [email protected]> wrote:
> >
> > > Looks excellent Miika,
> > >
> > > Small bug. At the end of each item in the second column of the Extra
> data
> > > tab there is a small unprintable character showing up as a square. I
> > > suspect that should not be?
> > >
> >
> > I am shooting in the dark but would this patch fix the issue?
>
> Why would the string be terminated by '\n' or '\r'? I must be missing
> something (need to look at that code in the larger context). Shouldn't we
> just strip all whitespace (like \r and \n)?
>

The input is read from a .txt file, that I am just guessing might have \r\n
line endings even though my sample file terminates with proper \n. The
string I am messing with here is fed to your code and should be null
terminated properly.

miika
_______________________________________________
subsurface mailing list
[email protected]
http://lists.subsurface-divelog.org/cgi-bin/mailman/listinfo/subsurface

Reply via email to