search

bitextor / pdf-extract

PDF parser and converter to HTML
GNU General Public License v3.0
83 stars 14 forks source link

ArrayList appears to exist solely to have its minimum taken #7

Closed kpu closed 5 years ago

kpu commented 5 years ago

The objlinetop object appears to exist as an ArrayList solely so that later code can take the minimum value.
https://github.com/bitextor/pdf-extract/blob/c1d9f74b47d735534eb4baed6b61713159f21193/PDFExtract/appClientModule/com/java/app/PDFExtract.java#L942 Replace with a double and take the minimum on the fly. Same for many of the other objects in this function.

dionwiggins commented 5 years ago

Data is no longer kept in the collection as part of optimizations. The rewrite has removed this issue. Min value is calculated on the fly.