#313 ✓resolved

Remove trailing space plugin is overeager

Reported by mscharley | September 15th, 2010 @ 08:35 AM

I'll probably look into this one, but just as a note for everyone, and get some feedback.

The remove trailing space plugin is too eager. It removes spaces at the end of lines that are entirely whitespace. Generally this happens when people leave whitespace in to maintain indentation. I consider this a bug. What do others think?

Comments and changes to this ticket

  • Daniel Lucraft

    Daniel Lucraft September 15th, 2010 @ 08:44 AM

    • State changed from “new” to “open”

    I'm not sure it's a bug per se, but I agree that it shouldn't do that. Just because it massively increases the I-just-checked-in-100-modified-files problem.

  • Daniel Lucraft

    Daniel Lucraft September 15th, 2010 @ 09:56 AM

    • State changed from “open” to “resolved”
  • Steven Hancock

    Steven Hancock September 15th, 2010 @ 11:58 PM

    Not a bug, that was intended behavior (to match the Gedit plugin I modeled it after, as well as coding standards for quite a few opensource projects that I know of). I see this has already been changed. I personally prefer not to leave the extra spaces and haven't really seen any need for them (Redcar's auto-indenter works fine for me, no need to "preserve indentation" that I've seen, at least for Ruby code). Probably the best thing to do is make this configurable since I think the way it origially worked is "correct" most of the time. I can do that later today. :)

Please Sign in or create a free account to add a new ticket.

With your very own profile, you can contribute to projects, track your activity, watch tickets, receive and update tickets through your email and much more.

New-ticket Create new ticket

Create your profile

Help contribute to this project by taking a few moments to create your personal profile. Create your profile ยป

A programmer's text editor for Gnome.

People watching this ticket