You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@commons.apache.org by "Henri Biestro (JIRA)" <ji...@apache.org> on 2016/08/31 10:42:20 UTC

[jira] [Resolved] (JEXL-213) rename JexlBuilder.loader(Charset arg) to JexlBuilder.charset(Charset arg)

     [ 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)