bugGnash - The GNU Flash player - Bugs: bug #21020, Magical Trevor fails an assertion...

 
 

bug #21020: Magical Trevor fails an assertion (morphs)

Submitted by:  Benjamin Wolsey <bwy>
Submitted on:  Mon 10 Sep 2007 03:08:43 PM UTC  
 
Category: NoneSeverity: 3 - Normal
Release: NoneStatus: Fixed
Privacy: PublicAssigned to: Sandro Santilli <strk>
Open/Closed: Closed

Add a New Comment(Rich Markup)
   

You are not logged in

Please log in, so followups can be emailed to you.

 

(Jump to the original submission Jump to the original submission)

Mon 10 Sep 2007 06:00:50 PM UTC, comment #9:

Yes, it works now.

Benjamin Wolsey <bwy>
Project Member
Mon 10 Sep 2007 05:14:55 PM UTC, comment #8:

I forced override of bounds with those read from definition.

Sandro Santilli <strk>
Project MemberIn charge of this item.
Mon 10 Sep 2007 04:49:37 PM UTC, comment #7:

Magical Trevor's hands are mostly not there, by the way, but I would think they are supposed to be. I wonder if that is the morph causing the problems.

Benjamin Wolsey <bwy>
Project Member
Mon 10 Sep 2007 04:42:44 PM UTC, comment #6:

Actually, it supposedly doesn't discard the bounds, but will try to recompute them. Could it be onoe of the shapes of the morph is really empty so that bounds computation gives a NULL rect ?

Should we force use of the defined bounds in this case ?

Sandro Santilli <strk>
Project MemberIn charge of this item.
Mon 10 Sep 2007 04:38:48 PM UTC, comment #5:

I think I've found the bug, shape_character_def::read will discard pre-set bounds. See morph2_character_def.

Sandro Santilli <strk>
Project MemberIn charge of this item.
Mon 10 Sep 2007 03:56:34 PM UTC, comment #4:

Anyway, the root cause is that either shape1 or shape2 bounds are not-finite after read. We may as well compute the bounds for the final shape anyway, I'm not even sure using the lerp of the two shapes give an accurate result..

Sandro Santilli <strk>
Project MemberIn charge of this item.
Mon 10 Sep 2007 03:38:28 PM UTC, comment #3:

The problem is rect::set_lerp not properly handlign NULL and WORLD rect cases.

The following questions should be answered, and documented in the rect::set_lerp method description:

Q: What's the lerp between a NULL and a NULL rect ?

Q: What's the lerp between a WORLD and WORLD rect ?

Q: What's the lerp between a WORLD and NULL rect ?

Q: What's the lerp between a NULL and a finite rect ?

Q: What's the lerp between a WORLD and a finite rect ?

Sandro Santilli <strk>
Project MemberIn charge of this item.
Mon 10 Sep 2007 03:19:18 PM UTC, comment #2:

SWF9
md5: 88bede49297c1699077ee39e55b45b78

Sandro Santilli <strk>
Project MemberIn charge of this item.
Mon 10 Sep 2007 03:11:16 PM UTC, comment #1:

Movie is: http://www.weebls-stuff.com/flash/8594trev3.swf

Benjamin Wolsey <bwy>
Project Member
Mon 10 Sep 2007 03:08:43 PM UTC, original submission:

SIGABRT:

../libgeometry/Range2d.h:622: T gnash::geometry::Range2d<T>::getMinX() const [with T = float]:

Fails assertion "isFinite()"

A backtrace if it's any use:

#3 0x00002b5b4446b7e3 in gnash::rect::set_lerp (this=<value optimized out>,
a=<value optimized out>, b=<value optimized out>, t=0)
at ../libgeometry/Range2d.h:622
#4 0x00002b5b4455dc47 in gnash::morph2_character_def::display (this=0xc03820,
inst=0xbe72f0) at morph2_character_def.cpp:122

Benjamin Wolsey <bwy>
Project Member

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach File(s):
   
   
Comment:
   

No files currently attached

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -unavailable- added by strk (Posted a comment)
  • -unavailable- added by bwy (Submitted the item)
  •  

    Do you think this task is very important?
    If so, you can click here to add your encouragement to it.
    This task has 0 encouragements so far.

    Only logged-in users can vote.

     

    Please enter the title of George Orwell's famous dystopian book (it's a date):

     

     

    Follow 4 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Mon 10 Sep 2007 07:08:22 PM UTCstrkStatusReady For Test=>Fixed
      Open/ClosedOpen=>Closed
    Mon 10 Sep 2007 05:14:55 PM UTCstrkStatusNone=>Ready For Test
      Assigned tocmusick=>strk

    Back to the top


    Powered by Savane 3.1-cleanup1