according to:
http://drupal.org/coding-standards

Files should be saved as Unix filetype, not windows. Windows uses an older format (\r\n) for line endings. Unix and most modern PHP editors tend to use Unix newlines which are just "\n". This means if I (for example) edit the module and then go to do a cvs diff to make a patch, cvs thinks the entire module has changed simply because Zend Studio uses Unix Filetypes and the newlines have all changed...

Comments

m1mic’s picture

Status: Active » Needs review
StatusFileSize
new95.66 KB

This patch converts the Windows style line endings (\r\n) to the Unix style line endings (\n) on the following files:

  • INSTALL.txt
  • README.txt
  • guestbook.install
  • guestbook.module
  • po/ru.po

I applied the patch against DRUPAL-5 in my contributed modules folder (not in the Guestbook folder)

m1mic’s picture

StatusFileSize
new94.28 KB

In case anybody needs a version of the previous patch for DRUPAL-5--1-0, here it is.

sun’s picture

Title: Coding Standard: File should be Unix not MSDOS (newline issue) » Wrong file formats (Unix not MSDOS; character encoding)

Additionally, character encoding of guestbook files is Latin1 and not UTF-8.
I bet that many bug reports and support issues for guestbook are also caused by wrong file formats.
Patch in http://drupal.org/node/208389 suffers from this issue, too.

sun’s picture

Title: Wrong file formats (Unix not MSDOS; character encoding) » Wrong file formats (CRLF) and character encoding

yay! After cleaning up the issue queue, I need to bump this issue to bring it into my attention. Sorry for that. :)

sun’s picture

Status: Needs review » Fixed
Anonymous’s picture

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for two weeks with no activity.