bugGnash - The GNU Flash player - Bugs: bug #20920, regression from 0.8.1

 
 

bug #20920: regression from 0.8.1

Submitter:  None
Submitted:  Tue 28 Aug 2007 10:14:58 AM UTC
   
 
Category:  None Severity:  4 - Important
Release:  None Status:  Fixed
Privacy:  Public Assigned to:  nihilus
Open/Closed:  Closed
* Mandatory Fields

Add a New Comment Rich Markup
   

Jump to the original submission

Wed 29 Aug 2007 11:13:03 AM UTC, comment #18: 

Some unit testing for movie_root anyone ?
Like calling set_background_alpha() and check get_background_alpha() might be enough to reproduce
this problem on frnd() change ...

Sandro Santilli <strk>
Group Member
Wed 29 Aug 2007 07:48:50 AM UTC, comment #17: 

Works for me.

Black background is due to a invisible background color (alpha=0 instead of alpha=255), see movie_root.h:

void set_background_alpha(float alpha)
{
  m_background_color.m_a = iclamp(frnd(alpha * 255.0f), 0, 255);
}

Udo Giacomozzi <udog>
Group Member
Wed 29 Aug 2007 05:45:59 AM UTC, comment #16: 

confirmed.
cvs-head works now, wasn't before strk's commit.

> Note that rint() is not the same as fchop().


It's funny that this little change caused such big regression(many of the swf files in our testsuite turned into black backgrounds).

Zou Lunkai <zoulunkai>
Wed 29 Aug 2007 04:20:41 AM UTC, comment #15: 

ahhh, indeed rint() seems to be handling things more arbitary.

Markus Gothe <nihilus>
Group Member
Wed 29 Aug 2007 02:38:14 AM UTC, comment #14: 

This change in particular introduced the regression:

- inline int    frnd(float f) { return fchop(f + 0.5f); }       // replace with inline asm if desired
+ inline int   frnd(float f) { return (int)rint(f + 0.5f); }   // replace with inline asm if desired

Note that rint() is not the same as fchop().


Sandro Santilli <strk>
Group Member
Wed 29 Aug 2007 02:15:01 AM UTC, comment #13: 

2007-08-28 00:00 works, so the culprit is a change between 00:00 and 05:00. The only change in the ChangeLog between these two periods is an entry in Markus log:

>       * libbase/math.h OS X has got xF().


Attached the patch which introduced the problem.

(file #13829)

Sandro Santilli <strk>
Group Member
Wed 29 Aug 2007 01:55:24 AM UTC, comment #12: 

ChangeLog-2007-08-28-5:00 fails.
Current range is:

2007-08-28 05:00 <-- fails
2007-08-27 20:00 <-- works

Sandro Santilli <strk>
Group Member
Wed 29 Aug 2007 01:42:03 AM UTC, comment #11: 

2007-08-28 15:00 fails.

Sandro Santilli <strk>
Group Member
Wed 29 Aug 2007 01:13:54 AM UTC, comment #10: 

2007-08-27 20:00 works. Last ChangeLog entry:

2007-08-27 Deanna Phillips <deanna>

        * testsuite/misc-ming.all/NetStream-SquareTestRunner.cpp: include
          <unistd.h>

Sandro Santilli <strk>
Group Member
Wed 29 Aug 2007 12:59:50 AM UTC, comment #9: 

2007-08-27-15:00 works. Last ChangeLog entry:

2007-08-27 Sandro Santilli <strk@keybit.net>

        * server/cxform.cpp, server/font.cpp, server/matrix.cpp,
          server/stream.h, server/parser/button_character_def.cpp,
          server/parser/edit_text_character_def.cpp,
          server/parser/filter_factory.cpp,
          server/parser/sound_definition.cpp,
          server/parser/video_stream_def.cpp,
          server/swf/PlaceObject2Tag.cpp, server/swf/tag_loaders.cpp,
          utilities/parser.cpp: read_uint(1) => read_bit();
        * server/stream.cpp (read_uint): htf_sweet.swf aborts if we
          assert bit_count <= 24, so change this base to <= 32 (will
          inspect later).

Sandro Santilli <strk>
Group Member
Wed 29 Aug 2007 12:28:02 AM UTC, comment #8: 

2007-08-27 works fine. gtk/agg. Last ChangeLog entry:

 2007-08-26 Deanna Phillips <deanna>   

    * doc/gnash.1: Updated.

Sandro Santilli <strk>
Group Member
Tue 28 Aug 2007 05:39:36 PM UTC, comment #7: 

By chance I noticed that the background color passed to begin_display() has the alpha (m_a) component set to zero. I'm pretty sure this is wrong...

Udo Giacomozzi <udog>
Group Member
Tue 28 Aug 2007 04:09:49 PM UTC, comment #6: 

It is a recent regression. A SWF I often use to start gnash for testing suffers from the same problem, and it was fine at the weekend, I think until the 26th at least.

Benjamin Wolsey <bwy>
Group Member
Tue 28 Aug 2007 02:05:48 PM UTC, comment #5: 

Apparently any movie where you can see the default background (which is defined by a solid color in the SWF file) shows a black background. gravity.swf (in testsuite) suffers the same problem.

Udo Giacomozzi <udog>
Group Member
Tue 28 Aug 2007 01:13:45 PM UTC, comment #4: 

simpleguess.swf has the same problem (swf4)
md5: 45c113fc0357358dc300b0041f22dfea

(should be somewhere in the tracker already)

Sandro Santilli <strk>
Group Member
Tue 28 Aug 2007 11:33:08 AM UTC, comment #3: 

GTK-OGL with HEAD as of 2007/08/28 13:30 CEST: black background

Udo Giacomozzi <udog>
Group Member
Tue 28 Aug 2007 11:16:36 AM UTC, comment #2: 

Yes, can reproduce:

GTK-AGG with HEAD as of 2007/08/28 11:58 CEST: black background
GTK-AGG with RELEASE 081 as of 2007/08/23 14:02 CEST: white background
GTK-OGL with HEAD as of 2007/08/23 12:13 CEST: white background

(still trying other variants)

Udo Giacomozzi <udog>
Group Member
Tue 28 Aug 2007 10:16:53 AM UTC, comment #1: 

Can you reproduce the problem ?

Sandro Santilli <strk>
Group Member
Tue 28 Aug 2007 10:14:58 AM UTC, original submission:  

http://foo.keybit.net/~strk/tmp/all.swf

The file above looks right with 0.8.1 (white background) and
wrong with head (black background)

Anonymous

 

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

Attach Files:
   
   
Comment:
   

Attached Files
file #13829:  culprit.patch added by strk (2KiB - text/x-patch - the patch introducing the regression)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by zoulunkai (Posted a comment)
  • -email is unavailable- added by nihilus (Posted a comment)
  • -email is unavailable- added by bwy (Posted a comment)
  • -email is unavailable- added by udog (Posted a comment)
  • -email is unavailable- added by strk (Posted a comment)
  •  

    There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

    Only logged-in users can vote.

     

    Follow 8 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2007-08-29 nihilus StatusReady For Test Fixed
        Open/ClosedOpen Closed
    2007-08-29 nihilus StatusNone Ready For Test
    2007-08-29 strk Attached File- Added culprit.patch, #13829
        Assigned toNone nihilus
    2007-08-28 strk Severity3 - Normal 4 - Important
        Carbon-Copy- Added zoulunkai
        Carbon-Copy- Added udog

    Back to the top

    Powered by Savane 3.13-4b48.
    Corresponding source code