[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: Small fix of column number in parser
From: |
John W. Eaton |
Subject: |
Re: Small fix of column number in parser |
Date: |
Thu, 28 Apr 2011 02:56:44 -0400 |
On 28-Apr-2011, Daniel Kraft wrote:
| As I understand it, this should be right -- it's more or less the other
| way round, if we skip a white-space character, we increment the counter.
| If we don't have white-space, we put it back on the stream, return and
| do nothing at all to the counter. Decrementing it was precisely the
| reason for reporting "column -1" above.
OK, I understand now. I think your patch is fine.
| There was no bug report for this problem -- but of course I can always
| create one to attach a patch which fixes a bug. In that case, should I
| also post to the mailing list that a new patch is available, or can this
| be seen right from the bug-list?
There's usually no need to post a message to the list if you report a
bug in the tracker. People who care to follow the bug reports are
already subscribed to the notification list for the tracker.
jwe