Re: RFC: Trailing blanks in source files

From: Pragnesh Sampat (pragnesh.sampat_at_timesys.com)
Date: 01/28/04

  • Next message: Maneesh Soni: "[PATCH 2.6] proc_check_dir-locking-fix" 
    To: "Randy.Dunlap" <rddunlap@osdl.org>
Date:	Tue, 27 Jan 2004 23:53:31 -0500

    > | > I frequently sneakily remove all newly-added trailing whitespace from the
    > | > patches people send me. In 15 years it'll all be gone.
    > |
    > | This would maybe warrant a bk option to remove trailing
    > | whitespace from modified lines. Preferably with an
    > | notification that it is happening so if for some reason you
    > | do want trailing whitespace you could abort or override.
    > |
    > | A patch filter that removed trailing whitespace from + lines
    > | could also be used.
    >
    > AFAIK Andrew already uses a script to check for trailing spaces,
    > and I do also. Yes, it could be modified to be a conversion filter,
    > but it doesn't happen frequently, and when it does, I want to let
    > the poster know about it, so I like to have the evidence handy. :)

    Recent versions of emacs, for those that use it, have whitespace.el (for
    cleaning up as you go along). It can cleanup different kinds of
    whitespace based on settings. Check C-h f whitespace-describe for more
    details. Got used to that heavily at one point, these days I am kind of
    neutral, sometimes I do it, sometimes I don't bother.

    -Pragnesh

    -
    To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
    the body of a message to majordomo@vger.kernel.org
    More majordomo info at http://vger.kernel.org/majordomo-info.html
    Please read the FAQ at http://www.tux.org/lkml/

  • Next message: Maneesh Soni: "[PATCH 2.6] proc_check_dir-locking-fix"

    Relevant Pages

    • Re: cciss updates for 2.6 [1 of 11]
      ... You seem to have used an email client which trims off all trailing ... whitespace. ... But `patch -l' happily applied everything so that's OK. ... send the line "unsubscribe linux-kernel" in ...
      (Linux-Kernel)
    • Re: [OT] Vim highlighting for trailing spaces
      ... >> And for vim trailing space, ... > It always displays trailing whitespace and spaces-before tabs... ... want to use pattern below to also highlight "tab after space" in the ...
      (Linux-Kernel)
    • Re: Options for DataTable string comparisons
      ... that is the correct interpretation of the issue: ignoring trailing ... whitespace in key comparisons should be optional. ... I'm aware that DataTable behaves like SQL Server, ... different from other databases' rules (e.g., Oracle). ...
      (microsoft.public.dotnet.framework.adonet)
    • Re: Pythonic way to trim and keep leading and trailing whitespace
      ... Pythonic way to trim and keep leading and trailing ... whitespace in a string. ... The only solution I can think of is regex, ... like this would be a common type of string processing). ...
      (comp.lang.python)
    • Re: [patch] zaurus family: battery cleanups
      ... This patch has a number of checkpatch errors. ... Mostly trailing ... whitespace, or just spacing issues.. ...
      (Linux-Kernel)