[Bug 143090] [PATCH] Let indent(1) handle widecharacter literals correctly

[bugzilla-noreply]

https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=143090

--- Comment #5 from commit-h...@freebsd.org ---
A commit references this bug:

Author: eadler
Date: Thu Mar  8 06:54:33 UTC 2018
New revision: 330633
URL: https://svnweb.freebsd.org/changeset/base/330633

Log:
  MFC r309220:

  indent(1): Properly handle the wide string literal and wide char constant L.

  indent(1) treated the "L" in "L'a'" as if it were an identifier and forced
  a space character after it, breaking valid code.

  PR:   143090

Changes:
_U  stable/11/
  stable/11/usr.bin/indent/indent.c
  stable/11/usr.bin/indent/indent_codes.h
  stable/11/usr.bin/indent/lexi.c

-- 
You are receiving this mail because:
You are the assignee for the bug.
___
freebsd-bugs@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/freebsd-bugs
To unsubscribe, send any mail to "freebsd-bugs-unsubscr...@freebsd.org"

[Bug 143090] [PATCH] Let indent(1) handle widecharacter literals correctly

[bugzilla-noreply]

https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=143090

Piotr Pawel Stefaniak  changed:

   What|Removed |Added

 CC||ps...@freebsd.org
 Status|In Progress |Closed
 Resolution|--- |FIXED

--- Comment #4 from Piotr Pawel Stefaniak  ---
Fixed by r309220.

-- 
You are receiving this mail because:
You are the assignee for the bug.
___
freebsd-bugs@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/freebsd-bugs
To unsubscribe, send any mail to "freebsd-bugs-unsubscr...@freebsd.org"

[Bug 143090] [PATCH] Let indent(1) handle widecharacter literals correctly

[bugzilla-noreply]

https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=143090

--- Comment #3 from commit-h...@freebsd.org ---
A commit references this bug:

Author: pfg
Date: Sun Nov 27 20:38:15 UTC 2016
New revision: 309220
URL: https://svnweb.freebsd.org/changeset/base/309220

Log:
  indent(1): Properly handle the wide string literal and wide char constant L.

  indent(1) treated the "L" in "L'a'" as if it were an identifier and forced
  a space character after it, breaking valid code.

  PR:   143090
  MFC after:2 weeks

Changes:
  head/usr.bin/indent/indent.c
  head/usr.bin/indent/indent_codes.h
  head/usr.bin/indent/lexi.c

-- 
You are receiving this mail because:
You are the assignee for the bug.
___
freebsd-bugs@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/freebsd-bugs
To unsubscribe, send any mail to "freebsd-bugs-unsubscr...@freebsd.org"

[Bug 143090] [PATCH] Let indent(1) handle widecharacter literals correctly

[bugzilla-noreply]

https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=143090

--- Comment #2 from Pedro F. Giffuni  ---
(In reply to Pedro F. Giffuni from comment #1)
Nevermind the commnet: it corresponds to another bug.

The new patch should be good though.

-- 
You are receiving this mail because:
You are the assignee for the bug.
___
freebsd-bugs@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/freebsd-bugs
To unsubscribe, send any mail to "freebsd-bugs-unsubscr...@freebsd.org"

[Bug 143090] [PATCH] Let indent(1) handle widecharacter literals correctly

[bugzilla-noreply]

https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=143090

Pedro F. Giffuni  changed:

   What|Removed |Added

 Attachment #103198|0   |1
is obsolete||
 CC||p...@freebsd.org

--- Comment #1 from Pedro F. Giffuni  ---
Created attachment 173382
  --> https://bugs.freebsd.org/bugzilla/attachment.cgi?id=173382=edit
Fix for current

Fix by Piotr Stephaniak:

"The patch from bugzilla is not enough, with that as-is, indent 
would be able to break valid code as pointed out by Peter Jeremy in 
https://lists.freebsd.org/pipermail/freebsd-hackers/2008-April/024336.html "

-- 
You are receiving this mail because:
You are the assignee for the bug.
___
freebsd-bugs@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/freebsd-bugs
To unsubscribe, send any mail to "freebsd-bugs-unsubscr...@freebsd.org"