GitHub user nwellnhof opened a pull request:

    https://github.com/apache/lucy-clownfish/pull/66

    UTF-8 checks, error messages

    Fixes for CLOWNFISH-96 and CLOWNFISH-97.

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/nwellnhof/lucy-clownfish invalid-utf8

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/lucy-clownfish/pull/66.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #66
    
----
commit cb0bb2ca3f77158c5620e4db0ec227f5719582e7
Author: Nick Wellnhofer <wellnho...@aevum.de>
Date:   2016-05-12T13:46:17Z

    Improve error message in case of invalid UTF-8
    
    Add a new string helper StrHelp_validate_utf8 and a corresponding
    macro that validates a UTF-8 string and throws an error if invalid
    UTF-8 is encountered. The error message shows where the invalid UTF-8
    was found and prints the offending bytes as hex.
    
    Obsoletes the old hack that wrote an error message directly to stderr.
    
    First part of CLOWNFISH-97.

commit 06b982309c0cf0797e3beaff87bf142707617b0a
Author: Nick Wellnhofer <wellnho...@aevum.de>
Date:   2016-05-12T13:53:54Z

    Make CB_VCatF throw on invalid UTF-8
    
    Additional test cases will be added when addressing CLOWNFISH-95.
    
    Fixes CLOWNFISH-96.

commit 5c7f068ad3e63785a78d3d12f9a6039b64edf24a
Author: Nick Wellnhofer <wellnho...@aevum.de>
Date:   2016-05-12T14:24:29Z

    Improve error message for invalid CB_VCatF specifiers
    
    Obsoletes the old hack that wrote an error message directly to stderr.
    
    Fixes CLOWNFISH-97.

----


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

Reply via email to