[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