00:00:25 Unstable branch on CRAWL.XTAHUA.COM updated to: 0.23-a0-642-g3120c2f (34) 01:27:38 Unstable branch on crawl.develz.org updated to: 0.23-a0-642-g3120c2f (34) 01:59:06 Windows builds of master branch on crawl.develz.org updated to: 0.23-a0-642-g3120c2f 02:32:06 -!- muffindrake1 is now known as muffindrake 02:58:37 Monster database of master branch on crawl.develz.org updated to: 0.23-a0-642-g3120c2f 03:01:13 Right, Mace of Variability, Ogres, what's next... 03:30:24 -!- amalloy_ is now known as amalloy 03:31:17 Experimental (bcrawl) branch on crawl.kelbi.org updated to: 0.23-a0-1278-gf71fcbb114 09:10:31 * Pinkbeast is a bit confused as to why the "midpoint" of most durations doesn't turn up anywhere near the midpoint 09:10:44 ... if anyone can explain it to me in short words I'd be grateful 09:38:07 Also messages for them running out seem to be in about a hundred places 10:13:45 03NormalPerson702 07https://github.com/crawl/crawl/pull/927 * 0.23-a0-643-g7af11aa: Convert spret_type to an enum class 10(22 minutes ago, 22 files, 459+ 454-) 13https://github.com/crawl/crawl/commit/7af11aa32c0b 10:13:45 03NormalPerson702 07https://github.com/crawl/crawl/pull/927 * 0.23-a0-644-gac94fc8: Rename spret_type to spret 10(7 minutes ago, 34 files, 653+ 653-) 13https://github.com/crawl/crawl/commit/ac94fc835142 11:48:38 Incorrect message shown on memorised spell which has become permanently useless. 13https://crawl.develz.org/mantis/view.php?id=11808 by damerell 12:05:28 Unstable branch on crawl.akrasiac.org updated to: 0.23-a0-642-g3120c2f (34) 12:20:21 IOOD says "This spell will have no effect right now because you can't see any valid targets", but could be used speculatively. 13https://crawl.develz.org/mantis/view.php?id=11809 by damerell 16:47:39 !seen elliptic 16:47:40 I last saw elliptic at Sat Dec 29 17:22:30 2018 UTC (1d 4h 25m 9s ago) quitting, saying 'Quit: Leaving'. 16:55:45 ebering: Sent you some bugs today, have fun :-) 17:04:31 send fixes too!! 17:04:32 !!!! 17:05:28 ebering: If I do you'll just curse me not following vanilla's whitespace conventions 17:06:20 there's util/checkwhite! 17:06:24 it'll do it for you 17:13:19 !tell elliptic I'm not sure how the qw in test/stress is kept up to date but it seems to be behind (see comment on pr 927) 17:13:21 ebering: OK, I'll let elliptic know. 17:14:21 ebering: I'm intentionally not following them because I'm doing it for me and hate left braces by themselves, but I could provide links of the form https://github.com/damerell/crawl/commit/f54d40c71df5a8692fd1a2a960848be6cd2f0309 if that would be helpful anyway 17:17:45 !tell elliptic current qw also chokes on acquirement scrolls because they no longer include a -- more -- if they're identified when read 17:17:46 ebering: OK, I'll let elliptic know. 17:19:54 ebering: elliptic maintains the qw in his repo and wasn't the person to check it into the crawl repo; if you want to update the one we have, you can feel free 17:20:15 Do any of our tests actually call that qw? 17:20:31 I recall that none didn't years back at the time it was added to the repo, but maybe one does now 17:20:47 if you want to get qw proper updated, you can also make a PR to elliptic's qw repo 17:20:51 qw in 0.22 is mainly broken because it doesn't know about things you can't do when engulfed, but it's got a weird problem with acquirement I haven't figured out yet 17:21:04 Which is https://github.com/elliptic/qw 17:21:17 ... that's elliptic's most recent, at least on github 17:21:51 yeah it's the one he updates, but as you see updates are pretty infrequent; he would probably merge a PR if someone made one though 17:22:19 Our tests don't run qw by default, but I wanted to use it to sweat PR 927 before merging 17:22:27 I've tracked down acquirement 17:22:33 ebering: Oh, do tell? 17:23:59 in plan_use_good_consumables it passes the string " b" (" c" if felid) to read2 17:24:04 the space being there to clear the -- more -- 17:24:08 but the more doesn't exist anymore 17:24:15 so deleting those two errant spaces fixes the issue 17:24:43 * Pinkbeast blinks, on looking apparently past-me has fixed that and forgotten about it. :-/ 17:24:53 Write more comments, past-me 17:27:05 ebering: Thanks for jogging present-me's memory :-/ 18:23:22 Unstable branch on underhound.eu updated to: 0.23-a0-642-g3120c2f5a9 (34) 21:24:38 New icon for "pinned" status with Wu Jian 13https://crawl.develz.org/mantis/view.php?id=11810 by kitchen_ace 21:41:00 hm. now that LRD can't destroy walls, why does it still allow you to target when there are no monsters in view? it can damage them, so that's a reason; but i'd rather take away that ability. is there a reason not ti? 22:05:49 would be strange to not be able to kill unseens with it, no? 22:06:21 can you even kill unseens with it 22:06:30 switch__: this is wrt bug 11809 and casting only on 'z', not force cast with 'Z' 22:08:22 amalloy: I think that as long as LRD can attack out of LOS it should be castable with 'z', since in principle it could have an effect 22:08:59 oic. i think an average player will assume that if an ability is castable, then it can affect their gameplay. so i agree with ^ 22:09:22 though, fireball can attack out of LOS and it prompts, so we're not 100% consistent here 22:09:34 I'll leave a note on the bug I guess 22:10:30 i'd take a lack of consistency in this sphere to be the result of assymetical development rather than intentional, it's probably pretty safe to make changes 22:14:21 comments like that aren't particularly helpful, we have the commit log and git blame 22:14:56 and logs to this channel 22:15:01 and the mailing list archives 22:15:27 do you disagree with what i said? 22:18:03 ebering: lots of stuff that "could" have an effect requires Z, because of invisible monsters 22:18:42 amalloy: yes, between when I sent an IRC message and added a note on the bug I came around to that point of view :) 22:19:03 if fireball can also damage out-of-los stuff i guess it's fine for LRD to do it too. i was just thinking, mostly we don't allow interaction with stuff out of los 22:22:25 and changing that seemed better than changing Z/z, if it was indeed a rare/new edge case 22:23:02 It is a rare edge case, and one that should probably be removed by removing the edge case 22:23:14 it's in the "future versions ideas" and has been for a long time 22:24:53 (removing the out of LOS attack, that is) 22:32:13 amalloy: Sorry, I didn't realise what a can of worms I was opening there... 23:05:08 03ebering02 07* 0.23-a0-643-g9ccc95a: Changelog through 0.23-a0-642-g3120c2f5 10(23 seconds ago, 1 file, 28+ 1-) 13https://github.com/crawl/crawl/commit/9ccc95a32006 23:17:05 Unstable branch on crawl.kelbi.org updated to: 0.23-a0-643-g9ccc95a320 (34) 23:19:02 -!- muffindrake1 is now known as muffindrake 23:59:34 Unstable branch on CRAWL.XTAHUA.COM updated to: 0.23-a0-643-g9ccc95a (34)