http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10625 --- Comment #9 from Kyle M Hall <kyle@bywatersolutions.com> ---
Since the use of File::Slurp means loading the entire input barcode file into memory, this patch also changes the performance characters of the inventory tool when processing very large files. Admittedly, this may not be a big deal in practice, but I would appreciate confirmation as to whether there are barcode scanners or other utilities that are *actually* producing files that use \r as a delimiter.
I agree, that using File::Slurp was also a concern of mine, but one would need to upload a barcode file with hundreds of thousands of barcodes to have a plain text file of any appreciable size, right? I can confirm at least one library is having this issue in practice, as the example file of barcodes was provided by the library having this issue. -- You are receiving this mail because: You are watching all bug changes.