On Tue, May 09, 2006 at 02:40:52AM +0200, Michael Kerrisk wrote:
> Justin,
> 
> I'm inclined to take this page, after fixing aa few small things.
> 
> A few comments:
> 
> > \fIresponse\fP should be a \fBNULL\fP-terminated string containing a
> 
> Please: "null terminated" (NULL and '\0' are NOT the same thing.)
Good point; I still like the hyphen, though.

> > .SH BUGS
> > The \fBrpmatch\fP() implementation looks at only the first character
> > of \fIresponse\fP.  As a consequence, "nyes" returns 0, and the
> > following all return 1:
> > .RS
> > .sp
> > \*(lqyno, never; not in a million years\*(rq
> > .br
> > .TP
> > \*(lqyover my dead body\*(rq
> > .br
> > .TP
> > \*(lqywhenever hell freezes over\*(rq
> > .sp
> > .RE
> 
> My humour filter is hesitating over this for a moment...
:)  puleeaaase????  ...  edit at will.

> > It would be preferrable to accept parse input strings much more
> > strictly, for example: \fBm/^y(es?)$/i\fP and \fBm/^n(o?)$/i\fP.
> 
> What is the notation "m/^y(es?)$/i" -- why not just "/^y(es?)$/".
The /i has meaning, since "YES" is accepted; I don't much care about
the leading "m" though.

> Any suggestions about other pages whose SEE ALSO should mention this
> page?
Not really ... I'll have to think about it.

Justin


-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]

Reply via email to