[jira] [Commented] (FUNCTOR-10) throw NullPointerException for illegal null values
[ https://issues.apache.org/jira/browse/FUNCTOR-10?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13191473#comment-13191473 ] Matt Benson commented on FUNCTOR-10: hmm, that was actually supposed to be {{org.apache.commons.functor.\_lang3.\_\_Validate}} > throw NullPointerException for illegal null values > -- > > Key: FUNCTOR-10 > URL: https://issues.apache.org/jira/browse/FUNCTOR-10 > Project: Commons Functor > Issue Type: Improvement >Reporter: Emmanuel Bourg >Assignee: Matt Benson > > for better alignment with JSE, {{[lang]}}, etc. Currently > {{IllegalArgumentException}} is thrown. > Per http://markmail.org/message/ythw55yad5lrvqrj -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] [Commented] (FUNCTOR-10) throw NullPointerException for illegal null values
[ https://issues.apache.org/jira/browse/FUNCTOR-10?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13191468#comment-13191468 ] Matt Benson commented on FUNCTOR-10: Yes; I would imagine that the IDEs would be fine to import if a user actually tried referencing the classes by these (albeit lightly) obfuscated names, but why would the user do this? We've accomplished the goal of protecting the user from accidentally importing these. It might be nice to make the relocator pluggable for the shade plugin, to make this easier, but I think at Commons we've gone long enough reinventing the wheel in every component just to avoid dependencies. Jar shading gives us the best of both worlds. > throw NullPointerException for illegal null values > -- > > Key: FUNCTOR-10 > URL: https://issues.apache.org/jira/browse/FUNCTOR-10 > Project: Commons Functor > Issue Type: Improvement >Reporter: Emmanuel Bourg >Assignee: Matt Benson > > for better alignment with JSE, {{[lang]}}, etc. Currently > {{IllegalArgumentException}} is thrown. > Per http://markmail.org/message/ythw55yad5lrvqrj -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] [Commented] (FUNCTOR-10) throw NullPointerException for illegal null values
[ https://issues.apache.org/jira/browse/FUNCTOR-10?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13191462#comment-13191462 ] Simone Tripodi commented on FUNCTOR-10: --- nice to hear the trick worked! The nicer thing is IMHO that names are still valid Java identifier parts, but not all IDEs are good on recognizing them :P Standard Eclipse, for example, doesn't resolve the Guice shaded internals, while Sonatype's MavenIDE (an Eclipse mod) does. > throw NullPointerException for illegal null values > -- > > Key: FUNCTOR-10 > URL: https://issues.apache.org/jira/browse/FUNCTOR-10 > Project: Commons Functor > Issue Type: Improvement >Reporter: Emmanuel Bourg >Assignee: Matt Benson > > for better alignment with JSE, {{[lang]}}, etc. Currently > {{IllegalArgumentException}} is thrown. > Per http://markmail.org/message/ythw55yad5lrvqrj -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] [Commented] (FUNCTOR-10) throw NullPointerException for illegal null values
[ https://issues.apache.org/jira/browse/FUNCTOR-10?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13191456#comment-13191456 ] Matt Benson commented on FUNCTOR-10: Thanks for the hint, Simo! Naming e.g. {{org.apache.commons.lang3.Validate}} to {{org.apache.commons.functor._lang3.__Validate}} and confirmed that the resulting jar is fully functional. > throw NullPointerException for illegal null values > -- > > Key: FUNCTOR-10 > URL: https://issues.apache.org/jira/browse/FUNCTOR-10 > Project: Commons Functor > Issue Type: Improvement >Reporter: Emmanuel Bourg >Assignee: Matt Benson > > for better alignment with JSE, {{[lang]}}, etc. Currently > {{IllegalArgumentException}} is thrown. > Per http://markmail.org/message/ythw55yad5lrvqrj -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] [Commented] (FUNCTOR-10) throw NullPointerException for illegal null values
[ https://issues.apache.org/jira/browse/FUNCTOR-10?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13191119#comment-13191119 ] Simone Tripodi commented on FUNCTOR-10: --- We could shade the {{Validate}} class by renaming it in a way some IDEs (i.e. Eclipse) is not able to resolve; Google Guice's folks use to do it by shading the classes with {{$}} as first class name char, it would look like: {{org.apache.commons.lang.$Validate}} > throw NullPointerException for illegal null values > -- > > Key: FUNCTOR-10 > URL: https://issues.apache.org/jira/browse/FUNCTOR-10 > Project: Commons Functor > Issue Type: Improvement >Reporter: Emmanuel Bourg >Assignee: Matt Benson > > for better alignment with JSE, {{[lang]}}, etc. Currently > {{IllegalArgumentException}} is thrown. > Per http://markmail.org/message/ythw55yad5lrvqrj -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] [Commented] (FUNCTOR-10) throw NullPointerException for illegal null values
[ https://issues.apache.org/jira/browse/FUNCTOR-10?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13190940#comment-13190940 ] Emmanuel Bourg commented on FUNCTOR-10: --- I tend to think that convenience for the end user is more important than convenience for the developer. In this case the code saved by the Validate class is marginal, I would rather code the null check manually. > throw NullPointerException for illegal null values > -- > > Key: FUNCTOR-10 > URL: https://issues.apache.org/jira/browse/FUNCTOR-10 > Project: Commons Functor > Issue Type: Improvement >Reporter: Emmanuel Bourg >Assignee: Matt Benson > > for better alignment with JSE, {{[lang]}}, etc. Currently > {{IllegalArgumentException}} is thrown. > Per http://markmail.org/message/ythw55yad5lrvqrj -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] [Commented] (FUNCTOR-10) throw NullPointerException for illegal null values
[ https://issues.apache.org/jira/browse/FUNCTOR-10?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13190889#comment-13190889 ] Matt Benson commented on FUNCTOR-10: Technically, yes, but should we avoid an available convenience in cases like this? > throw NullPointerException for illegal null values > -- > > Key: FUNCTOR-10 > URL: https://issues.apache.org/jira/browse/FUNCTOR-10 > Project: Commons Functor > Issue Type: Improvement >Reporter: Emmanuel Bourg >Assignee: Matt Benson > > for better alignment with JSE, {{[lang]}}, etc. Currently > {{IllegalArgumentException}} is thrown. > Per http://markmail.org/message/ythw55yad5lrvqrj -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] [Commented] (FUNCTOR-10) throw NullPointerException for illegal null values
[ https://issues.apache.org/jira/browse/FUNCTOR-10?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13190803#comment-13190803 ] Emmanuel Bourg commented on FUNCTOR-10: --- Doesn't this create a risk that people will use the Validate class shaded into [functor]? Auto import in modern IDEs is not always smart :) > throw NullPointerException for illegal null values > -- > > Key: FUNCTOR-10 > URL: https://issues.apache.org/jira/browse/FUNCTOR-10 > Project: Commons Functor > Issue Type: Improvement >Reporter: Emmanuel Bourg >Assignee: Matt Benson > > for better alignment with JSE, {{[lang]}}, etc. Currently > {{IllegalArgumentException}} is thrown. > Per http://markmail.org/message/ythw55yad5lrvqrj -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] [Commented] (FUNCTOR-10) throw NullPointerException for illegal null values
[ https://issues.apache.org/jira/browse/FUNCTOR-10?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13190530#comment-13190530 ] Matt Benson commented on FUNCTOR-10: using lang3 Validate.notNull + maven-shade-plugin > throw NullPointerException for illegal null values > -- > > Key: FUNCTOR-10 > URL: https://issues.apache.org/jira/browse/FUNCTOR-10 > Project: Commons Functor > Issue Type: Improvement >Reporter: Emmanuel Bourg >Assignee: Matt Benson > > for better alignment with JSE, {{[lang]}}, etc. Currently > {{IllegalArgumentException}} is thrown. > Per http://markmail.org/message/ythw55yad5lrvqrj -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa For more information on JIRA, see: http://www.atlassian.com/software/jira