search

cowtowncoder / java-merge-sort

Basic stand-alone disk-based N-way merge sort component for Java
Apache License 2.0
84 stars 31 forks source link

Windows CRLF bug fix, renamed _skipCR to _skipLF to reflect the byte the... #7

Closed ndikan closed 11 years ago

ndikan commented 11 years ago

... method is skipping, not the byte that indicates it needs to be called.

This is a proposed fix for Issue 6

cowtowncoder commented 11 years ago

Very good catch & excellent point wrt bad name. Should have added unit tests for this originally; I develop on unix so I didn't observe the issue, but should definitely test for it. Thank you for the patch, will merge it.

ndikan commented 11 years ago

Great! And thanks for the excellent library!

cowtowncoder commented 11 years ago

You are welcome. And once again thank you for the patch!