> Thanks, Miguel. I've found two instances in the 10.2 code where this
> was causing null pointer errors (one in Isosurface and one in Normix).

oops :-(

> I've fixed it in 10.x.10, although I think I still have to add
> synchronization to the methods that create the trouble-causing arrays.
>
> If I have this right, the problem is that you cannot:
>
>    static int[] myarray = new int[128];
>
> without first checking to see if the array already exists, and that
> check should be within a synched method. Yes?

This is OK. myarray will only be initialized when the class is first loaded.

> Do you want to do it yourself in 10.2/10.3? I guess I'd appreciate it
> if you took a look, since you know that g3d code a lot better than I
> do, and that seems to be where there problem is mainly. I looked at
> Colix, but I think those are all carefully synched already. But there
> may be other places, like the geodesics, where that isn't the case.
> Could you look?

Yes, I'll take a look.


Miguel



_______________________________________________
Jmol-developers mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/jmol-developers

Reply via email to