[PATCH 5/9] jscript: Use jsdisp_define_property for Error object properties.

[Jacek Caban]

Signed-off-by: Jacek Caban <jacek(a)codeweavers.com>
---
 dlls/jscript/error.c     | 13 +++++++++----
 dlls/mshtml/tests/es5.js |  2 ++
 2 files changed, 11 insertions(+), 4 deletions(-)

[Marvin]

Hi,

While running your changed tests on Windows, I think I found new failures.
Being a bot and all I'm not very good at pattern recognition, so I might be
wrong, but could you please double-check?
Full results can be found at
https://testbot.winehq.org/JobDetails.pl?Key=38318

Your paranoid android.


=== wvistau64 (32 bit script) ===
script.c:644: Test failed: L"desc.enumerable = true expected false"

=== wvistau64_zh_CN (32 bit script) ===
script.c:644: Test failed: L"desc.enumerable = true expected false"

=== wvistau64_fr (32 bit script) ===
script.c:644: Test failed: L"desc.enumerable = true expected false"

=== wvistau64_he (32 bit script) ===
script.c:644: Test failed: L"desc.enumerable = true expected false"

=== wvistau64 (64 bit script) ===
script.c:644: Test failed: L"desc.enumerable = true expected false"

[Jacek Caban]

On 05/11/2018 04:33 PM, Marvin wrote:
> === wvistau64 (32 bit script) ===
> script.c:644: Test failed: L"desc.enumerable = true expected false"
>
> === wvistau64_zh_CN (32 bit script) ===
> script.c:644: Test failed: L"desc.enumerable = true expected false"
>
> === wvistau64_fr (32 bit script) ===
> script.c:644: Test failed: L"desc.enumerable = true expected false"
>
> === wvistau64_he (32 bit script) ===
> script.c:644: Test failed: L"desc.enumerable = true expected false"
>
> === wvistau64 (64 bit script) ===
> script.c:644: Test failed: L"desc.enumerable = true expected false"

It looks like IE9 is broken. Please ignore patches 5-9, I will send a
fixed version later.


Thanks,

Jacek