Issue 32420 - Tools Config class should support utf8 files
Summary: Tools Config class should support utf8 files
Status: CLOSED FIXED
Alias: None
Product: App Dev
Classification: Unclassified
Component: api (show other issues)
Version: 3.3.0 or older (OOo)
Hardware: All All
: P3 Trivial
Target Milestone: ---
Assignee: hennes.rohling
QA Contact: issues@api
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2004-08-02 10:17 UTC by tino.rachui
Modified: 2013-02-24 21:08 UTC (History)
1 user (show)

See Also:
Issue Type: ENHANCEMENT
Latest Confirmation in: ---
Developer Difficulty: ---

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this issue.
Description tino.rachui 2004-08-02 10:17:15 UTC 
The Unicode 2.0 standard suggests to use a byte-order-mark at the beginning of a
file not just as a way of identifying the proper endian-ness, but to tell
whether it is a Unicode file or not. Utf8 file for instance start with 0xEF 0xBB
0xBF. The Config class stumbles  over such  utf8 files.
Comment 1 tino.rachui 2004-08-02 10:34:47 UTC 
TRA->PL: Please give me a hint when this one is fixed, then I can remove a
work-around in shell/source/tools/lngconvex which uses the Config class.
Comment 2 philipp.lohmann 2004-08-02 15:17:02 UTC 
accepted
Comment 3 liyuan 2007-06-11 07:37:49 UTC 
send patch file to pl to review.
Comment 4 philipp.lohmann 2007-06-18 12:48:18 UTC 
fixed by liyuan in CWS vcl79
Comment 5 philipp.lohmann 2007-06-18 14:15:31 UTC 
set to fixed
Comment 6 philipp.lohmann 2007-06-19 11:58:08 UTC 
please verify in CWS vcl79
Comment 7 philipp.lohmann 2007-07-02 11:03:56 UTC 
pl->hro: obr mentioned that shell/source/tools/lngconvex might be your's
nowadays ? Could you please verify this issue then ?
Comment 8 hennes.rohling 2007-07-03 11:06:09 UTC 
@pl: Check for nRead >= 3 in Line 161 in config.cxx

Verified.
Comment 9 hennes.rohling 2007-08-24 15:51:39 UTC 
Closed