Revert "getpart.pm: Strip carriage returns to fix Windows support"
authorMarc Hoersken <info@marc-hoersken.de>
Sat, 6 Apr 2013 22:28:15 +0000 (00:28 +0200)
committerMarc Hoersken <info@marc-hoersken.de>
Sat, 6 Apr 2013 22:28:15 +0000 (00:28 +0200)
This reverts commit e51b23c925a2721cf7c29b2b376d3d8903cfb067.
As discussed on the mailinglist, this was not the correct approach.

tests/getpart.pm

index 2b4abf3..92a2378 100644 (file)
@@ -213,10 +213,9 @@ sub compareparts {
  # we cannot compare arrays index per index since with the base64 chunks,
  # they may not be "evenly" distributed
 
- if($^O eq "MSWin32" || $^O eq "msys") {
-     $first =~ s/\r\n/\n/g;
-     $second =~ s/\r\n/\n/g;
- }
+ # NOTE: this no longer strips off carriage returns from the arrays. Is that
+ # really necessary? It ruins the testing of newlines. I believe it was once
+ # added to enable tests on win32.
 
  if($first ne $second) {
      return 1;