[jira] [Commented] (OFBIZ-10430) Use `in` and spread operator in `gradlewSubprocess`

["Jacques Le Roux (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/OFBIZ-10430?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16508162#comment-16508162 ] 

Jacques Le Roux commented on OFBIZ-10430:
-----------------------------------------

Thanks Mathieu,

I like the Groovy way: +1

If nobody is against I'll commit in 1 week

> Use `in` and spread operator in `gradlewSubprocess`
> ---------------------------------------------------
>
>                 Key: OFBIZ-10430
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-10430
>             Project: OFBiz
>          Issue Type: Improvement
>            Reporter: Mathieu Lirzin
>            Priority: Minor
>             Fix For: Trunk
>
>         Attachments: OFBIZ-10430_Use-in-and-spread-operator-in-gradlewSubprocess.patch
>
>
> No functional change.  The idea of using the spread operator is to have less mutation involved.  In Groovy, using operator overloads like {{in}} is more idiomatic  than using explicitly the associated method.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)