similar to: [PATCH] Remove trailing whitespaces

When I create a patch, I run clang-format-diff, so that's fine. What I want to do is to find trailing whitespaces in others patches. On Mon, Nov 14, 2016 at 2:30 PM, Mehdi Amini <mehdi.amini at apple.com> wrote: > Hi, > > clang-format removes them. > If you are (like me) likely to forget to run "git clang-format HEAD~” > before pushing, adding a pre-commit (or

I wonder if Phabricator has an ability to highlight trailing whitespaces, because I often find trailing whitespaces after they are submitted. If they were highlighted, they would have been removed before submitting. Does anyone know about this? -------------- next part -------------- An HTML attachment was scrubbed... URL:

I want to remove the leading and training spaces for most of the resources in my rails application For a new POST request, rails does the xml parsing and passes the params hash to the controller. Some of the parameters have leading and/or trailing whitespaces which are causing problems (e.g. new resources being created rather than using the old ones). I don''t want to put the code in all

So, I forwarded the request for highlighting trailing whitespaces to phabricator upstream (https://secure.phabricator.com/T11879), and upstream folks suggest we enable the Lint feature in Arcanist ( https://secure.phabricator.com/book/phabricator/article/arcanist_lint/). This will enforce the check when `arc diff` is run (reviewers wouldn't see the warnings though). There are two linters we

Why isn’t it in the LLVM repo? > On Nov 16, 2016, at 7:44 AM, Johannes Doerfert <doerfert at cs.uni-saarland.de> wrote: > > We have a clang format based arcanist linter (and some others) in the > Polly repository. When arcanist is used to create a review, the linter > result is shown online. We also have an arcanist add-on to run the lit > tests and show their result in

2015-10-14 1:45 UTC+02:00, Gene Cumm via Syslinux <syslinux at zytor.com>: > On Tue, Oct 13, 2015 at 5:17 PM, Geert Stappers via Syslinux > <syslinux at zytor.com> wrote: >> On Sat, Oct 10, 2015 at 03:10:26PM +0300, Ady via Syslinux wrote: >>> From: Geert Stappers <stappers at nero.gpm.stappers.nl> >>> > >>> > com32/mboot/map.c:

I see a lot of commits fixing whitespace, my own included. Trailing whitespace is very annoying and many of us have our editors setup to remove it. This creates a problem when modifying other files that have trailing whitespace on just about every empty line and comment. It requires sifting through the diff and extracting the real changes. Of course we could just disable trailing whitespace

Full_Name: Bj?rn-Helge Mevik Version: 2.1.0 OS: GNU/Debian 3.0 Linux Submission from: (NULL) (80.111.104.162) In Appendix B R coding standards of the Writing R Extensions manual, Emacs/ESS users are encouraged to use (add-hook 'ess-mode-hook (lambda () (ess-set-style 'C++) ;; [snip] (add-hook 'local-write-file-hooks

Hi, I am writing tool to simplify automated refactorings. One of prerequisites is have clean codebase, so a refactorer can be simple and created formatting inconsistencies can be eliminated by formatter. My plan to keep codebase clean is first run a cleanup systemwide, then keep it by hook/ periodicaly rerunning cleanup. I put it for now here. https://github.com/neleai/stylepp I ran a scripts

Why not put all this into a pre-commit filter in CVS and be done with it? We'd never be bothered with it again as it would never be committed again. Reid. On Thu, 2005-04-21 at 15:11 -0500, Misha Brukman wrote: > Dear LLVMers, > > If you live on the bleeding edge (i.e. CVS version), please read! > > On Wed, Apr 20, 2005 at 12:12:54PM +0200, Markus F.X.J. Oberhumer wrote:

On 11/14, Mehdi Amini via llvm-dev wrote: > Ideally I’d even really like to have a both checking for revision on > phab, clang-formatting them, and post a comment when there is a > mismatch :) I'd like that! -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 195 bytes Desc: Digital signature URL:

Adhere to kernel coding style. Signed-off-by: Thomas Zimmermann <tzimmermann at suse.de> Acked-by: Alex Deucher <alexander.deucher at amd.com> Acked-by: Sam Ravnborg <sam at ravnborg.org> Cc: Alex Deucher <alexander.deucher at amd.com> Cc: Christian K?nig <christian.koenig at amd.com> --- drivers/gpu/drm/amd/amdgpu/amdgpu_device.c | 6 +++--- 1 file changed, 3

Adhere to kernel coding style. Signed-off-by: Thomas Zimmermann <tzimmermann at suse.de> Acked-by: Alex Deucher <alexander.deucher at amd.com> Acked-by: Sam Ravnborg <sam at ravnborg.org> Cc: Alex Deucher <alexander.deucher at amd.com> Cc: Christian K?nig <christian.koenig at amd.com> --- drivers/gpu/drm/amd/amdgpu/amdgpu_device.c | 6 +++--- 1 file changed, 3

Hi, I wasted some time because there was a space character at the end of an ignore line. I wrote a small patch which removes all whitespace at the end of all ignore lines. Patch is attached. Thomas -- Thomas Guettler, http://www.thomas-guettler.de/ E-Mail: guettli (*) thomas-guettler + de Spam Catcher: niemand.leermann at thomas-guettler.de -------------- next part -------------- ---

Package: logcheck Version: 1.2.34 Severity: normal the patterns for pure-ftpd in ignore.d.server are not matching a user with a trailing whitespace. here a some examples: Feb 18 13:02:33 web1 pure-ftpd: (stupid-pure-ftpd @84.56.131.73) [NOTICE] /example/example.txt downloaded (5908 bytes, 152196.03KB/sec) Feb 18 13:16:14 web1 pure-ftpd: (stupid-pure-ftpd @84.56.131.73) [INFO] Logout. every

Reviewed-by: Christian K?nig <christian.koenig at amd.com> on patch #1 and #15. Acked-by: Christian K?nig <christian.koenig at amd.com> on patch #2 and #16. Regards, Christian. Am 01.12.20 um 11:35 schrieb Thomas Zimmermann: > Adhere to kernel coding style. > > Signed-off-by: Thomas Zimmermann <tzimmermann at suse.de> > Acked-by: Alex Deucher <alexander.deucher

Dear LLVMers, If you live on the bleeding edge (i.e. CVS version), please read! On Wed, Apr 20, 2005 at 12:12:54PM +0200, Markus F.X.J. Oberhumer wrote: > Do you really want external patches for this ? A simple Perl script > that runs on all *.h and *.cpp files, and a local commit from your > side would be much simpler. I'm in the process of doing just this as we speak. What this

Hi Am 01.12.20 um 11:40 schrieb Christian K?nig: > Reviewed-by: Christian K?nig <christian.koenig at amd.com> on patch #1 and > #15. > > Acked-by: Christian K?nig <christian.koenig at amd.com> on patch #2 and #16. Could you add these patches to the AMD tree? Best regards Thomas > > Regards, > Christian. > > Am 01.12.20 um 11:35 schrieb Thomas

On Thu, 21 Apr 2005, Reid Spencer wrote: > Why not put all this into a pre-commit filter in CVS and be done with > it? We'd never be bothered with it again as it would never be committed > again. I'd rather not have CVS commit scripts mucking with the code. If you want to have the nightly tester whine about source code with spaces at the end of lines (like it whines about

Am 02.12.20 um 08:59 schrieb Thomas Zimmermann: > Hi > > Am 01.12.20 um 11:40 schrieb Christian K?nig: >> Reviewed-by: Christian K?nig <christian.koenig at amd.com> on patch #1 >> and #15. >> >> Acked-by: Christian K?nig <christian.koenig at amd.com> on patch #2 and >> #16. > > Could you add these patches to the AMD tree? Alex is usually