If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

dehydra_types.c:180: Assertion failed:val != JSVAL_VOID

VERIFIED FIXED

Status

()

Core
Rewriting and Analysis
VERIFIED FIXED
10 years ago
10 years ago

People

(Reporter: Francois, Assigned: (dormant account))

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

10 years ago
User-Agent:       Mozilla/4.0 (compatible; MSIE 6.0; Windows NT 5.1; SV1; .NET CLR 1.1.4322; InfoPath.1)
Build Identifier: Trunk

Function dehydra_attachTemplateStuff does handle templates arguments with class code constant and type, doesnt handle following class codes :
declaration, exceptional, expression, unary, binary



Reproducible: Always

Steps to Reproduce:
compile 
gcc-4.3.0/gcc/testsuite/g++.dg/template/access9.C   (expression case)
gcc-4.3.0/gcc/testsuite/g++.dg/template/class1.C    (unary case)
gcc-4.3.0/gcc/testsuite/g++.dg/template/spec19.C    (exceptional case)
gcc-4.3.0/gcc/testsuite/g++.dg/template/unify1.C    (binary case)
gcc-4.3.0/gcc/testsuite/g++.dg/template/qualttp10.C (declaration case)


Actual Results:  
dehydra_types.c:180: Assertion failed:val != JSVAL_VOID


It could be nice to use dejagnu framework to test dehydra, as we can inherit g++ existing tests. Not investigated, some other xassert-related or ICE are shown after using the current test suite.

Comment 1

10 years ago
> It could be nice to use dejagnu framework to test dehydra, as we can inherit
> g++ existing tests.

There is already a minimalistic unit test framework based on python in xhydra and perhaps it is possible to plug dejagnu tests of g++ into it or at the makefile level.
(Assignee)

Updated

10 years ago
Blocks: 423898
(Assignee)

Comment 2

10 years ago
Created attachment 321859 [details] [diff] [review]
Changed fallback logic

Good catch, thanks.
Assignee: nobody → tglek
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
Attachment #321859 - Flags: review?(dmandelin)
Attachment #321859 - Flags: review?(dmandelin) → review+
(Assignee)

Comment 3

10 years ago
pushed
Status: ASSIGNED → RESOLVED
Last Resolved: 10 years ago
Resolution: --- → FIXED
(Reporter)

Comment 4

10 years ago
It works !
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.