Boost C++ Libraries: Ticket #10131: [string_algo] trim_all_*() functions leaves a single asterisk in the string.

attachment set

wburkhardt@… — Wed, 18 Jun 2014 09:45:50 GMT

attachment → main.cpp

Example to demonstrate the asterisk bug in string_algo

status changed

Marshall Clow — Wed, 18 Jun 2014 09:59:04 GMT

status new → assigned

I think it's a bug. Similarly, if you change the input to TEST_???***_TEST, you get an output of TEST_?_TEST

Steven Watanabe — Thu, 26 Feb 2015 22:01:46 GMT

This is definitely not a bug. The function is behaving exactly as expected. From the documentation, "trim_all removes all trailing and leading spaces from a sequence (string). In addition, spaces in the middle of the sequence are truncated to just one character." In this case, you're defining "whitespace" to be a set of characters. It sounds like the function that you really want is std::remove_copy_if.

severity changed

Fri, 27 Feb 2015 07:44:20 GMT

severity Problem → Not Applicable

Replying to steven_watanabe:

This is definitely not a bug. The function is behaving exactly as expected. From the documentation, "trim_all removes all trailing and leading spaces from a sequence (string). In addition, spaces in the middle of the sequence are truncated to just one character." In this case, you're defining "whitespace" to be a set of characters. It sounds like the function that you really want is std::remove_copy_if.

I agree. Despite having opened this bug 8 months ago, I cannot see this as a bug today. Not even for the documentation. As you state: "The function is behaving exactly as expected." I change the severity to "Not Applicable" as I cannot close the ticket myself.

status changed; resolution set

Marshall Clow — Fri, 27 Feb 2015 16:35:30 GMT

status assigned → closed
resolution → invalid