struts-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Chris Pratt <thechrispr...@gmail.com>
Subject Re: [GitHub] struts pull request: WW-4590 - Allow to use multiple names in resu...
Date Tue, 19 Jan 2016 20:49:04 GMT
I can't see a reason not to support both.
  (*Chris*)

On Tue, Jan 19, 2016 at 12:06 PM, swiftelan <git@git.apache.org> wrote:

> Github user swiftelan commented on the pull request:
>
>     https://github.com/apache/struts/pull/75#issuecomment-172970495
>
>     @aleksandr-m
>
>     An annotation attribute that is defined as an array does not have to
> have the curly braces to define a single value for that attribute.
>
>     Basing a Java API for configuration on XML syntax is a poor
> implementation. Why not support the entire XML configuration syntax in the
> annotation? Java has APIs to express a collection of items. It is much
> easier to utilize an array than a comma separated value attribute. The
> array allows the developer to clearly express their intent. Utilizing an
> array does not require parsing when the configuration value is accessed.
>
>
> ---
> If your project is set up for it, you can reply to this email and have your
> reply appear on GitHub as well. If your project does not have this feature
> enabled and wishes so, or if the feature is enabled but not working, please
> contact infrastructure at infrastructure@apache.org or file a JIRA ticket
> with INFRA.
> ---
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@struts.apache.org
> For additional commands, e-mail: dev-help@struts.apache.org
>
>

Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message