Re: Patch: FYI: javah, volatile, and CNI

[David Daney <ddaney at avtrex dot com>]

Tom Tromey wrote:
> > > > > > "David" == David Daney <ddaney@avtrex.com> writes:
>
> > > This fixes the 'volatile' problem that Andrew pointed out.
> > > I just put this in Classpath, the libjava change also updates the .h
> > > files.
>
> David> Perhaps, but I am getting these unexpected changes from 'svn st' with
> David> a java-maintainer-mode build:
>
> The files look ok to me.  I just updated, 'svn status' claims I have
> no diffs, and I spot-checked a file:
>
>     opsy. grep volatile java/lang/SecurityManager.h
>       static ::java::lang::SecurityManager * volatile current;
>
> So ... did you rebuild using the latest gjavah, i.e., the one in svn?
>
> I have my build set up to use the gjavah classes in $srcdir via a
> bootstrap JVM.  That way gjavah changes are immediately put into use.

As you suggested, I was using the wrong gjavah.  I will rebuild with the 
new one and see what happens.

David Daney