commons-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From sebb <seb...@gmail.com>
Subject Re: svn commit: r835458 - in /commons/proper/jexl/trunk: src/main/java/org/apache/commons/jexl/ src/main/java/org/apache/commons/jexl/scripting/ src/test/java/org/apache/commons/jexl/ xdocs/
Date Thu, 12 Nov 2009 23:05:51 GMT
On 12/11/2009, Henrib <hbiestro@gmail.com> wrote:
>
>  Given JEXL usage - aka embedded in something else-, Runtime exceptions are
>  easier to work with and give more choice. JexlException and
>  UnifiedJEXL.Exception are already Runtime exceptions and are sufficient;
>  having a checked Exception for parsing does not bring any value and was
>  inconsistent with the approach of having a minimal footprint.
>
>  As for the process itself, it just seemed given the size of the JEXL
>  community that such a change would not foster much interest as a discussion
>  (using JEXL-61 as an example-). My bad.

I saw JEXL-61, and it seemed OK, so did not bother commenting.

>  I'll soon try to cut the RC1 so if you've got any (other) strong concerns
>  about what has been done so far, please tell me now.

I can't think of anything offhand just at the moment (I've just fixed
the typos introduced in r834535).

>  sebb-2-2 wrote:
>  >
>  > On 12/11/2009, henrib@apache.org <henrib@apache.org> wrote:
>  >> Author: henrib
>  >>  Date: Thu Nov 12 17:22:06 2009
>  >>  New Revision: 835458
>  >>
>  >>  URL: http://svn.apache.org/viewvc?rev=835458&view=rev
>  >>  Log:
>  >>  Changed exception handling; ParseException is no longer exposed through
>  >> public APIs, only JexlException and UnifiedJEXL.Exception are used, the
>  >> API now only uses unchecked exceptions.
>  >
>  > What's the rationale for this change?
>  >
>  > It may be a good idea (or it may not); either way I think it should
>  > have been discussed on the dev list first and/or via a JIRA
>  > enhancement request.
>  >
>  > S///
>  >
>
> > ---------------------------------------------------------------------
>  > To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
>  > For additional commands, e-mail: dev-help@commons.apache.org
>  >
>  >
>  >
>
>
>  --
>  View this message in context: http://old.nabble.com/svn-commit%3A-r835458---in--commons-proper-jexl-trunk%3A-src-main-java-org-apache-commons-jexl--src-main-java-org-apache-commons-jexl-scripting--src-test-java-org-apache-commons-jexl--xdocs--tp26322698p26325285.html
>  Sent from the Commons - Dev mailing list archive at Nabble.com.
>
>
>  ---------------------------------------------------------------------
>  To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
>  For additional commands, e-mail: dev-help@commons.apache.org
>
>

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
For additional commands, e-mail: dev-help@commons.apache.org


Mime
View raw message