#exult@irc.freenode.net logs for 17 Jan 2004 (GMT)

Archive Today Yesterday Tomorrow
Exult homepage

[00:24:05] --> Kirben has joined #exult
[00:24:05] --- ChanServ gives channel operator status to Kirben
[00:41:54] <-- Matt_O has left IRC ("Client Exiting")
[00:57:22] <-- Andersen has left IRC ("FUTUE TE ET IPSUM CABALLUM")
[00:58:07] --> Andersen has joined #exult
[01:46:57] --> Cahaan has joined #exult
[02:59:45] <servus> http://nuvie.sourceforge.net/temp/Nuvie05.jpg :D
[04:15:26] <-- Andersen has left IRC ("FUTUE TE ET IPSUM CABALLUM")
[04:21:50] --> Matt_O has joined #exult
[07:42:24] <-- Cahaan has left IRC (Read error: 54 (Connection reset by peer))
[09:20:38] --> Ember has joined #exult
[09:31:54] <-- Darke has left IRC (Read error: 60 (Operation timed out))
[10:03:09] --- Ember is now known as Darke
[10:03:14] --- ChanServ gives channel operator status to Darke
[13:08:25] <wjp> hm, looks like Jeff wants to release 1.2 next week
[13:21:16] --> Colourless has joined #Exult
[13:21:21] --- ChanServ gives channel operator status to Colourless
[13:23:32] <wjp> hi
[13:23:44] <Colourless> hi
[13:24:04] * wjp wonders if anything has to be done before 1.2
[13:24:14] <wjp> I guess at least that fmopl bug has to be fixed
[13:27:51] <wjp> OplDriver::LoadU7VBank doesn't check if the file it tries to open exists
[13:27:55] <wjp> (and doesn't catch any exceptions)
[13:32:14] <wjp> Colourless: is it ok if LoadMT32Bank just returns silently if the file it wants to open isn't found?
[13:32:33] <Colourless> that should be fine
[13:33:32] <Colourless> it should still work fine without the bank
[13:34:01] <Colourless> there just wont be a drum track (for the opening tune) if it fails to load the bank
[13:34:33] <wjp> currently it's pretty much expected that it won't be able to load it
[13:35:00] <Colourless> i'm curious why it doesn't actually work now. it used to be fine
[13:35:42] <servus> I am off to buy...
[13:35:46] <servus> A SHRUBBERY!
[13:35:56] <wjp> ni!
[13:35:58] <servus> Not too big of one, but a nice one.
[13:36:05] * servus waves
[13:36:15] <servus> Check out the original lighting in Nuvie :D
[13:36:51] <Colourless> might have worked in a version before i committed it to cvs
[13:37:16] * servus . o O ( Ni ni ni ni ni ni ni )
[13:37:18] <-- servus has left IRC ()
[13:37:57] <wjp> LoadXMIDIBank does catch a file_open_exception, strangely
[13:39:30] <Colourless> ah ha
[13:39:36] <Colourless> i now know why :-)
[13:39:59] <Colourless> when i last changed Midi.cc i made it attempt to load the patches
[13:40:16] <wjp> ah, so load_patches got called when it didn't start a game yet
[13:40:27] <Colourless> where originally it would only load them once a game had been started
[13:40:47] <Colourless> the fix of course it to get it to catch the exception
[13:40:58] <Colourless> since loading the patches is useful
[13:41:00] <wjp> I copied the exception bit from XMIDI to U7V
[13:41:39] <Colourless> good good
[13:41:45] <wjp> but I don't like having that error message in a 'normal' startup sequence
[13:42:11] <Colourless> it shouldn't be an error message. More of a 'warning'
[13:42:51] <wjp> but still, it's confusing to give a warning/error which can't be fixed
[13:43:03] <wjp> (fixed by the user, that is)
[13:44:38] <Colourless> but they 'can' fix it
[13:44:44] <Colourless> sort fo
[13:44:47] <Colourless> depending on the os
[13:45:07] <Colourless> if exult is installed in the bg directory it will work with no error :-)
[13:45:08] <wjp> I wouldn't call having a 'static' subdirectory in the current directory a fi x:-)
[13:45:16] <wjp> s/ x/x /
[13:45:58] <Colourless> hey, if you wanted to you could make the static dir's path default to the value for bg
[13:46:08] <Darke> You could always mkdir/rmdir a directory if desperate. *grin*
[13:46:28] <wjp> that might be possible
[13:46:56] <wjp> hm, I wonder why Game::game_type defaults to BLACK_GATE
[13:47:05] <wjp> NONE should be more appropriate
[13:47:49] <Colourless> if you change default static dir you need to catch when you return to the Exult menu from SI
[13:50:32] <wjp> hm
[14:02:48] --> Cahaan has joined #exult
[14:03:27] * wjp is confused
[14:03:45] <wjp> is_system_path_defined("<PATCH>") apparently is always returning true
[14:03:57] <wjp> apparently because <PATCH> is set to an empty string
[14:04:31] <wjp> there were a couple of places in get_system_path where path_map[key] was accessed without checking if the entry exists, but I got rid of those
[14:06:50] <wjp> ah
[14:07:22] <wjp> SERPENTISLE_PATCH was set to ""
[14:10:37] <wjp> ok, path_map can't be filled up with empty entries now anymore, so is_system_path_defined actually returns something sensible now
[14:11:07] --> Andersen has joined #exult
[14:15:17] <wjp> hm, btw, shouldn't we use Mix_GetError after Mix_OpenAudio (instead of SDL_GetError) ?
[14:15:26] <wjp> (in Audio.cc)
[14:15:31] <Colourless> you'd think so
[14:18:49] <wjp> (re. patches:) we could also add a BG_STATIC system path similar to SERPENT_STATIC
[14:19:26] <wjp> hm, come to think of it, we already define BLACKGATE_STATIC, SERPENTISLE_STATIC
[14:19:34] <wjp> (but only after loading the patches currently)
[14:20:07] <wjp> and not when running exult --game <something>
[14:20:51] <wjp> we could set these (by calling get_game_paths) automatically for all games in the config
[14:22:35] --> SB-X has joined #exult
[14:33:54] <-- Andersen has left IRC (Read error: 110 (Connection timed out))
[14:43:49] <wjp> hm, SERPENT_STATIC is 'broken' in a way
[14:44:05] <wjp> it is set to serpentisle/path + "/static"
[14:44:43] <wjp> I guess I'll just replace that by serpentisle_STATIC everywhere
[14:45:02] <Colourless> huh?
[14:45:03] <wjp> that way at least it's handled consistently
[14:46:04] <wjp> (there's also a serpentisle/static config setting that could override the static dir)
[14:55:50] <-- SB-X has left IRC (Read error: 104 (Connection reset by peer))
[14:55:51] --> sbx has joined #exult
[14:58:22] --> Cah_AW has joined #exult
[14:58:38] <-- Cahaan has left IRC (Read error: 104 (Connection reset by peer))
[15:01:28] <wjp> ok, this should fix it
[15:08:12] <wjp> committed
[15:08:22] <wjp> Colourless: could you briefly check if I didn't break anything too badly? :-)
[15:08:36] <wjp> (anything fmopl-related, that is)
[15:09:00] <Colourless> well, it's as simple as this. if it works, then it should be ok.
[15:09:29] <Colourless> of course i'll still check
[15:10:28] <Kirben> Could OpenGL path issue be fixed for next release too ?
[15:10:35] <wjp> which issue is that?
[15:10:43] * wjp looks at bug tracker
[15:10:54] <wjp> is it in there?
[15:11:15] <Kirben> It open file before paths are set if I remember right.
[15:12:27] <Colourless> probably doing the same 'static' trick i was doing
[15:12:42] <wjp> opengl backend isn't really all that supported, though
[15:12:55] <Colourless> it's also not going to be enabled in the release
[15:14:27] <Kirben> gamewin.cc line 280 I think.
[15:15:37] <wjp> yes, indeed
[15:16:02] <wjp> nice comment: "Hope this is okay to do" :-)
[15:16:39] <wjp> I wonder why we don't load exult's own palette at that point
[15:17:32] <Colourless> because it's the palette that is actually used for 'everything' opengl :-)
[15:17:36] <Colourless> IIRC
[15:17:37] <wjp> but given my extensive (*cough*) knowledge of the opengl backend, I'd rather leave it alone :-)
[15:17:40] <wjp> ah :-)
[15:18:03] <wjp> Kirben: you could file a bug report and assign it to Jeff
[15:18:04] <Colourless> hey, if you want to change things so it actually changes palettes when you choose options.... :-)
[15:18:14] <wjp> Colourless: I don't; I really don't :-)
[15:18:21] <Kirben> ok
[15:18:41] * Colourless doesn't want to either. He is knee deep in other OpenGL code at the moment
[15:18:50] <wjp> q2?
[15:18:54] <Colourless> yeah
[15:19:19] <Colourless> i still don't know what i'm doing what i am doing though :-)
[15:19:23] <wjp> let's see... what other things should be cleaned up before release?
[15:19:26] <Colourless> s/what/why/
[15:20:05] <wjp> heh :-)
[15:20:13] <wjp> just messing around a bit? :-)
[15:23:47] <Colourless> yes just a bit
[15:23:56] <Colourless> wow, the exult output has cleaned up a bit since the last time i compiled it
[15:24:31] <wjp> at exult startup?
[15:24:35] <Colourless> yes
[15:24:40] <-- Cah_AW has left IRC (herbert.freenode.net irc.freenode.net)
[15:24:44] * wjp cleaned that up a few weeks ago
[15:26:52] --> Cah_AW has joined #exult
[15:27:54] <wjp> hm
[15:28:13] <wjp> I get a stuck note when the first song that plays in a new BG game finishes
[15:28:16] <wjp> (fmopl)
[15:28:27] <wjp> which is really annoying :-)
[15:31:09] <Colourless> so there is
[15:48:19] <Colourless> ok, well i know what the problem is but properly fixing it might be a challenge, but i can fix it 'unproperly' :-)
[15:49:55] <-- Rixa has left IRC (herbert.freenode.net irc.freenode.net)
[15:52:03] --> Rixa has joined #exult
[15:55:55] <Colourless> committed fix for the hanging notes problem
[15:59:06] <wjp> seems to work
[15:59:55] <wjp> wow, fmopl really produces a lot of output :-)
[16:00:28] <Colourless> don't run in debug mode :-)
[16:03:04] * wjp blinks
[16:03:07] <wjp> peculiar test:
[16:03:30] <wjp> if (!U7exists(U7NBUF_DAT) && !U7exists(NPC_DAT) && !Game::is_editing())
[16:03:45] <wjp> (when checking if gamedat is valid)
[16:04:42] <Colourless> the !U7exists(U7NBUF_DAT) should be U7exists(U7NBUF_DAT)
[16:04:56] <Colourless> actually that entire line should probably be
[16:05:05] <Colourless> if (U7exists(U7NBUF_DAT) || (!U7exists(NPC_DAT) && !Game::is_editing()))
[16:05:12] <wjp> yes, that would make much more sense
[16:05:18] <wjp> since both conditions are errors
[16:06:04] <wjp> that should fix a crash when running with an original gamedat
[16:06:42] <-- sbx has left IRC (Read error: 104 (Connection reset by peer))
[16:06:43] --> EsBee-Eks has joined #exult
[16:43:59] <wjp> Colourless: any idea what to do with all those 'Und sched. opcode 0x00' messages?
[16:44:47] <Colourless> ignore them?
[16:45:31] <wjp> that would be one option :-)
[17:15:33] <-- EsBee-Eks has left IRC (Read error: 104 (Connection reset by peer))
[17:15:33] --> sbx has joined #exult
[17:29:01] <-- sbx has left IRC ("...")
[18:26:55] <-- Kirben has left IRC (Read error: 104 (Connection reset by peer))
[21:56:31] <-- Colourless has left IRC ("casts invisibility")
[23:58:43] --> Kirben has joined #exult
[23:58:43] --- ChanServ gives channel operator status to Kirben