03:35:31 Experimental (bcrawl) branch on underhound.eu updated to: 0.23-a0-5261-gd9800d219b 09:14:06 03CrawlOdds02 07* 0.35-a0-527-g064b58b805: Separate hidden spells for divine exegesis 10(8 months ago, 5 files, 24+ 2-) 13https://github.com/crawl/crawl/commit/064b58b80551 13:28:21 03CrawlOdds02 07* 0.35-a0-528-g197fc7faa9: Update @ attack delay to be rounded to the nearest integer 10(2 hours ago, 1 file, 2+ 2-) 13https://github.com/crawl/crawl/commit/197fc7faa9ea 13:28:21 03CrawlOdds02 07* 0.35-a0-529-g2f29fc7f20: Describe attack delay changes on all items 10(49 minutes ago, 7 files, 133+ 121-) 13https://github.com/crawl/crawl/commit/2f29fc7f2009 15:43:38 Unstable branch on underhound.eu updated to: 0.35-a0-529-g2f29fc7f20 (34) 22:36:04 Unstable branch on crawl.develz.org updated to: 0.35-a0-529-g2f29fc7f20 (34) 22:38:17 03DracoOmega02 07* 0.35-a0-530-g74ec0dc3e0: Fix monster bot not handling unicode monster glyphs properly 10(2 minutes ago, 1 file, 2+ 1-) 13https://github.com/crawl/crawl/commit/74ec0dc3e08c 22:38:17 03DracoOmega02 07* 0.35-a0-531-gbd6a7f100e: Fix monster bot not handling AF_BURSTSHROOM 10(88 seconds ago, 1 file, 2+ 0-) 13https://github.com/crawl/crawl/commit/bd6a7f100e29 22:41:51 <04d​racoomega> @Odds A minor quibble with your last commits (which look like a good refactoring of all those outparams at this point, tbh >.>). You added an ignore_temp argument to some functions, but almost all of the many examples of similar arguments in other functions have inverse semantics (ie: include_temp, true by default). It feels like it's probably best if the methods all agree with each other in that regard, where plausible (instead 22:41:52 of referencing which ones want true and which want false.) (...though I will admit that 'ignore_temp' may actually make more sense to me as an argument than 'temp', whose semantics are a little more vague. Does that mean it would be a good idea to flip all the others??? >.>) 23:00:00 Windows builds of master branch on crawl.develz.org updated to: 0.35-a0-531-gbd6a7f100e 23:27:22 <11O​dds> Ah interesting… the other ones that the “preview stats” code path calls use the “ignore temp” direction, which is what I was being consistent with (not that I thought especially hard here). And yeah I agree on semantics I think - when doing the forbidding change I messed up the direction of “include temp” more times than I care to admit in ways that I think I wouldn’t have for “ignore temp” 23:28:10 <04d​racoomega> Yeah, I tend to agree 23:29:19 <11O​dds> Maybe I’ll go flip the include temp ones 23:31:23 <04d​racoomega> Makes sense to me 23:33:46 Unstable branch on cbro.berotato.org updated to: 0.35-a0-531-gbd6a7f100e (34) 23:56:08 Monster database of master branch on crawl.develz.org updated to: 0.35-a0-531-gbd6a7f100e