User login

Ask Agaric: Whitespace at the end of lines in a Drupal module or theme?

Is this annoying or a useful feature. My IDE removes trailing whitespace. The disadvantage is that it takes you more time to see the real changes/fixes.

It is a good thing that your editor deletes whitespace at the end of lines, that conforms to coding standards. While your making small typo fixes like that, you should fix the indentation also; I see some things where you removed space at the end but the indentation was left at far too much; each progressive indent should be two spaces (and no tabs used at all).

So while fixing typos, please see:

http://drupal.org/coding-standards

(For committing to Drupal core, however, note that you don't want a patch to do both code cleanup and functionality improvements at the same time— this should be done in separate patches.)

Resolution

Comments

Post new comment

The content of this field is kept private and will not be shown publicly.
  • You may post code using <code>...</code> (generic) or <?php ... ?> (highlighted PHP) tags.
  • You can use Markdown syntax to format and style the text. Also see Markdown Extra for tables, footnotes, and more.
  • Web page addresses and e-mail addresses turn into links automatically.
  • Allowed HTML tags: <a> <em> <strong> <cite> <code> <ul> <ol> <li> <dl> <dt> <dd> <img> <blockquote> <small> <h2> <h3> <h4> <h5> <h6> <sub> <sup> <p> <br> <strike> <table> <tr> <td> <thead> <th> <tbody> <tt> <output>
  • Lines and paragraphs break automatically.

More information about formatting options

By submitting this form, you accept the Mollom privacy policy.