Double-check unused Double Field code
mmarchini opened this issue · comments
The if branch on https://github.com/nodejs/llnode/blob/master/src/llv8.cc#L1178-L1202 is essentially a noop: double value
is set but never used, since we'll always return Value()
. Not sure if we should be returning double value
or if we can remove this code.