Robert Scholte rfscholte at
Sat Jul 8 16:59:54 UTC 2017


I'm kind of surprised by the number of methods of Runtime.Version:

Based on my experience with versions I would advice to reduce the number  
of methods.

With Maven we had this interface:

However, in the end all those separate segments didn't add a thing and are  
actually causing unnecessary complex code in case we were doing other  
implementations with ArtifactVersion. ArtifactVersion is quite an  
important interface in our code, so there's no simple solution to simplify  
this mistake.

I'm really pleased that for Aether (which is now back at Maven as Artifact  
Resolver) it is all reduced to this:

This is all we need: parsing + comparing (+ #toString()). If none of the  
additional methods of Runtime.Version is used *yet*, you should consider  
to remove them.


More information about the jpms-spec-observers mailing list