RFR [8044342] build failure on Windows noticed with recent smartcardio fix.

[Valerie (Yu-Ching) Peng]

1) Since we are rolling back to the earlier code, how about just use the 
original code? I somehow find this easier to parse than the 2 nested 
almost-identical for-loops in the current fix.
     char* cp = spec;
     while (*cp != 0) {
         cp += (strlen(cp) + 1);
         ++cnt;
     }

2) Add back the NULL check for the malloc call, e.g.
     if (tab == NULL) {
         throwOutOfMemoryError(env, NULL);
         return NULL;
     }

Thanks,
Valerie


On 05/29/14 11:28, Ivan Gerasimov wrote:
> Hi!
>
> With the recent code modification I used a pcsc-lite specific constant 
> which is only defined on *nix platforms.
> As a result, windows builds failed. Sorry about that!
>
> Would you please review the partial backout fix: Dynamic array 
> allocation is restored.
>
> BUGURL: https://bugs.openjdk.java.net/browse/JDK-8044342
> WEBREV: http://cr.openjdk.java.net/~igerasim/8044342/0/webrev/
>
> Sincerely yours,
> Ivan