+1

2016-10-17 1:51 GMT+08:00 James Peach <jpe...@apache.org>:

>
> > On Oct 16, 2016, at 9:18 AM, Leif Hedstrom <zw...@apache.org> wrote:
> >
> > I'll try to review next week, but I'm +1 on the concept. I assume we
> have places were we use 0 but really should use nullptr?
>
> The only weird think I found is that we used NULL for an invalid pthread_t
> (which is an integral type in glibc). I added ink_thread_null() to capture
> the platform-specific value for this (pretty sure 0 is never a valid
> pthread_t on any platform).
>
> >
> > -- Leif
> >
> >> On Oct 16, 2016, at 9:15 AM, James Peach <jpe...@apache.org> wrote:
> >>
> >> Hi all,
> >>
> >> I’ve posted a RFC pull request <https://github.com/apache/
> trafficserver/pull/1112> that converts all uses of NULL to nullptr. The
> primary motivation for this is that there are circumstances (primarily
> template usage) where nullptr is required to detect the difference between
> a null pointer and a integer 0.
> >>
> >> thanks,
> >> James
> >
>
>

Reply via email to