Bug 466738

Summary: Restarting a level crashes xmoto
Product: [Fedora] Fedora Reporter: Rodd Clarkson <rodd>
Component: xmotoAssignee: Gwyn Ciesla <gwync>
Status: CLOSED RAWHIDE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: high Docs Contact:
Priority: medium    
Version: rawhideCC: gwync, hdegoede
Target Milestone: ---   
Target Release: ---   
Hardware: i386   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2008-10-15 14:46:54 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:

Description Rodd Clarkson 2008-10-13 11:09:51 UTC
Description of problem:

When I play xmoto in f9, restarting a level works.

When I play xmoto in f10 (rawhide) if I press enter or play level again after failing a level, xmoto quits.


Version-Release number of selected component (if applicable):

xmoto-0.4.2-3.fc10.i386


How reproducible:

Every time.

Comment 1 Gwyn Ciesla 2008-10-13 12:48:13 UTC
Can you run this from a terminal, and report any error messages?  A stack trace would also be helpful.  Thanks!

Comment 2 Rodd Clarkson 2008-10-14 10:47:18 UTC
[rodd@moose ~]$ xmoto 

ODE INTERNAL ERROR 1: assertion "g_bODEInitialized" failed in dCloseODE() [odeinit.cpp]
Aborted
[rodd@moose ~]$ 


If this isn't enough, then I might need a little reminding how to get a stack trace.  (It's been a while ;-[)

Comment 3 Gwyn Ciesla 2008-10-14 12:32:51 UTC
Checking with the ODE maintainer, to see if my patch is bollocks. . .

Comment 4 Hans de Goede 2008-10-15 14:46:54 UTC
Interesting xmoto was already calling CloseOde() upon destruction of the BikePlayer object, but not initOde, as it was doing this initOde needs to be called from the BikePlayer init routine instead of from main, so that it gets initialized each time the BikePlayer gets reset (as it first gets closed then).

I've just committed a new patch doing this to CVS and a fixed xmoto is building now, closing this.

Comment 5 Gwyn Ciesla 2008-10-15 15:00:29 UTC
Thanks, Hans!