libc++ review: add validation to Stage 2 of num_get

Project Infrastructure LLVM Dev List Archives
#1

LLVM developers:

Does anyone have time and interest to review a libc++ change?

    https://reviews.llvm.org/D26920

Eric

#2

+cfe-dev, -llvm-dev

LLVM developers:

Does anyone have time and interest to review a libc++ change?

    ⚙ D26920 [libc++] Add validation to Stage 2 of num_get

Reviews for libc++ belong here on cfe-dev.

Cheers,

Jon

#3

Hi Eric (CC Eric and Marshall),

libc++ patches usually go to cfe-commits. I'm CCing Eric Fiselier and Marshall Clow, the primary libc++ maintainers.

Shoaib

#4

libc++ patches usually go to cfe-commits.

Oh, I see that now. The list description sounded like it was only for commit
notifications from Subversion.

I'm CCing Eric Fiselier and Marshall Clow, the primary libc++ maintainers.

Thank you for helping me learn my way around.

Eric

#5

cfe-commits is meant for both patches and commit notifications. The standard
procedure is to add cfe-commits as a subscriber when you're creating a libc++
patch using `arc diff`, which sends an email to the list.

    > libc++ patches usually go to cfe-commits.
    
    Oh, I see that now. The list description sounded like it was only for commit
    notifications from Subversion.
    
    > I'm CCing Eric Fiselier and Marshall Clow, the primary libc++ maintainers.
    
    Thank you for helping me learn my way around.
    
    Eric