similar to: Best practice for removing leading and trailing whitespaces

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:

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

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

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:

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

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

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.

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

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

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

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

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

I have created a scaffold Admin/Radicals for doing CRUD. However, I''m not sure exactly where the scaffold uses the save() method. For a new entry, it creates form "radical" referencing method create(). The code for create() is as follows: def create @radical = Radical.new(params[:radical]) if @radical.save flash[:notice] = ''Radical was

On Wed, Dec 2, 2020 at 3:53 AM Thomas Zimmermann <tzimmermann at suse.de> wrote: > > Hi > > Am 02.12.20 um 09:43 schrieb Christian K?nig: > > 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

On Wed, Dec 2, 2020 at 3:53 AM Thomas Zimmermann <tzimmermann at suse.de> wrote: > > Hi > > Am 02.12.20 um 09:43 schrieb Christian K?nig: > > 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

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

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

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