[m-rev.] for review: add is_(leading|trailing)_surrogate/1 to char module

Paul Bone paul at bone.id.au
Mon Dec 29 11:47:40 AEDT 2014


On Mon, Dec 22, 2014 at 02:39:20PM +0800, Sebastian Godelet wrote:
> On Mon, 22 Dec 2014 16:18:07 +1100
> Paul Bone <paul at bone.id.au> wrote:
> 
> > 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.
> 
> Opened a pull request:
> https://github.com/Mercury-Language/mercury/pull/17
> 

Thanks,

All done.


-- 
Paul Bone



More information about the reviews mailing list