The "other case" is someone else's code. I did not want to change it.
I use null because I think it is cleaner action in case you work with  
binary data.
Actually in both case I think the cleaner way to re-set it is to assign  
null.


On Fri, 25 Jan 2013 18:14:47 +0200, Markus Staab  
<notifications@github.com> wrote:

>> -                    var skipCallbackInvocation =  
>> _trackMessageSize(message, _request, _response);
>> -
>> -                    if (!skipCallbackInvocation) {
>> -                        _invokeCallback();
>> -                        _response.responseBody = '';
>> +                    var isString =  typeof(message) == 'string';
>> +                    if(isString){
>> +                        var skipCallbackInvocation =  
>> _trackMessageSize(message, _request, _response);
>> +                       if (!skipCallbackInvocation) {
>> +                           _invokeCallback();
>> +                           _response.responseBody = '';
>> +                       }
>> +                    }else{
>> +                       _response.responseBody = message;
>> +                       _invokeCallback();
>> +                       _response.responseBody = null;
>
> why do you use `null` to reset the body and in the other case `''`?
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/Atmosphere/atmosphere/pull/856/files#r2776162


-- 
Using Opera's revolutionary email client: http://www.opera.com/mail/