[tools-discuss] Re: webrev updates

2006-09-13 Thread Stacey Marshall
Hi Dan, I have given the webrev a cursory review and have not noticed anything that looks broken. But as a previous contributor, reviewer and a user I have some nits: The changes add to the mishmash of ksh vie sh functionality. For example "[[" vie "[" for test and "print" vie "echo" for output

Re: [tools-discuss] Re: webrev updates

2006-07-26 Thread Dan Price
On Wed 26 Jul 2006 at 03:09PM, Stacey Marshall wrote: > Hmm, > > from home I found my suggested fix for the Next Prev. buttons did not work > and I have thus had > to revert back to: > > parent.lhs.location.replace(parent.lhs.location.pathname + "#" + num); > parent.rhs.location.repl

[tools-discuss] Re: webrev updates

2006-07-26 Thread Stacey Marshall
Hmm, from home I found my suggested fix for the Next Prev. buttons did not work and I have thus had to revert back to: parent.lhs.location.replace(parent.lhs.location.pathname + "#" + num); parent.rhs.location.replace(parent.rhs.location.pathname + "#" + num); To have both the b

[tools-discuss] Re: webrev updates

2006-07-26 Thread Stacey Marshall
Hi Daniel, Perhaps others have commented, I don't see the replies on the thread! Certainly like the idea of placing the specific SCCS comments at the top of each page and providing the PDF file (something I create separately). The idea of creating a patch is novel too. With regard to the 'Fra