The stack requirements (char message[BUFSIZE]) for this method increasing by 256 all the time seems unnecessary, a fixed string plus two numerics doesn't need 256 characters anyway, does it? Not sure it matters that much in this case since it isn't in any kind of recursive call situation.
Fixed buffers bumping up the stack size needed always make my eye twitch. Worked in compilers too long. :^( But I don't see anything logically wrong with the change. -kto On May 24, 2012, at 12:46 AM, Nils Loodin wrote: > when the 'demo' java_crw_demo (which is used by hprof) tries to mirror the > constant pool of a class, a few of the constants introduced by invokedynamic > ins't recognized. > Fairly simple fix. For my simple repro, it seems to work with this patch. > > Also slightly improved the error message for future niceness. > > Webrev here: > http://cr.openjdk.java.net/~nloodin/7143353/webrev.00/ > > Regards, > Nils Loodin
