Patch for Review: _Jv_TempUTFString + JV_TEMP_UTF_STRING

Bryce McKinlay bryce@mckinlay.net.nz
Fri Aug 15 04:37:00 GMT 2003


On Friday, Aug 15, 2003, at 10:06 Pacific/Auckland, Mohan Embar wrote:

> Will the nth time be a charm?
>
> Here is the patch, rewritten to use the existing CNI functions.
> To recap this supersedes and nullifies the cni.h + gcj.texi
> patches as well as all other jvm.h patches.

I'd still like to see it use JvAllocBytes instead of _Jv_Malloc. That 
way you can get rid of the destructor and the "heapAllocated" field. 
Even if it were slower, which it isn't, it will only be used in the 
rare cases where the string is > 256 bytes.

Apart from that, its ok by me.

Regards

Bryce.



More information about the Java-patches mailing list