Hello.

Per Jessen wrote:
> let me know - it works here.
Good :)

> If you don't like the added
> debug output, well ... :-)
I don't like it because you
have a single string duplicating
many times. Just putting it
right before the switch() would
be OKey I think. Also for 'A'
you are adding that string after
"ch=*(in++);" and for M - before,
which seems buggy.
Decorating the bugfix with
a debug changes is not a problem
in general but it should be more
accurate.
But anyway this doesn't as matter
as the change you propose for
unicode_utils.c, which, unless
you can justify it of course,
is not recommended to apply at
all.
-
To unsubscribe from this list: send the line "unsubscribe linux-msdos" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to