valerybokov commented on pull request #107: URL: https://github.com/apache/pdfbox/pull/107#issuecomment-813298597
> > Do you want to say there is difference in memory usage for a field and local variable? But it is only one additional parameter. > > Yes I assume that. It's pushed on the stack with each call. It's twice as much parameters than before, and this in a code segment that is highly recursive. (page tree) Okay. Let use old version, but how about potential NPE in PageIterator.next (sanitizeType(next) - next variable can be null)? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
