sschwieb commented on pull request #127: URL: https://github.com/apache/pdfbox/pull/127#issuecomment-898964813
> What is the cache for? To save the time to call "new Area(path)" ? Oh sorry, I missed that question. But yes, that was the motivation. The `Area` constructor calls `AreaOp.calculate`, which calls `AreaOp.pruneEdges`, which can be very slow (and even cause that OOM error). So the idea was to optimise for that. But feel free to ignore the cache - I could do some benchmarking with & without it, to figure out how big the impact really is (and create a separate PR if it turns out to make a difference). -- 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. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
