-
Notifications
You must be signed in to change notification settings - Fork 70
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
I'll carry on working on the tutorial mainly in the branch, but I'd rather be able to get it tested via nightlies a bit at a time to spread out the feedback. The scenario support and new start menu are also needing testing. So further merges from this branch will happen in future.
- Loading branch information
Showing
31 changed files
with
5,537 additions
and
147 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,21 @@ | ||
( | ||
{ | ||
"file" = "oolite-standard.oolite-save"; | ||
"name" = "[oolite-scenario-standard-name]"; | ||
"description" = "[oolite-scenario-standard-description]"; | ||
"model" = "cobra3-trader"; | ||
}, | ||
{ | ||
"file" = "oolite-tutorial.oolite-save"; | ||
"name" = "[oolite-scenario-tutorial-name]"; | ||
"description" = "[oolite-scenario-tutorial-description]"; | ||
"model" = "coriolis-station"; | ||
"scenario" = "oolite-tutorial"; | ||
}, | ||
{ | ||
"file" = "oolite-strict.oolite-save"; | ||
"name" = "[oolite-scenario-strict-name]"; | ||
"description" = "[oolite-scenario-strict-description]"; | ||
"model" = "cobra3-trader"; | ||
} | ||
) |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
b17b39a
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Two bugs found with this commit on Windows:
AL lib: (EE) alc_cleanup: 1 device not closed
08:29:40.089 [exception]: ***** Exception checking controls [pollDemoControls]: NSRangeException : Index -3 is out of range 4 (in 'objectAtIndex:')
08:29:40.222 [exception]: ***** Exception checking controls [pollDemoControls]: NSRangeException : Index -3 is out of range 4 (in 'objectAtIndex:')
08:29:40.292 [exception]: ***** Exception checking controls [pollDemoControls]: NSRangeException : Index -3 is out of range 4 (in 'objectAtIndex:')
(...)
The bugs happen both with and without OXPs.
b17b39a
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
During a very quick smoke test, I got this on launching in the tutorial:
b17b39a
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
in the log. That might be causing the problems on Windows. I'll have a look at solutions to that.
I think I know where the appendString exception is coming from, too, though that doesn't happen here either.
b17b39a
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
5ac0101 in the branch should fix these up. Only the demoControls exception happened here though, so the others are a bit speculative.