Re: Discarding non-ASCII input

[Ingo Schwarze <schwarze@usta.de>]

Hi Joerg,

Joerg Sonnenberger wrote on Sun, Feb 05, 2012 at 12:58:03PM +0100:
> On Sun, Feb 05, 2012 at 11:17:44AM +0100, Ingo Schwarze wrote:

>> You might also wish to replace the "ignoring byte" by NULL
>> when changing this.

> If you mean \0, that wouldn't help too much, since it still doesn't
> count as argument. The idea is to translate something that most likely
> is an argument into something that preserves this.

Sorry for being too terse and not clear; i just meant the error
message a few lines above your change, as in the following UNTESTED
patch, to be added to what you already have.

I'm fine with the '?' you propose and don't want to change that
to '\0'.

Thanks,
  Ingo


Index: read.c
===================================================================
RCS file: /cvs/src/usr.bin/mandoc/read.c,v
retrieving revision 1.5
diff -u -p -r1.5 read.c
--- read.c	5 Nov 2011 16:02:18 -0000	1.5
+++ read.c	5 Feb 2012 12:02:04 -0000
@@ -325,7 +325,7 @@ mparse_buf_r(struct mparse *curp, struct
 			if ( ! (isascii(c) && 
 					(isgraph(c) || isblank(c)))) {
 				mandoc_msg(MANDOCERR_BADCHAR, curp,
-						curp->line, pos, "ignoring byte");
+						curp->line, pos, NULL);
 				i++;
 				continue;
 			}
--
 To unsubscribe send an email to discuss+unsubscribe@mdocml.bsd.lv