Closed Bug 434463 Opened 16 years ago Closed 16 years ago

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

Categories

(Developer Infrastructure :: Source Code Analysis, defect)

x86
Linux
defect
Not set
normal

Tracking

(Not tracked)

VERIFIED FIXED

People

(Reporter: francois.rigault, Assigned: taras.mozilla)

References

Details

Attachments

(1 file)

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.
> 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.
Blocks: 423898
Good catch, thanks.
Assignee: nobody → tglek
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
Attachment #321859 - Flags: review?(dmandelin)
Attachment #321859 - Flags: review?(dmandelin) → review+
pushed
Status: ASSIGNED → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
It works !
Status: RESOLVED → VERIFIED
Product: Core → Firefox Build System
Product: Firefox Build System → Developer Infrastructure
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: