git.net

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [1/2] ant git commit: Deprecate CollectionUtils and Enumerations; reduce explicit use of Enumeration


2018-05-20 21:05 GMT+02:00 Gintautas Grigelionis <g.grigelionis@xxxxxxxxx>:

> 2018-05-18 16:11 GMT+02:00 Johan Corveleyn <jcorvel@xxxxxxxxx>:
>
>> 2) There is clearly no consensus here on the merits / desirability of
>> these bulk changes. Gintas, I understand your desire to move things
>> along, and you clearly have an opinion about it, but IMHO you really
>> need to stop doing any more bulk changes as long as there is no
>> consensus.
>>
>
> This time, I tried to start a discussion first, and I got no response.
> There were attempts to modernize the codebase about a year ago. AFAICS
> there was no discussion then.
> What was that saying: "don't vote, don't complain"? I'd like some
> discussion, that's all.
>
> I hear that the consensus is "new stuff is for new code, or at least
> whenever one happens to change old code fixing the bugs".
> I'm not quite sure whether that's a good idea, we should at least try to
> achieve some consistency
> -- and not necessarily sticking to the bottom line.
> Or maybe I'm too old school seeing major version changes as necessitating
> a porting effort ;-)
>

Just another .02€ -- if we agree on porting guidelines, there's no need to
squabble about "personal preferences".
At the moment, the only "global" issues I see are "informally deprecated"
JRE APIs (Vector, Hashtable, Stack,...) and the use of streams/lambdas.

Gintas


( ! ) Warning: include(msgfooter.php): failed to open stream: No such file or directory in /var/www/git/apache-ant-development/msg01962.html on line 100
Call Stack
#TimeMemoryFunctionLocation
10.0007358376{main}( ).../msg01962.html:0

( ! ) Warning: include(): Failed opening 'msgfooter.php' for inclusion (include_path='.:/var/www/git') in /var/www/git/apache-ant-development/msg01962.html on line 100
Call Stack
#TimeMemoryFunctionLocation
10.0007358376{main}( ).../msg01962.html:0