As title, very simple patch so setting directly to ready to be committed.

CommentFileSizeAuthor
remove-empty-lines.patch24.19 KBStefan Nagtegaal

Comments

Cvbge’s picture

-1
Many editors (e.g. vim) add newline automatically. IIRC it's better to have it, don't remember the reasoning.
Why would you want to remove it? (And please don't tell me it's for performance...)

dries’s picture

Status: Reviewed & tested by the community » Closed (won't fix)

Eeh?

Geary’s picture

The reason for removing newlines at the end of files is to prevent spurious newlines from being inserted in the HTML output.

But a better solution is to remove the ?> at the end of each PHP file instead, as Drupal is doing now. Then it doesn't matter if there are newlines at the end.