| commit | 0deb4908a24c03c03cc9a7a0353f47a947648852 | |
| author | Mark Tompsett <mtompset@hotmail.com> | |
| Sun, 17 Aug 2014 22:44:04 +0000 (17 18:44 -0400) | ||
| committer | Tomas Cohen Arazi <tomascohen@gmail.com> | |
| Wed, 20 Aug 2014 20:55:50 +0000 (20 17:55 -0300) | ||
| tree | 8667489e2a9e8e022963f4cb448c2c4f0f15d650 | treesnapshot (tar.gz zip) |
| parent | cd8ca4717c8ee3055a36497db20f263832c83659 | commitdiff |
Bug 4162 - Followup: only count potential barcodes
The $lines_read++; was before the code which skipped empty
lines. By moving it after the next unless $barcode; code, the
$lines_read variable only counts potential barcodes, and does
not include empty lines.
TEST PLAN
---------
1) Apply patch.
2) Attempt a valid barcode file with empty lines.
-- The potential barcode count should exclude the empty lines.
Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Empty lines are now no longer counted as potential barcodes.
Passes tests and QA script.
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
The $lines_read++; was before the code which skipped empty
lines. By moving it after the next unless $barcode; code, the
$lines_read variable only counts potential barcodes, and does
not include empty lines.
TEST PLAN
---------
1) Apply patch.
2) Attempt a valid barcode file with empty lines.
-- The potential barcode count should exclude the empty lines.
Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Empty lines are now no longer counted as potential barcodes.
Passes tests and QA script.
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
| tools/inventory.pl | diffblobblamehistory |