Patches item #3421072, was opened at 2011-10-10 09:03 Message generated for change (Comment added) made by mukau You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=478072&aid=3421072...

Please note that this message will contain a full copy of the comment thread, including the initial issue submission, for this request, not just the latest update. Category: puredata Group: None Status: Open Resolution: None Priority: 5 Private: No Submitted By: Marvin Humphrey (creamygoodness) Assigned to: Bryan Jurish (mukau) Summary: Streamline s_utf8.c

Initial Comment: There are a number of UTF-8 manipulation routines which were included when the library was added with the justification that they might be used someday. However, time has passed, they are still unused, and some of them have bugs -- so it is time to delete them with a justification of YAGNI (You Ain't Gonna Need It).

----------------------------------------------------------------------

Comment By: Bryan Jurish (mukau)

Date: 2011-10-10 21:16

Message: Looks good to me. I can't test anything right now, but Martin's suggestions seem safe and clear. Give it a whirl...

I didn't see any fix for the NUL-termination problems I saw suggested on the #dataflow transcript though (and I never noticed any such segfaults myself): has that problem been confirmed?

----------------------------------------------------------------------

Comment By: Hans-Christoph Steiner (eighthave) Date: 2011-10-10 18:37

Message: Makes sense to me. I'm assigning it to Bryan Jurish, the original author of the UTF-8 support, to see if he agrees.

----------------------------------------------------------------------

You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=478072&aid=3421072...