Jonathan Gibbons wrote: > Andrew Haley wrote: >> I'm going to change langtools/jaxp/jaxws to >> >> ifeq ($(DEBUG_CLASSFILES), true) >> ANT_OPTIONS += -Djavac.debug=true >> ANT_OPTIONS += -Djavac.debuglevel=source,lines,vars >> endif > > Yes. If all you want to do is to provide a consistent way of enabling > all debug info in classfiles, then I agree with this proposed change. > I'm sorry, I misread the earlier email and confused Kelly's proposed > change with your actual changeset.
I cannot tell whether this reply from Jonathan is sufficient permission for me to commit this change. I guess it is, but as this is my first commit I'm not sure exactly what the policy is. I know that my previous patch somewhat broke debuginfo, so I want to fix it ASAP, but I don't want to break approval policy. Damned if I do, and damned if I don't... Andrew.
