Get rid of all warnings and errors on Windows.

This increases msvc compatibility.
Oops, this is a remnant from investigating the separate memory management issue in NanRawString. If s is deleted, I get a runtime segfault.

On June 6, 2014 8:28:31 PM EEST, Ben Noordhuis <notifications@github.com> wrote:
>>      assert((bc & 1) == 0 && "UCS2 string byte count was not even");
>>      NanReturnValue(retval);
>>    } else {
>> -    NanReturnValue(
>> -      NanNew<v8::String>(
>> -          static_cast<char *>(s)
>> -        , (flags & v8::String::NO_NULL_TERMINATION) ? bc : -1)
>> -    );
>> +    v8::Local<v8::String> retval = NanNew(
>> +        static_cast<char *>(s)
>> +      , (flags & v8::String::NO_NULL_TERMINATION) ?
>static_cast<int>(bc) : -1);
>> +    s = NULL;
>
>Dead store?
>
>---
>Reply to this email directly or view it on GitHub:
>https://github.com/rvagg/nan/pull/149/files#r13500923