Request for review- RFE 8005716
Alan Bateman
Alan.Bateman at oracle.com
Tue Mar 12 12:36:08 UTC 2013
On 11/03/2013 15:37, BILL PITTORE wrote:
> On 3/11/2013 9:40 AM, Alan Bateman wrote:
>> On 08/03/2013 02:22, BILL PITTORE wrote:
>>> Moved the string allocation into buildJniFunctionName as Alan
>>> suggested. Built and tested on windows and linux. Updated the webrev:
>>>
>>> http://cr.openjdk.java.net/~bpittore/8005716/jdk-webrev.02/
>>>
>>>
>>> bill
>> I see this updates the method descriptions to take on board Jeremy's
>> comment on the possibility of the library being statically linked
>> with the main executable with or without the VM. To be complete, I
>> think this will require an update to the UnstatisfiedLinkError
>> description too.
>>
>> Thanks for moving the sizing/allocation of the function name into
>> buildJniFunctionName as that is cleaner and safer. Is FILENAME_MAX
>> really the right limit to impose?
> Maybe JVM_MAXPATHLEN is the better choice.
As it's a symbol rather than a file path then it's probably not right
either. I guess the question is whether a limit needs to be checked here
or not.
-Alan.
More information about the core-libs-dev
mailing list