bugGnash - The GNU Flash player - Bugs: bug #34245, [REGRESSION] FB gui doesn't build...

 
 

bug #34245: [REGRESSION] FB gui doesn't build with double buffering enabled

Submitted by:  Sandro Santilli <strk>
Submitted on:  Wed Sep 7 15:26:29 2011  
 
Category: gui-fbSeverity: 4 - Important
Release: masterStatus: Fixed
Privacy: PublicAssigned to: Rob Savoye <rsavoye>
Open/Closed: Closed

Add a New Comment (Rich MarkupRich 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 Dec 5 13:00:46 2011, comment #16:

Yup, this is fixed. Thank you.

Am I right that all the new features I added in FB gui for handling of -j and -k and -X and -Y were removed ? I don't seem to be able to decide on geometry for the video window (which always seem to be re-rendering the whole available screen rather than just the movie portion)

Sandro Santilli <strk>
Project Member
Mon Dec 5 10:20:04 2011, comment #15:

I'll test as soon as I'll have a chance. Meanwhile I've built on one system and got this warning which you may want to clean:

gui/fb/fb.cpp:495: warning: unused parameter ‘ranges’

Sandro Santilli <strk>
Project Member
Mon Dec 5 05:15:08 2011, comment #14:

Double buffering with AGG in a framebuffer works again as of #3702f10. It's also enabled now by default since it does look better, but has a configure option to disable it.

Rob Savoye <rsavoye>
Project AdministratorIn charge of this item.
Thu Sep 29 14:20:04 2011, comment #13:

Depends when we do a release. If we do a release soon, then no. If we aim for a release in a few weeks, then maybe yes.

Rob Savoye <rsavoye>
Project AdministratorIn charge of this item.
Thu Sep 29 10:12:47 2011, comment #12:

I think this should be fixed before the release.
Do you agree, Rob ?

Sandro Santilli <strk>
Project Member
Tue Sep 20 13:17:52 2011, comment #11:

I mean getting back what used to work: FB/AGG with double buffering.

I don't think any other GUI was using that configure switch (--enable-double-buffering).

GTK is likely using double buffering internally, no matter ENABLE_DOUBLE_BUFFERING define.

Double buffering for FB/OpenVG (if needed) can wait.

Sandro Santilli <strk>
Project Member
Tue Sep 20 13:11:55 2011, comment #10:

You mean enabled only for AGG and the FB ?

Rob Savoye <rsavoye>
Project AdministratorIn charge of this item.
Tue Sep 20 07:10:50 2011, comment #9:

Can we have double-buffering only enabled for FB while we wait for the refactoring ? That way whoever uses FB doesn't suffer from the regression, and this bug can be converted into a enhancement request

Sandro Santilli <strk>
Project Member
Wed Sep 7 16:30:04 2011, comment #8:

A regression is something that used to work and now doesn't.

Benjamin Wolsey <bwy>
Project Member
Wed Sep 7 16:19:03 2011, comment #7:

Like I said, double buffering now needs to be refactored, so I don't see this as a regression at all. It was never enabled by default, and none of our shipped packages nor releases ever had this enabled. Rather than being a bug it should be a future task.

Rob Savoye <rsavoye>
Project AdministratorIn charge of this item.
Wed Sep 7 16:15:59 2011, comment #6:

I've just checked. Double buffering in FB/AGG works as of Gnash 0.8.10dev (premerge-20729-71d22cb)

W/out double buffering the playback is horribly flickering, while with is a pleasure.

Being or not enabled by default doesn't change the fact that FB/AGG used to work nicely with double buffering and now isn't anymore. That's the regression part.

Sandro Santilli <strk>
Project Member
Wed Sep 7 16:07:46 2011, comment #5:

It didn't work months ago when I tested it, but as all the code has changed heavily around it, I don't think it matters. This wouldn't be a regression anyway, as it was never enabled by default anyway. The double buffering code needs to be refactored so it works with hardware based buffering, which the current implementation failed to do.

Rob Savoye <rsavoye>
Project AdministratorIn charge of this item.
Wed Sep 7 16:04:18 2011, comment #4:

I'm not sure I can confirm double buffering not being implemented before. I belive I added that option after getting too much flickering.

Do you want me to check it or will you ?

Sandro Santilli <strk>
Project Member
Wed Sep 7 15:57:01 2011, comment #3:

Yes, as it never worked anyway, and it'll be easy to add back in whenever somebody (maybe me) implements double buffering.

Rob Savoye <rsavoye>
Project AdministratorIn charge of this item.
Wed Sep 7 15:55:06 2011, comment #2:

I confirm I have ENABLE_DOUBLE_BUFFERING defined (as per --enable-doublebuf).

Should we take the chance to drop the configure switch as well ?

Sandro Santilli <strk>
Project Member
Wed Sep 7 15:46:21 2011, comment #1:

The FB GUI built for me on 10 different GNU/Linux distributions, so I would check if you have ENABLE_DOUBLE_BUFFERING set. The original code for double buffering never was finished by whomever worked on that. To double buffer now will require some refactoring to support hardware accelerated graphics. That was left as a future task.

Rob Savoye <rsavoye>
Project AdministratorIn charge of this item.
Wed Sep 7 15:26:29 2011, original submission:

My usual configure line:

'--enable-maintainer-mode' '--prefix=/extra/gnash-dev' '--disable-jemalloc' '--enable-fps-debug' '--with-swfdec-testsuite=/usr/src/swfdec/swfdec/test' '--enable-http-testsuite=http://localhost/' '--enable-python' '--enable-extensions=fileio,mysql' '--enable-doublebuf' '--enable-red5-testing' '--disable-static' '--enable-gui=gtk,qt4,sdl,fb,dump' '--enable-media=ffmpeg,gst'

Triggers a build failure:

/usr/src/gnash/gnash-head/gui/fb/fb.cpp: In destructor ‘virtual gnash::gui::FBGu
i::~FBGui()’:
/usr/src/gnash/gnash-head/gui/fb/fb.cpp:182: error: ‘buffer’ was not declared in this scope
/usr/src/gnash/gnash-head/gui/fb/fb.cpp: In member function ‘virtual void gnash::gui::FBGui::renderBuffer()’:
/usr/src/gnash/gnash-head/gui/fb/fb.cpp:381: error: ‘var_screeninfo’ was not declared in this scope
/usr/src/gnash/gnash-head/gui/fb/fb.cpp:383: error: ‘_drawbounds’ was not declared in this scope
/usr/src/gnash/gnash-head/gui/fb/fb.cpp:396: error: ‘m_rowsize’ was not declared in this scope
/usr/src/gnash/gnash-head/gui/fb/fb.cpp:398: error: ‘fbmem’ was not declared in this scope
/usr/src/gnash/gnash-head/gui/fb/fb.cpp:398: error: ‘buffer’ was not declared in this scope
make[4]: *** [fb_gnash-fb.o] Error 1

I'm not sure which other architecture will have this problem, as the FB gui was disabled by default by a recent commit of Rob.

Sandro Santilli <strk>
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 bwy (Posted a comment)
  • -unavailable- added by rsavoye (Updated the item)
  • -unavailable- added by strk (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 12 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Mon Dec 5 13:00:46 2011strkStatusReady For Test=>Fixed
      Open/ClosedOpen=>Closed
    Mon Dec 5 05:15:08 2011rsavoyeCategorybuild=>gui-fb
      StatusPostponed=>Ready For Test
    Wed Sep 7 16:19:03 2011rsavoyeStatusConfirmed=>Postponed
      Assigned toNone=>rsavoye
    Wed Sep 7 16:15:59 2011strkStatusNeed Info=>Confirmed
      Summary[REGRESSION] FB gui doesn\'t build=>[REGRESSION] FB gui doesn't build with double buffering enabled
    Wed Sep 7 16:07:46 2011rsavoyeSummary[REGRESSION] FB gui doesn\'t build with double buffering enabled=>[REGRESSION] FB gui doesn't build
    Wed Sep 7 16:05:07 2011strkSummary[REGRESSION] FB gui doesn\'t build=>[REGRESSION] FB gui doesn't build with double buffering enabled
    Wed Sep 7 15:46:21 2011rsavoyeSeverity5 - Blocker=>4 - Important
      StatusNone=>Need Info

    Back to the top


    Powered by Savane 3.1-cleanup