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