[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