[m-rev.] for review: add is_(leading|trailing)_surrogate/1 to char module
Paul Bone
paul at bone.id.au
Mon Dec 22 16:18:07 AEDT 2014
On Sun, Dec 21, 2014 at 10:24:37AM +0800, Sebastian Godelet wrote:
> For review by anyone.
>
> For input validation and character encoding transformations
> it is essential to check for surrogate characters and their correct
> sequence.
>
> NEWS:
> Announce the addition of is_(leading|trailing)_surrogate/1.
>
> library/char.m:
> Add is_leading_surrogate/1 which succeeds if a character is a
> leading surrogate character.
> is_trailing_surrogate/1 succeeds if the character is a trailing
> surrogate character.
>
> Sebastian.
Thanks Sebastian,
This looks good to me. If you open a pull request I'll merge the changes.
Cheers.
--
Paul Bone
More information about the reviews
mailing list