Re: [PATCH 0/5] MR5048: jscript: ArrayBuffer and DataView implementations.

[Jacek Caban (＠jacek)]

Jacek Caban (@jacek) commented about dlls/jscript/arraybuf.c:
> - DataViewInstance *view;
> - HRESULT hres;
> - DWORD offset;
> - BYTE *data;
> - double n;
> -
> - if(!(view = dataview_this(vthis)))
> -
>    return JS_E_NOT_DATAVIEW;
> - if(is_undefined(argv[0]) || is_undefined(argv[1]))
> -
>    return JS_E_DATAVIEW_NO_ARGUMENT;
> -
> - hres = to_integer(ctx, argv[0], &n);
> - if(FAILED(hres))
> -
>    return hres;
> -
> - if(n < 0.0 || n >= view->size)

You could just use `n + type_size` here, avoiding later recheck.