[
https://issues.apache.org/jira/browse/LUCENE-1887?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12753381#action_12753381
]
Luis Alves commented on LUCENE-1887:
------------------------------------
I agree with you on the compatibility, I'll remove the deprecated from the
getLocalizedMessage method on
NLS class and submit again
{code}
public static String getLocalizedMessage(String key, Locale locale, Object[]
args)
{code}
I just think it is ugly having to create a Object array for every call to the
Message classes.
See Example below.
Original or with my patch
{code}
throw new ParseException(new
MessageImpl(QueryParserMessages.INVALID_SYNTAX_ESCAPE_NONE_HEX_UNICODE, c));
{code}
Example using your patch:
{code}
throw new ParseException(new
MessageImpl(QueryParserMessages.INVALID_SYNTAX_ESCAPE_NONE_HEX_UNICODE, new
Object[]{c}));
{code}
This is not a big thing for me.
If we don't care that the users of these classes in 2.9 are required to create
the object arrays for most method calls, then
I am totally ok with your patch.
We can clean the new QueryParser code, of those object arrays in 3.0.
Robert or Mark pick any of the patches, they both look good to me.
Need to go home, I'll check the issue later.
> o.a.l.messages should be moved to it's own contrib
> --------------------------------------------------
>
> Key: LUCENE-1887
> URL: https://issues.apache.org/jira/browse/LUCENE-1887
> Project: Lucene - Java
> Issue Type: Improvement
> Components: contrib/*
> Reporter: Hoss Man
> Priority: Minor
> Attachments: lucene-1877-new.patch, LUCENE-1887.patch
>
>
> contrib/queryParser contains an org.apache.lucene.messages package containing
> some generallized code that (claims in it's javadocs) is not specific to the
> queryParser.
> If this is truely general purpose code, it should probably be moved out of
> hte queryParser contrib -- either into it's own contrib, or into the core
> (it's very small)
> *EDIT:* alternate suggestion to rename package to fall under the
> o.a.l.queryParser namespace retracted due to comments in favor of
> (eventually) promoting to it's own contrib
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
