#gemrb@irc.freenode.net logs for 29 Dec 2010 (GMT)

Archive Today Yesterday Tomorrow
GemRB homepage

[00:22:30] <-- Maighstir has left IRC (Read error: Connection reset by peer)
[00:52:39] <-- _pickle has left IRC (Remote host closed the connection)
[01:23:23] <-- edheldil_ has left IRC (Ping timeout: 240 seconds)
[02:00:25] --> raevol has joined #GemRb
[05:56:08] <-- raevol has left IRC (Quit: Leaving.)
[06:09:37] <-- maighstir|phone has left IRC (Ping timeout: 260 seconds)
[07:45:30] <-- spike411 has left IRC (Quit: Manga & anime pokec na Jabberu: manga.cz@conf.netlab.cz)
[08:26:48] --> lynxlynxlynx has joined #GemRb
[08:26:49] --- ChanServ gives channel operator status to lynxlynxlynx
[09:05:01] --> edheldil_ has joined #GemRb
[09:38:50] <CIA-29> GemRB: 03lynxlupodian * r0d41b0fa2c23 10gemrb/gemrb/core/Scriptable/ (ActorBlock.cpp ActorBlock.h): moved all the casting-prereqs checks into the new Scriptable::CanCast
[09:38:53] <CIA-29> GemRB: 03lynxlupodian * rb7227298a581 10gemrb/gemrb/override/ (bg2/wildmag.2da shared/dummy.spl): unhardcoded one wild surge
[09:38:55] <CIA-29> GemRB: 03lynxlupodian * r4b89197675b8 10gemrb/gemrb/override/bg2/wildmag.2da: wildmag.2da: converted to unix-style eol
[09:39:22] <edheldil> busy as bee, lynx :)
[09:40:02] <lynxlynxlynx> hack mode engaged :)
[09:40:18] <fuzzie> :-)
[10:06:17] <CIA-29> GemRB: 03lynxlupodian * r0b6fcf82c7bc 10gemrb/gemrb/core/DisplayMessage.cpp: added format example to DisplayMessage::DisplayConstantStringAction
[10:06:20] <CIA-29> GemRB: 03lynxlupodian * r745b8eae784a 10gemrb/gemrb/ (7 files in 7 dirs): added the "Wild Surge:" string reference
[10:22:59] --- barra_home is now known as barraAway
[10:34:14] --> spike411 has joined #GemRb
[13:10:22] <lynxlynxlynx> int Scriptable::CastSpell( const ieResRef SpellResRef,
[13:11:04] <lynxlynxlynx> what does the const do there? i thought SpellResRef would be immutable, but i can change it (the compiler didn't complain either)
[13:13:27] <wjp> what can you change exactly?
[13:13:58] <lynxlynxlynx> SpellResRef = someotherstuff
[13:14:32] <wjp> yes, you can change the pointer, but not the data pointed to
[13:15:10] <wjp> specifically that assignment doesn't affect the caller
[13:15:28] <lynxlynxlynx> mhm
[13:15:32] <wjp> the argument type is essentially 'const char*'
[15:57:32] --> Maighstir has joined #GemRb
[16:12:48] <CIA-29> GemRB: 03lynxlupodian * r601b47857f94 10gemrb/gemrb/ (5 files in 3 dirs): added Actor::GetCasterLevel
[16:12:49] <CIA-29> GemRB: 03lynxlupodian * r2cc59eb7262f 10gemrb/gemrb/core/Scriptable/Actor.cpp: fixed the kit check in Actor::GetWildMod (mages are different)
[16:12:52] <CIA-29> GemRB: 03lynxlupodian * r18b076a6ce81 10gemrb/gemrb/core/Scriptable/ (Actor.cpp Actor.h): avoid rerolling the wildmage level mod, since the bonus is needed multiple times per cast
[16:38:41] <lynxlynxlynx> so close, yet so far
[17:00:49] <CIA-29> GemRB: 03lynxlupodian * r56edb8ac7912 10gemrb/gemrb/plugins/FXOpcodes/FXOpcodes.cpp: fx_force_surge_modifier stores the value in the second parameter
[17:15:24] <CIA-29> GemRB: 03lynxlupodian * r47ead53eec1f 10gemrb/gemrb/core/Scriptable/ActorBlock.cpp:
[17:15:24] <CIA-29> GemRB: clear the specially forced surge after use
[17:15:24] <CIA-29> GemRB: (+a forgotten line from 18b076a6ce813c)
[17:21:31] <-- spike411 has left IRC (Quit: Manga & anime pokec na Jabberu: manga.cz@conf.netlab.cz)
[17:43:11] <-- edheldil_ has left IRC (Ping timeout: 272 seconds)
[17:45:22] <-- lynxlynxlynx has left IRC (Ping timeout: 276 seconds)
[17:53:37] --> lynxlynxlynx has joined #GemRb
[17:53:37] <-- lynxlynxlynx has left IRC (Changing host)
[17:53:37] --> lynxlynxlynx has joined #GemRb
[17:53:37] --- ChanServ gives channel operator status to lynxlynxlynx
[18:22:31] --- barraAway is now known as barra_home
[19:06:21] --> barra_away has joined #GemRb
[19:09:23] <-- barra_home has left IRC (Ping timeout: 240 seconds)
[19:44:44] --- barra_away is now known as barra_home
[20:03:23] --> edheldil_ has joined #GemRb
[20:17:53] <lynxlynxlynx> fuzzie: here?
[20:17:58] <fuzzie> yes
[20:18:11] <lynxlynxlynx> woah, that was speedy
[20:18:22] <fuzzie> just logged back onto irc :-)
[20:18:46] <lynxlynxlynx> i found out why we get those invalid spell header errors ([Spell]: Cannot retrieve spell header! required header: 6, maximum: 6)
[20:19:34] <lynxlynxlynx> SpellNoDec uses a shortcut for casting, so the variable isn't reset and is thus reused from before (it's not passed around)
[20:20:08] <lynxlynxlynx> i vaguely remember you mentioning that the whole lot of them was broken - did you have anything else in mind?
[20:20:13] <fuzzie> i hadn't seen any of those :)
[20:20:25] <fuzzie> i didn't look at what you changed earlier, if anything..
[20:20:34] <lynxlynxlynx> no, there's nothing related there
[20:21:20] <fuzzie> but i wasn't intending to change the shared variables.
[20:21:32] <lynxlynxlynx> at first i thought about just changing GetExtHeader to use the last one when the level is too high, but then i did this tracing and it seems it would only hide the problem
[20:21:46] <fuzzie> i was probably talking about the fact some of the instant actions aren't actually instant, which i think is still broken.
[20:23:46] <lynxlynxlynx> ok
[20:25:37] <fuzzie> but SpellHeader is always set in SpellCast and reset later? so LastTarget is the problem?
[20:27:23] <lynxlynxlynx> yes
[20:27:54] <lynxlynxlynx> for some reason gdb doesn't want to print it
[20:28:38] <fuzzie> it seems like that would be a problem in all circumstances
[20:29:31] <fuzzie> you could clear it in ReleaseCurrentAction()?
[20:29:39] <lynxlynxlynx> it's not, i have to repeat the process several times to reproduce
[20:31:53] <lynxlynxlynx> i'll try that
[20:32:21] <fuzzie> i don't know if that is disasterous :) but i don't see anything which would break
[20:40:03] <lynxlynxlynx> appears to have helped
[20:40:12] <lynxlynxlynx> fried several summons
[20:40:35] <lynxlynxlynx> we have a bug that makes testing easier - the dweomer spell never depletes
[20:45:02] <lynxlynxlynx> yep, can't make it happen again :)
[20:45:37] <lynxlynxlynx> hehe
[20:45:55] <lynxlynxlynx> it did now that i said it can't
[20:48:05] <lynxlynxlynx> same trace, required header: 15, maximum: 5
[20:48:26] <lynxlynxlynx> i also added LastTargetPos clearing in there, for the pointy casts
[20:50:47] --- barra_home is now known as barraAway
[20:51:50] <lynxlynxlynx> ah, this one is related to wild magic
[20:52:32] <lynxlynxlynx> haven't figured out how to swap spells properly, so now it looks like it took the header number of the new one and applied it to the original
[20:52:49] <lynxlynxlynx> need to look at it with a normal mage
[20:55:39] <fuzzie> the relocated SF_SIMPLIFIED_DURATION always unnerves me :)
[20:59:18] --> Bo_Thomsen has joined #GemRb
[21:01:25] <lynxlynxlynx> nothing to worry about; yet
[21:08:42] <-- Bo_Thomsen has left #GemRb
[22:07:46] --> pupnik has joined #GemRb
[22:40:28] <lynxlynxlynx> i'm puzzled as where to put this wild surge stuff
[22:41:03] <lynxlynxlynx> i have the basics written up and polished, but there are two things that make it problematic
[22:41:06] <fuzzie> Scriptable::SpellCast doesn't work?
[22:41:34] <lynxlynxlynx> that's where i added it (ok, both CastSpell* that call it)
[22:42:13] <lynxlynxlynx> one problem is that i need to communicate this back high up since CastSpell*End needs to know about the new spell to create the right projectile
[22:42:23] <lynxlynxlynx> and possibly others too, since there are many entry points
[22:42:35] <fuzzie> oh, that will be a nighmare
[22:42:40] <fuzzie> since you can't cange the actions
[22:42:42] <fuzzie> change
[22:42:52] <lynxlynxlynx> the other problem are the hardcoded surges, which require very lowlevel changes to be applied
[22:43:14] <lynxlynxlynx> doubling the spell duration or projectile speed is even lower than here
[22:43:25] <lynxlynxlynx> so it's streched both ways :(
[22:47:45] <fuzzie> Uc.
[23:01:10] <lynxlynxlynx> any ideas? the best i could come up with is out-of-band structural bloat, so both ends can be dealt with
[23:01:57] <fuzzie> sorry, was trying to deal with slight disaster
[23:02:31] <lynxlynxlynx> deal with it, there is no rush
[23:02:32] <fuzzie> the most obvious approach does indeed sound like some out-of-band stuff, the same as LastTarget etc
[23:05:12] <fuzzie> and disaster is over :)
[23:14:10] <lynxlynxlynx> i'll commit what i have, to save it and make it easier to ponder upon
[23:16:46] <CIA-29> GemRB: 03lynxlupodian * r37437b45d02d 10gemrb/gemrb/core/Scriptable/ (ActorBlock.cpp ActorBlock.h): try to indulge the born to be wild
[23:34:26] <-- lynxlynxlynx has left IRC (Read error: Connection reset by peer)
[23:37:54] <-- pupnik has left IRC (Quit: leaving)
[23:40:13] --> pupnik has joined #GemRb
[23:53:37] <-- Maighstir has left IRC (Quit: ~ Trillian Astra - www.trillian.im ~)