search

nodejs / CTC

Node.js Core Technical Committee & Collaborators
80 stars 27 forks source link

Buffer constructor: random-fill in Node.js 8.0.0? #90

Closed Trott closed 7 years ago

Trott commented 7 years ago

@nodejs/ctc Next vote that was started in the CTC meeting. If you are not listed below (or if your vote has changed or been recorded incorrectly), please comment! Thanks!

Should we random-fill in Node.js 8.0.0?

YES: @addaleax, @cjihrig, @targos, @fhinkel

NO: @mscdex, @ChALkeR, @indutny, @jasnell, @mylesborins, @shigeki, @misterdjules, @trevnorris, @rvagg, @thefourtheye

ABSTAIN: @trott, @Fishrock123, @ofrobots, @evanlucas

MY VOTE DEPENDS ON WHETHER OR NOT WE BACKPORT AND/OR WHICH RELEASE LINES WE BACKPORT TO:

There is a separate issue for the question of whether we zero-fill. If both pass and there's not consensus over which one to do, we'll have another vote.

indutny commented 7 years ago

NO

jasnell commented 7 years ago

No (my vote is for zero-fill in Node.js 8.0.0)

MylesBorins commented 7 years ago

what james said

Fishrock123 commented 7 years ago

abstain (favor zero-fill)

ofrobots commented 7 years ago

Abstain (favour zero fill).

evanlucas commented 7 years ago

Abstain (favor zero fill)

shigeki commented 7 years ago

No

misterdjules commented 7 years ago

My vote is no.

trevnorris commented 7 years ago

nope

rvagg commented 7 years ago

no

addaleax commented 7 years ago

If I’m counting correctly this has 9 No votes and 4 abstentions, which means that this can be closed with “No” as the result.

thefourtheye commented 7 years ago

No - Zero Fill.