Reviewers: Neil Puttock, Message: On 2009/11/12 20:38:57, Neil Puttock wrote:
Hi Patrick,
This looks fine to me.
Thanks, Neil. I pushed the patches. -Patrick Description: Source_file: remove mbrtowc() in favor of utf8_char_len() - Move the routine to determine length of UTF-8 characters into a separate function. - Use this new function in Source_file::get_counts() instead of the problematic mbrtowc() function. - In get_counts(), revert the line_char variable to its original behavior: keeping track of the character count, not the byte offset. - Introduce a new parameter to get_counts() to keep track of the byte offset. - Use the byte offset data in Source_file::quote_input() to make sure the source line is split at the correct place. - Remove outdated configure checks, since mbrtowc() is no longer used. Please review this at http://codereview.appspot.com/154046 Affected files: M config.hh.in M configure.in M lily/include/input.hh M lily/include/misc.hh M lily/include/source-file.hh M lily/input-scheme.cc M lily/input.cc M lily/misc.cc M lily/source-file.cc M lily/text-interface.cc _______________________________________________ lilypond-devel mailing list lilypond-devel@gnu.org http://lists.gnu.org/mailman/listinfo/lilypond-devel