To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=15456





------- Additional comments from [EMAIL PROTECTED] Mon Dec  3 15:43:27 +0000 
2007 -------
@mib:

I agree with your suggestion on UTF-8.  Thanks for your response.  I will revise
the patch shortly.

I have a few comments regarding the handling of iso-8859-2.

>we have to understand
that after applying the patch, documents will only loaded correctly into text
editors whose encoding is set to ISO 8859-2/Windows 1250. 

But that's the (unfortunate) design of code pages; there is no way to ensure
that a text document encoded in one encoding is displayed correctly in another
encoding, which is also why we prefer unicode today.  But that doesn't mean that
we should refuse to export such document with that encoding if the user
explicitly chooses to do so by setting the encoding in the Option dialog.

>If a document is
loaded into an editor that has a different encoding set, characters above 128
may be displayed incorrectly.

This is expected, and that's why the document includes the encoding information
so that the editor knows which codepage to use when loading.

>Using the entities here as it is the case today ensures that documents are
editable regardless of of the encoding that is set in the editor.

Don't almost all editors these days offer encoding options when opening a text
document?  I'm a bit puzzled by this response; it appears that the editor you
use doesn't seem to support anything other than ASCII range...

Or, is there any valid use case where this behavior (of encoding everything
above 128 regardless of user-selected encoding) is needed?

Thank you for your response.

Kohei

---------------------------------------------------------------------
Please do not reply to this automatically generated notification from
Issue Tracker. Please log onto the website and enter your comments.
http://qa.openoffice.org/issue_handling/project_issues.html#notification

---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]


---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to