>>>>> "Angus" == Angus Leeming <[EMAIL PROTECTED]> writes:

Angus> It is perfectly clear. But 3/ doesn't work out of the box ATM.

Angus> \viewer "pdf" "\"C:\Program Files\Adobe\Acrobat
Angus> 7.0\Reader\AcroRd32.exe\" $$i"

Angus> So, at the very least something like this would be needed.
Angus> Frankly, it makes me feel very nervous.

Angus> - if (lexrc.next()) { 
Angus> + if (lexrc.next(true)) 

This seems pretty reasonable.

Angus> - os << "\\viewer \"" << cit->name() << "\" \"" 
Angus> -    << cit->viewer() << "\"\n"; 
Angus> + os << "\\viewer \\\"" << cit->name() << "\\\" \"" 
Angus> +    << cit->viewer() << "\\\"\n";

This seems very strange to me. Shouldn't it be:

+ os << "\\viewer \\\"" << cit->name() << "\" \"" 
+    << subst(cit->viewer(), "\"", "\\\"") << "\"\n";

I understand this can make you nervous, but we do have a problem with
double quotes in command names.

JMarc

Reply via email to