Filter special characters from PS output. Fix 1605

Filter special characters from PS output. Fix 1605

[Keith, 2011-04-12 07:04:42]

This solves the critical bug,
but I would like a future patch to replace this with a mapping to replace ( by
\( etc.

(string-delete has arguments in the order opposite to Scheme documentation to
compensate for a bug in Guile, fixed in Guile 1.9.)

[Valentin Villenave, 2011-04-12 07:30:09]

On 2011/04/12 07:04:42, Keith wrote:
> This solves the critical bug,
> but I would like a future patch to replace this with a mapping to replace ( by
> \( etc.
> 
> (string-delete has arguments in the order opposite to Scheme documentation to
> compensate for a bug in Guile, fixed in Guile 1.9.)

Don't we risk losing data because of this? e.g. a piece whose title would be
something like:
\header {
  title = "Allegro"
  subtitle = "(A Little Night Music \ Mozart)"
}

[dak, 2011-04-12 07:50:45]

http://codereview.appspot.com/4382052/diff/1/scm/framework-ps.scm
File scm/framework-ps.scm (right):

http://codereview.appspot.com/4382052/diff/1/scm/framework-ps.scm#newcode412
scm/framework-ps.scm:412: (markup->string val) illegals)))))
Apart from creating a new char set on every round, this seems far too adhoc to 
me.

I think one should define a function ps-quote for quoting PostScript strings (a
similar function will be useful for every other framework) and make sure that
every user-defined string is passed through it before reaching PostScript.

[Bertrand Bordage, 2011-04-12 10:20:07]

I made another patch here :
http://codereview.appspot.com/4377054/

[Keith, 2011-04-12 17:17:03]

On 2011/04/12 07:30:09, Valentin Villenave wrote:
> Don't we risk losing data because of this? 
Yes, but only in the PDF information fields, the printed words are unchanged.
But the correct solution is...

On 2011/04/12 07:50:45, dak wrote:
> I think one should define a function ps-quote

On 2011/04/12 10:20:07, Bertrand Bordage wrote:
> I made another patch here :
> http://codereview.appspot.com/4377054/

Nice. I'm closing this one.