Closed (won't fix)
Project:
Drupal core
Version:
x.y.z
Component:
base system
Priority:
Normal
Category:
Bug report
Assigned:
Reporter:
Created:
31 Oct 2005 at 14:59 UTC
Updated:
2 Nov 2005 at 15:38 UTC
As title, very simple patch so setting directly to ready to be committed.
| Comment | File | Size | Author |
|---|---|---|---|
| remove-empty-lines.patch | 24.19 KB | Stefan Nagtegaal |
Comments
Comment #1
Cvbge commented-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...)
Comment #2
dries commentedEeh?
Comment #3
Geary commentedThe 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.