You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@maven.apache.org by "Benjamin Bentmann (JIRA)" <ji...@codehaus.org> on 2008/02/19 23:36:29 UTC

[jira] Created: (MPLUGIN-76) Fixed file encoding employed by PluginXDocGenerator

Fixed file encoding employed by PluginXDocGenerator
---------------------------------------------------

                 Key: MPLUGIN-76
                 URL: http://jira.codehaus.org/browse/MPLUGIN-76
             Project: Maven 2.x Plugin Tools
          Issue Type: Bug
          Components: API
    Affects Versions: 2.3
            Reporter: Benjamin Bentmann
         Attachments: file-encoding.patch

The class currenlty uses the platform's default encoding (say ISO-8859-1) to write a XML file without a XML declaration such that readers will assume UTF-8 instead. The patch enforces UTF-8 output and (as a best practice) adds the XML declaration to the output file.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] Issue Comment Edited: (MPLUGIN-76) Fixed file encoding employed by PluginXDocGenerator

Posted by "Benjamin Bentmann (JIRA)" <ji...@codehaus.org>.
    [ http://jira.codehaus.org/browse/MPLUGIN-76?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_124395 ] 

bentmann edited comment on MPLUGIN-76 at 2/19/08 5:11 PM:
-------------------------------------------------------------------

Well, in the end, it's quit the same code, i.e. an OutputStreamWriter on top of a FileOutputStream. What I don't like about {{WriterFactory.newWriter()}} is that its method signature ony returns a {{Writer}} though it could have been declared to return {{OutputStreamWriter}} which is a little more convenient in use (especially it offers {{getEncoding()}}). For some funny thing, {{WriterFactory.newXmlWriter()}} does not return only {{Writer}}...

      was (Author: bentmann):
    Well, in the end, it's quit the same code, i.e. an OutputStreamWriter on top of a FileOutputStream. What I don't like about {{WriterFactory.newWriter()} is that its method signature ony returns a {{Writer}} though it could have been declared to return {{OutputStreamWriter}} which is a little more convenient in use (especially it offers {{getEncoding()}}). For some funny thing, {{WriterFactory.newXmlWriter()}} does not return only {{Writer}}...
  
> Fixed file encoding employed by PluginXDocGenerator
> ---------------------------------------------------
>
>                 Key: MPLUGIN-76
>                 URL: http://jira.codehaus.org/browse/MPLUGIN-76
>             Project: Maven 2.x Plugin Tools
>          Issue Type: Bug
>          Components: API
>    Affects Versions: 2.3
>            Reporter: Benjamin Bentmann
>         Attachments: file-encoding.patch
>
>
> The class currenlty uses the platform's default encoding (say ISO-8859-1) to write a XML file without a XML declaration such that readers will assume UTF-8 instead. The patch enforces UTF-8 output and (as a best practice) adds the XML declaration to the output file.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] Commented: (MPLUGIN-76) Fixed file encoding employed by PluginXDocGenerator

Posted by "Benjamin Bentmann (JIRA)" <ji...@codehaus.org>.
    [ http://jira.codehaus.org/browse/MPLUGIN-76?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_124395 ] 

Benjamin Bentmann commented on MPLUGIN-76:
------------------------------------------

Well, in the end, it's quit the same code, i.e. an OutputStreamWriter on top of a FileOutputStream. What I don't like about {{WriterFactory.newWriter()} is that its method signature ony returns a {{Writer}} though it could have been declared to return {{OutputStreamWriter}} which is a little more convenient in use (especially it offers {{getEncoding()}}). For some funny thing, {{WriterFactory.newXmlWriter()}} does not return only {{Writer}}...

> Fixed file encoding employed by PluginXDocGenerator
> ---------------------------------------------------
>
>                 Key: MPLUGIN-76
>                 URL: http://jira.codehaus.org/browse/MPLUGIN-76
>             Project: Maven 2.x Plugin Tools
>          Issue Type: Bug
>          Components: API
>    Affects Versions: 2.3
>            Reporter: Benjamin Bentmann
>         Attachments: file-encoding.patch
>
>
> The class currenlty uses the platform's default encoding (say ISO-8859-1) to write a XML file without a XML declaration such that readers will assume UTF-8 instead. The patch enforces UTF-8 output and (as a best practice) adds the XML declaration to the output file.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] Commented: (MPLUGIN-76) Fixed file encoding employed by PluginXDocGenerator

Posted by "Vincent Siveton (JIRA)" <ji...@codehaus.org>.
    [ http://jira.codehaus.org/browse/MPLUGIN-76?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_124391 ] 

Vincent Siveton commented on MPLUGIN-76:
----------------------------------------

Better should be to use WriterFactory.newWriter( file, encoding )

> Fixed file encoding employed by PluginXDocGenerator
> ---------------------------------------------------
>
>                 Key: MPLUGIN-76
>                 URL: http://jira.codehaus.org/browse/MPLUGIN-76
>             Project: Maven 2.x Plugin Tools
>          Issue Type: Bug
>          Components: API
>    Affects Versions: 2.3
>            Reporter: Benjamin Bentmann
>         Attachments: file-encoding.patch
>
>
> The class currenlty uses the platform's default encoding (say ISO-8859-1) to write a XML file without a XML declaration such that readers will assume UTF-8 instead. The patch enforces UTF-8 output and (as a best practice) adds the XML declaration to the output file.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] Closed: (MPLUGIN-76) Fixed file encoding employed by PluginXDocGenerator

Posted by "Vincent Siveton (JIRA)" <ji...@codehaus.org>.
     [ http://jira.codehaus.org/browse/MPLUGIN-76?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Vincent Siveton closed MPLUGIN-76.
----------------------------------

         Assignee: Vincent Siveton
       Resolution: Fixed
    Fix Version/s: 2.4

Patch applied

> Fixed file encoding employed by PluginXDocGenerator
> ---------------------------------------------------
>
>                 Key: MPLUGIN-76
>                 URL: http://jira.codehaus.org/browse/MPLUGIN-76
>             Project: Maven 2.x Plugin Tools
>          Issue Type: Bug
>          Components: API
>    Affects Versions: 2.3
>            Reporter: Benjamin Bentmann
>            Assignee: Vincent Siveton
>             Fix For: 2.4
>
>         Attachments: file-encoding.patch
>
>
> The class currenlty uses the platform's default encoding (say ISO-8859-1) to write a XML file without a XML declaration such that readers will assume UTF-8 instead. The patch enforces UTF-8 output and (as a best practice) adds the XML declaration to the output file.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira