https://gcc.gnu.org/bugzilla/show_bug.cgi?id=37475
--- Comment #12 from Kristian Spangsege <kristian.spangsege at gmail dot com> --- These changes look good to me. A couple of points: * Should the `else`s still be there now that the preceding branches terminate with `break`? * `do_out()` has the same problem as `do_in()`. * If there is already testing of `codecvt`, should a test be added for the "empty output" case? * This could break code that expects the incorrect behavior.