Re: [PATCH 21/21] jscript: Pack jsval_t to 64-bit structure on i386

2012-09-18 Thread Jacek Caban 
Hi Juan,

On 09/17/12 22:42, Juan Lang wrote:
 Hi Jacek,

 + * that NaN value representation has 52 (almost) free bytes.

 You mean bits, yes?

Yes, of course.

 While you're at it,
 + * jsval_t structure is used to represent JavaScript dynamicaly-typed values.

 dynamically is spelled with two l's.

Thanks for catching those. I will fix it while resubmitting after I will
fix exception value propagation.

Thanks,
Jacek

Re: [PATCH 21/21] jscript: Pack jsval_t to 64-bit structure on i386

2012-09-17 Thread Jacek Caban 
Please ignore this last patch. It has one more dependency that I didn't
send yet.

On 09/17/12 15:21, Jacek Caban wrote:
 ---
  dlls/jscript/jsutils.c |4 +-
  dlls/jscript/jsval.h   |  143
 +--
  2 files changed, 114 insertions(+), 33 deletions(-)

Re: [PATCH 21/21] jscript: Pack jsval_t to 64-bit structure on i386

2012-09-17 Thread Juan Lang 
Hi Jacek,

+ * that NaN value representation has 52 (almost) free bytes.

You mean bits, yes?

While you're at it,
+ * jsval_t structure is used to represent JavaScript dynamicaly-typed values.

dynamically is spelled with two l's.

Thanks,
--Juan