commons-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Henri Biestro (JIRA)" <j...@apache.org>
Subject [jira] [Resolved] (JEXL-213) rename JexlBuilder.loader(Charset arg) to JexlBuilder.charset(Charset arg)
Date Wed, 31 Aug 2016 10:42:20 GMT

     [ https://issues.apache.org/jira/browse/JEXL-213?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]

Henri Biestro resolved JEXL-213.
--------------------------------
       Resolution: Fixed
         Assignee: Henri Biestro
    Fix Version/s: 3.1


Add charset(Charset arg) to JexlBuilder, deprecated loader(Charset...) which is indeed a mistake
(but public API).

src/main/java/org/apache/commons/jexl3/JexlBuilder.java
Committed revision 1758578.


> rename JexlBuilder.loader(Charset arg) to JexlBuilder.charset(Charset arg)
> --------------------------------------------------------------------------
>
>                 Key: JEXL-213
>                 URL: https://issues.apache.org/jira/browse/JEXL-213
>             Project: Commons JEXL
>          Issue Type: Bug
>    Affects Versions: 3.0
>            Reporter: Dmitri Blinov
>            Assignee: Henri Biestro
>            Priority: Minor
>             Fix For: 3.1
>
>
> There seems to be a copy-paste bug in method in {{JexlBuilder}} class
> {code}
> public JexlBuilder loader(Charset arg) {
>         this.charset = arg;
>         return this;
> }
> {code}
> obviously the method should be named {{charset}}



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Mime
View raw message