similar to: Highlighting trailing whitespaces on Phab?

Displaying 20 results from an estimated 10000 matches similar to: "Highlighting trailing whitespaces on Phab?"

2016 Nov 14
3
Highlighting trailing whitespaces on Phab?
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
2016 Nov 16
2
Highlighting trailing whitespaces on Phab?
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
2016 Nov 16
2
Highlighting trailing whitespaces on Phab?
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:
2016 Nov 16
2
Highlighting trailing whitespaces on Phab?
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
2010 Jun 25
3
Best practice for removing leading and trailing whitespaces
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
2014 Dec 04
0
[PATCH] Remove trailing whitespaces
This patch removes trailing whitespaces from several files (mostly .c and .h) -------------- next part -------------- A non-text attachment was scrubbed... Name: trail_ws.patch Type: application/octet-stream Size: 11576 bytes Desc: not available Url : http://lists.xiph.org/pipermail/flac-dev/attachments/20141204/4a490d10/attachment-0001.obj
2020 Jan 23
1
Re: [common PATCH] Trim whitespaces from commands read from file
On Thu, Jan 23, 2020 at 10:32:17PM +0100, Martin Kletzander wrote: >On Thu, Jan 23, 2020 at 03:16:47PM +0000, Richard W.M. Jones wrote: >>On Thu, Jan 23, 2020 at 02:04:53PM +0100, Martin Kletzander wrote: >>> On Thu, Jan 23, 2020 at 12:16:50PM +0100, Martin Kletzander wrote: >>> >Signed-off-by: Martin Kletzander <mkletzan@redhat.com> >>> >>> I
2005 May 22
1
R-exts.texi: nuke-trailing-whitespace has changed name (PR#7888)
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
2011 May 05
3
Remove all whitespaces
Hi I got a string that looks something like this 1 2 3 4 5 6 7 8 9 ... and I want it to be 123456789... So I want to remove all spaces (or whitespaces) from my string. Anyone know a good way of doing this? //Joel -- View this message in context: http://r.789695.n4.nabble.com/Remove-all-whitespaces-tp3497867p3497867.html Sent from the R help mailing list archive at Nabble.com.
2005 Feb 19
2
Bug#296017: logcheck: ignore.d.server pure-ftpd user with trailing whitespace
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
2020 Dec 01
1
[PATCH v2 01/20] drm/amdgpu: Fix trailing whitespaces
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
2020 Dec 01
1
[PATCH v2 01/20] drm/amdgpu: Fix trailing whitespaces
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
2015 Oct 13
3
com32/mboot/map.c: removed trailing spaces
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: removed trailing spaces > > > > They were introduced by the patch for ELF64 support. > > > IMHO, the trivial trailing-space cleanup could be included in the same > commit too, instead of adding an
2020 Dec 01
1
[PATCH v2 01/20] drm/amdgpu: Fix trailing whitespaces
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
2002 Mar 17
1
syncing directories with whitespaces
Hi, [Sorry for this very newbie-question but I really can't find this in the man-pages, faq or mailarchives.] How do I rsync directories that includes whitespaces? I like to do something like this but I get "No such file or directory": $ rsync -e ssh -av --delete myhost.org:"/ftp/my dir/" "/ftp-mirror/my dir/" Only solution to this as I can find is to create
2020 Dec 02
1
[PATCH v2 01/20] drm/amdgpu: Fix trailing whitespaces
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
2020 Dec 02
1
[PATCH v2 01/20] drm/amdgpu: Fix trailing whitespaces
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
2005 Apr 21
5
[LLVMdev] Trailing whitespace removal (important for CVS users!)
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
2020 Jan 23
6
Re: [common PATCH] Trim whitespaces from commands read from file
On Thu, Jan 23, 2020 at 02:04:53PM +0100, Martin Kletzander wrote: > On Thu, Jan 23, 2020 at 12:16:50PM +0100, Martin Kletzander wrote: > >Signed-off-by: Martin Kletzander <mkletzan@redhat.com> > > I forgot to mention this is trying to fix the following BZ: > > https://bugzilla.redhat.com/show_bug.cgi?id=1351000 > >--- > >mlcustomize/customize_cmdline.ml |
2011 Jul 26
2
[PATCH 5/7] Xen: fix whitespaces, tabs coding style issue in drivers/xen/xenbus/xenbus_client.c
From: Ruslan Pisarev <ruslan at rpisarev.org.ua> This is a patch to the xenbus_client.c file that fixed up whitespaces, tabs errors found by the checkpatch.pl tools. Signed-off-by: Ruslan Pisarev <ruslan at rpisarev.org.ua> --- drivers/xen/xenbus/xenbus_client.c | 18 +++++++++--------- 1 files changed, 9 insertions(+), 9 deletions(-) diff --git