This is the mail archive of the java-patches@gcc.gnu.org mailing list for the Java project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: RFC: AccessController merge


Andrew Haley wrote:
> I'm a bit concerned about efficiency.  For one example: OK, we walk
> the stack -- we have to do that -- but we create a string for every
> method name , and the only purpose of doing so AFAICS is to compare
> it with "doPrivileged".  We then throw away all of those strings we
> so laboriously constructed...

I see what you mean.  Another thing is that if there is a doPrivileged
there then a chunk of the generated stack trace will be ignored.  I
guess Casey wanted the code he put in stacktrace.cc to be generic.

> I suppose this should go in, but we should study profile data and
> make efficiency improvments if required before release.

Should I commit now, or wait for other comments?

Cheers,
Gary


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]