#gemrb@irc.freenode.net logs for 22 Jan 2015 (GMT)

Archive Today Yesterday Tomorrow
GemRB homepage

[01:05:30] <-- turtleman_ has left IRC (Read error: Connection reset by peer)
[01:31:14] <-- |Cable| has left IRC (Ping timeout: 246 seconds)
[01:43:40] <-- brada has left IRC (Quit: brada)
[01:47:37] --> |Cable| has joined #gemrb
[01:56:20] --> brada has joined #gemrb
[06:09:05] <-- Lightkey has left IRC (Ping timeout: 265 seconds)
[06:21:49] --> Lightkey has joined #gemrb
[07:51:16] <-- fuzzie has left IRC (Ping timeout: 255 seconds)
[08:17:34] --> fuzzie has joined #gemrb
[08:37:27] <-- brada has left IRC (Quit: brada)
[15:00:05] <lynxlynxlynx> the scrolling on append is pretty! :)
[15:19:40] <edheldil> where can it be seen?
[15:29:51] <lynxlynxlynx> pause the game
[15:30:15] <lynxlynxlynx> maybe just a bit too slow, but that's trivial to change
[15:47:42] <Pepelka> [commit] lynxlynxlynx: DCClassDonePress: restore the main window, so we redraw properly https://github.com/gemrb/gemrb/commit/d9b16c14dddcb661ebd7d50b9ac398f265753493
[15:47:44] <Pepelka> [commit] lynxlynxlynx: avoid dualclassing problems with (near) full inventories https://github.com/gemrb/gemrb/commit/a824423ed8a3e778149c23f68ff268c2577d491a
[15:47:45] <Pepelka> [commit] lynxlynxlynx: Charanimations: removed extraneus ; https://github.com/gemrb/gemrb/commit/7b86af97ddc7e11621f45453eda59e5dcee26369
[15:47:46] <Pepelka> [commit] lynxlynxlynx: CHUImporter::GetWindow: make sure the animation factory is fine https://github.com/gemrb/gemrb/commit/bbe4bad13d94f74d5a2e0c19180b0a1b0a09c7ca
[15:47:47] <Pepelka> [commit] lynxlynxlynx: Projectile::GetTarget: avoid possible crash https://github.com/gemrb/gemrb/commit/942ebd800b8e362f92e278b455669de644822e8c
[17:41:42] <Pepelka> [commit] lynxlynxlynx: Scrollbar: mark IsOpaque const (like the rest) https://github.com/gemrb/gemrb/commit/2bbbeb1d48e971d7f3227a9b849f561c5b6ce5ba
[17:41:43] <Pepelka> [commit] lynxlynxlynx: Map::BlockSearchMap: removed dead line https://github.com/gemrb/gemrb/commit/7834d28466bf9397a7a86ebe58c572123a73ec5f
[17:41:44] <Pepelka> [commit] lynxlynxlynx: Container::CreateGroundIconCover: removed debug code from 2005 https://github.com/gemrb/gemrb/commit/c85e11499be53ccbcf9108f84e40988fb15f96a8
[17:41:46] <Pepelka> [commit] lynxlynxlynx: fx_teleport_to_target: avoid potential crash https://github.com/gemrb/gemrb/commit/7e0e4936c633c14f1f86067fd23591223236cb50
[17:41:47] <Pepelka> [commit] lynxlynxlynx: bg2::cg: one spacing fix and one not working spacing fix https://github.com/gemrb/gemrb/commit/d878986f18f1303f4284351a6312d013bc7a15f8
[18:03:46] --> brada has joined #gemrb
[18:15:52] <brada> yeah, i simply guessed at the animation speed
[18:16:01] <brada> but it is easily changed
[18:17:03] <brada> also made the chapter text much better, but again maybe too fast/slow
[18:21:28] <lynxlynxlynx> my last commit has a good test case for you
[18:21:48] <lynxlynxlynx> non-newline-only strings eat leading newlines
[18:25:11] <brada> was there text before that, is it an append?
[18:25:35] <brada> ah
[18:25:48] <brada> so jsut a guess here
[18:26:10] <brada> i would think that since it is an append then that first newline is just to move down to the next line
[18:26:57] <brada> so 1 \n to move away from the partial line then n - \n of empty lines
[18:36:17] <lynxlynxlynx> it moves one line, not 10
[18:36:23] <lynxlynxlynx> should be two, just made it obvious
[18:40:01] <brada> oh i see so its dropping contiguous newlines, but only if the strings also contain other text afterwards?
[18:42:17] <brada> i should start keeping track of these as test cases...
[18:42:37] <brada> maybe try to automate something…
[18:44:10] <lynxlynxlynx> can we create a ta in software?
[18:44:36] <lynxlynxlynx> the minimal test is probably too slim
[19:05:33] --> Driim has joined #gemrb
[20:06:51] <brada> yes, unfortunately that seems to be the case
[20:07:07] <brada> but the demo maybe can be hijacked with some test cases
[20:08:01] <lynxlynxlynx> yes, it has a functioning ta
[20:08:19] <lynxlynxlynx> i'll set it up
[20:08:31] <lynxlynxlynx> guirec vs none is fine now btw
[20:08:52] <lynxlynxlynx> it's the character generators that need a lot of fixing
[20:08:58] <lynxlynxlynx> some have no newlines at all
[20:36:22] <-- Driim has left IRC (Quit: Leaving)
[21:10:09] <lynxlynxlynx> fuzzie: 1. Object: is the same as using [0], right?
[21:10:26] <fuzzie> no
[21:10:35] <fuzzie> I think :)
[21:10:52] <fuzzie> there are different checks applied
[21:14:18] <lynxlynxlynx> ok
[21:14:47] <lynxlynxlynx> perhaps just our default, since the particular action doesn't take a target
[21:40:45] <lynxlynxlynx> hmpf, i can't find where objects[0] is set; this time it's not Targets::AddTarget
[21:42:35] --> kpederse1 has joined #gemrb
[21:45:23] <-- kpedersen has left IRC (Ping timeout: 240 seconds)
[22:37:16] <lynxlynxlynx> seems all 6 actions (all of them, killed everyone else) get the same treatment (good) - all their objects[] are set to and then the actionoverride fails
[22:37:55] <lynxlynxlynx> i'm breaking everywhere they're set, but can't find this and it doesn't look like it'd happen at startup
[22:38:57] <lynxlynxlynx> it's a custom script i dumped as dplayer3.bcs
[22:39:08] <lynxlynxlynx> can't get more trivial than this: http://sprunge.us/YYQX
[22:40:40] <Pepelka> [commit] lynxlynxlynx: iwd2: reduced elemental feet circles to match their anims https://github.com/gemrb/gemrb/commit/dc22e6c8bf1048cd18777fb52968264cc65cf373
[22:40:41] <Pepelka> [commit] lynxlynxlynx: iwd2: fixed everything related to the party ai button https://github.com/gemrb/gemrb/commit/6ffdaeee5f618f1fd9e6c25a23ab03fd7fdbd870
[22:40:42] <Pepelka> [commit] lynxlynxlynx: SetupMenuWindowControls: iwd2 needs no conditional display of gears https://github.com/gemrb/gemrb/commit/a98cfdba1c2bc97e866f8cf96b924959803da3d9
[22:55:58] <lynxlynxlynx> oh, but it does happen at start -.-
[22:59:11] <lynxlynxlynx> GemRB::DecodeObject gets called with "0 0 0 0 0 0 0 0 0 0 0 0 0 \"\"0 0 OB", so we end up storing as the object name
[22:59:24] <lynxlynxlynx> sounds like a parser issue