[
https://issues.apache.org/jira/browse/CRYPTO-103?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15367423#comment-15367423
]
Sebb commented on CRYPTO-103:
-----------------------------
I was forgetting that the version is also used to load the library.
But I think the method does not belong in NativeCodeLoader.
Also it would be useful to have that info available (CRYPTO-104).
A better solution might be to put the code into the Crypto class (*) and then
use it from the NativeCodeLoader.
(*) using a less ambiguous name, e.g. getComponentVersion()
> NativeCodeLoader.getVersion() seems badly named
> -----------------------------------------------
>
> Key: CRYPTO-103
> URL: https://issues.apache.org/jira/browse/CRYPTO-103
> Project: Commons Crypto
> Issue Type: Bug
> Reporter: Sebb
>
> The NativeCodeLoader.getVersion() method seems badly named, as it has nothing
> to do with the native code that has been loaded.
> It only relates to the version of the Java code in the jar.
> I would expect the method to return the version details from the library
> itself.
> If there is a need to return the Java code version, it should be done from a
> method in another class, e.g. Utils or Crypto. Note: also the extraction
> should be done once, e.g. using IODH or using a synchronised/volatile lock.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
