Re: [9fans] ls

Re: [9fans] ls

[rog]

> we went back and forth on this topic and eventually
> settled on quoting by default because it was just too nice.

acme mail should probably do the same thing when
generating "cp" commands for message components.

the fix is just:
		Bprint(w->body, "\tcp %s%sbody%s %q\n", rootdir, name, ext(m->type), dest);

in /acme/mail/src/mesg.c:/^mimedisplay

and, in /acme/mail/src/mail.c:

int isrcspecial(int c)
{
	char *s;
	for (s = "`^#*[]=|\\?${}()'<>&;"; *s; s++)
		if (*s == c)
			return 1;
	return 0;
}

in threadmain():

	doquote = isrcspecial;
	quotefmtinstall();

Re: [9fans] ls

[Dan Cross]

> and, in /acme/mail/src/mail.c:
>
> int isrcspecial(int c)
> {
> 	char *s;
> 	for (s = "`^#*[]=|\\?${}()'<>&;"; *s; s++)
> 		if (*s == c)
> 			return 1;
> 	return 0;
> }

Yuck.  I feel compelled to ask again: can we please put this in a
library?

	- Dan C.

Re: [9fans] ls

[Dan Cross]

Oh, man.  I hate it when I post something about a thing that just got
changed.  Russ, you're simply too quick for me.

	- Dan C.

Re: [9fans] ls

[Russ Cox]

cross@math.psu.edu:
> This seems like a generally useful thing (that is, quoting on all of
> the above characters), and indeed when I wrote walk I was somewhat
> dismayed that I had to pull something that seemed so basic and
> universal out of another program.  Even then I wasn't sure that I got
> the entire list right, and indeed, I didn't.  Any chance of putting the
> obvious lsquote() like function in libc?

It's now in the library as needsrcquote.  Ls has been updated
(and fixed! it wasn't getting \ (er, \\) right.)

rog@vitanuova.com:
>> we went back and forth on this topic and eventually
>> settled on quoting by default because it was just too nice.
>
> acme mail should probably do the same thing when
> generating "cp" commands for message components.

Acme Mail has also been updated.

Russ