On 08/08/2012 01:33, David Holmes wrote:
On 8/08/2012 2:40 AM, Xueming Shen wrote:

Andrew,

Since we are here:-) are we also supposed to "free" the old_temp at #250
and old_temp and old_ev at the end?

No. They are aliases for temp and encoding_variant, which are freed at the end. There are only ever at most two things to free.

Andrew: this is fine by me, but needs TL approval (Alan?)
You are a reviewer on the jdk8 project so Andrew should be all set. In any case, the update looks good to me too.

-Alan.

Reply via email to