Performance issue with the redundancy operations

[yyzdtccjdtc]

rust-csv/csv-core/src/reader.rs

Line 669 in 41c71ed

self.line += (input[nin] == b'\n') as u64;

For this line of code, it will always update the value of self.line, even if the value of it does not change. This introduces some redundant operations.

If we change the code to this, we can eliminate these redundant operations.

if input[nin] == b'\n'{
     self.line += 1;
}

According to my tests with the ema example with this application (https://github.com/greyblake/ta-rs), the average execution time decreased from 2.74s to 2.68s, which is a 1.02x speedup.

Hope this information helps!

[BurntSushi]

Interesting. Seems like the compiler should be smarter than that here. But happy to make this change. PRs are welcome.