Bug 3433

Summary: [kpdf] Use C locale when printing PS
Product: poppler Reporter: Kristian Høgsberg <krh>
Component: generalAssignee: Kristian Høgsberg <krh>
Status: RESOLVED FIXED QA Contact:
Severity: normal    
Priority: high CC: m_kretzschmar
Version: unspecified   
Hardware: x86 (IA32)   
OS: Linux (All)   
Whiteboard:
i915 platform: i915 features:
Bug Depends on:    
Bug Blocks: 3427    
Attachments: Patch to change numeric locale while printing.
Illustrates other LC_NUMERIC problems that make the printing patch useless for now

Description Kristian Høgsberg 2005-05-30 04:01:10 UTC
PS output in some locales because 1.23 becomes 1,23.
Comment 1 Kristian Høgsberg 2005-05-30 04:02:43 UTC
Created attachment 2797 [details] [review]
Patch to change numeric locale while printing.
Comment 2 Martin Kretzschmar 2005-06-06 07:58:00 UTC
Created attachment 2834 [details] [review]
Illustrates other LC_NUMERIC problems that make the printing patch useless for now

These are other places where xpdf code switches to LC_NUMERIC to C and tries to
switch back. The patch fixes those to actually switch back. Please don't apply
it as is.

The printing patch is not necessary ATM because as soon as we read "PDF-1.x" we
switch to C (and fail to switch back).
Comment 3 Kristian Høgsberg 2005-06-06 08:37:03 UTC
What I'd like to do for printing is to change PSOutputDev::writePSFmt() to
special case %g in the format string and print those in a local independent way.
 I fixed a similar problem in the cairo pdf backend recently, see
_cairo_output_stream_vprintf in:

  http://cvs.freedesktop.org/cairo/cairo/src/cairo-output-stream.c?view=markup

But we need to fix the few other places where we're parsing floats as well of
course.
Comment 4 Albert Astals Cid 2010-02-09 14:38:19 UTC
kpdf died ages ago

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.