00:00:26 <|amethyst> I guess there is at least one function pointer typedef that doesn't match but where void would be safe to remove 00:00:31 <|amethyst> typedef BOOL (WINAPI *fc_func)(void); 00:01:14 <|amethyst> static void _run_test(const string &name, void (*func)(void)) 00:01:45 <|amethyst> looks like all the others are casts 00:01:58 Unstable branch on crawl.s-z.org updated to: 0.15-a0-999-gf638c35 (34) 00:02:23 -!- RZX has quit [Remote host closed the connection] 00:02:30 <|amethyst> so /[a-z)](void)/ 00:05:08 |amethyst: Nice regular expressions. 00:05:13 :D 00:05:28 -!- Kalir has quit [Changing host] 00:05:47 -!- CSDCScripter has quit [Read error: Connection reset by peer] 00:05:49 'z(void)' and 'z)(void)' would both match? 00:06:19 s/(?<=[a-z)])(void)// as a nice bonus from that, too 00:06:42 does anyone happen to know if tilegen shows up in the Dock because it's using SDL? 00:07:38 Hmm. Of course the [a-z)] means I'll need to use a backreferance. 00:08:06 <|amethyst> reaverb: what language? 00:08:13 |amethyst: Just vim. 00:08:26 <|amethyst> reaverb: /[a-z)]\zs(void)/ 00:08:26 uncertain how to persuade vim to accept pcre 00:08:36 \zs? 00:08:41 * floatingatoll hopes for TIL 00:08:43 <|amethyst> \zs says to start the match (and hence the replacement) at that point 00:08:46 ooooo 00:08:47 <|amethyst> in vim regex 00:08:56 <|amethyst> in Perl it's called \K and is rather new 00:09:04 hey, that's nice to know 00:09:08 thanks! 00:09:10 yeah, seems similar to those lookahead perl regexps 00:09:13 is it the same thing? 00:09:17 approximately 00:09:17 or lookbehind 00:09:18 rather 00:09:21 it removes the need to lookbehind 00:09:22 <|amethyst> it's simpler than lookbehind 00:09:31 you start the match at one place and the 'replacement' pointer later on 00:09:46 so instead of having to force them to be identical using a 'zero-width' lookbehind 00:09:48 <|amethyst> vim has lookahead and lookbehind too 00:09:55 you just say some stuff, and then \zs or \K depending, and it does the obvious thing 00:09:59 <|amethyst> (in fact, better lookbehind than Perl does) 00:10:07 since variable-width lookbehinds aren't an option in many perl/pcre, this is TERRIFIC 00:10:09 -!- hurdos has quit [Ping timeout: 258 seconds] 00:10:19 <|amethyst> (that's why vim's lookbehind is better) 00:10:23 heh 00:10:25 hrm, is there any functional difference? 00:10:37 compared to lookbehind 00:10:44 <|amethyst> gammafunk: with /g there is 00:10:47 ah 00:13:00 <|amethyst> The Perl documentation doesn't make this clear 00:13:13 <|amethyst> it says \K is a more efficient special case of (?<=), but 00:13:20 heh 00:14:00 <|amethyst> /(?<=.).a/ matches "banana" twice (both "na"s) 00:14:23 with /g and without the anchoring thing, right 00:14:25 ? 00:14:31 <|amethyst> /.\K.a/ matches "banana" once: it doesn't match the second 'na' because the preceding 'a' was already consumed 00:14:35 <|amethyst> err, yeah 00:14:36 -!- CSDCScripter has quit [Read error: Connection reset by peer] 00:15:23 \G i think it was 00:15:29 -!- demiskeleton_ has quit [Quit: Leaving] 00:16:54 ah, so lookbehind can match things that were already matched in a previous instance 00:17:15 <|amethyst> yes; it does what it says on the tin :) 00:17:27 right 00:18:08 that's good syntax though, so glad I learned about it 00:18:35 Unstable branch on crawl.develz.org updated to: 0.15-a0-999-gf638c35 (34) 00:18:47 often when I get to thinking about lookbehind I start thinking if there's a simpler solution 00:18:52 <|amethyst> yeah, in the 80% or whatever of cases where \K works, it's much nicer than (?<=) 00:19:02 -!- NeremWorld has quit [Ping timeout: 240 seconds] 00:19:29 <|amethyst> Best crawl comment: 00:19:30 <|amethyst> # Looks like no one told Larry Wall properties of regular expression, 00:19:30 <|amethyst> # including the part where they can't do arbitrarily nested parentheses. 00:19:36 heh 00:20:14 I saw that, didn't read the code really, and thought "must be an |amethyst comment" 00:20:24 <|amethyst> no, my English is better than that 00:20:28 <|amethyst> that's a kilobyte comment :) 00:20:35 heh, makes perfect sense 00:20:39 Hmm, for some reason " :argdo! %s/[a-z)]\zs(void)/()/ge | update " isn't catching everything. 00:20:42 <|amethyst> I did improve the regexp somewhat 00:21:16 |amethyst: Oh, sorry, I must not have seen it. 00:21:33 Geryon (03&) | Spd: 10 | HD: 15 | HP: 300 | AC/EV: 15/6 | Dam: 35(reach) | 05demonic, 10items, 10doors, fighter, see invisible, fly | Res: 06magic(120), 12drown, 04rot, 13neg+++, 13torm | Vul: 08holy++ | XP: 4427 | Sp: sum.hell beast | Sz: Giant | Int: normal. 00:21:33 %??geryon 00:21:40 <|amethyst> reaverb: no, I meant the one for nested parens in unbrace 00:21:49 |amethyst: Oh, sorry. 00:22:39 <|amethyst> reaverb: what's an example of something it missed? Seems to catch everything for me 00:24:13 |amethyst: Anything not caught by s/)(void)/(void)/ge 00:24:32 -!- CSDCScripter has quit [Read error: Connection reset by peer] 00:24:40 It caught only a couple function pointers, 00:25:04 |amethyst: Wow, never mind. 00:25:34 <|amethyst> mistyped "a-z"? :) 00:25:38 |amethyst: I've been using git reset —hard HEAD to reset after failed regrexes and apparently I commited at one point. 00:25:51 <|amethyst> ah 00:25:52 So it was going back to a version with the other (void) s removed. 00:25:53 (master>) ftl 00:26:22 also, you can right click in the adium input field and turn off autocorrect/autocomplete if you tire of it replacing -- with emdash 00:26:32 (this comes up a lot at work, figured it might be relevant :) 00:26:54 <|amethyst> replacing -- with emdash seems wrong 00:26:59 -!- bonghitz has quit [Quit: Leaving...] 00:27:03 <|amethyst> surely it should be en dash :) 00:27:09 |amethyst: It's my IRC client. 00:27:14 It's ridiculous. 00:27:32 floatingatoll: Oh, hmm. 00:27:33 comes from using OS X native NSInput or whatever fields, which ship autocorrect until disabled 00:27:42 How did you know it was Adium? /whois? 00:27:52 no, just statistical guess 00:28:01 reaverb: is Adium that bad? 00:28:03 there was a chance of Colloquy :) 00:28:14 the system-wide disable is under Keyboard prefs somewhere 00:28:18 or, per-input field 00:28:21 (might persist!) 00:28:32 <|amethyst> The system-wide disable is a hammer 00:28:45 reaverb: yeah whois does reveal you're using Adium 00:29:58 ... 00:30:14 I could probably change Real Name in the options somehow. 00:30:15 <|amethyst> I should go back to telling my client to lie in /ctcp version 00:30:24 reaverb: connection properties 00:30:29 -!- rast- has joined ##crawl-dev 00:30:59 floatingatoll: Ah. 00:31:32 is there any sense of order to enum.h? 00:31:38 I guess it's a security risk to reveal your irc client? 00:31:42 floatingatoll: Not really. 00:31:57 back in the day of "some clients run shell scripts with irc-provided input", yeah. nowadays, shrug. 00:32:15 it looks like enum.h is kinda sorted 00:32:19 -!- rast has quit [Ping timeout: 252 seconds] 00:32:28 -!- rast- is now known as rast 00:32:29 why does it say 'Global (ick) enums'? 00:33:42 floatingatoll: It's kind of bad almost the entire codebase knows the list of gods. 00:34:08 Also if you change enum.h you basically have to completely recompile. 00:34:12 true 00:34:21 and breaking it up into N files has readability issues, probably 00:35:17 does this resolve an attack.cc TODO correctly? http://atoll.pastebin.mozilla.org/5223937 00:35:17 floatingatoll: I've actually thoughten about doing that, not sure how bad the readability issues would be. 00:35:31 compiles, but i lack the deep C++ knowledge that compels functionality from this code 00:35:42 reaverb: on the plus side, the enum names make good filenames 00:35:46 unique! 00:36:13 there's probably a downside about needing to cat enums/*.h | vim -R - to read them all in one place 00:36:27 or some sort of merge-split script like was being discussed for monsters 00:37:07 -!- CSDCScripter has quit [Read error: Connection reset by peer] 00:37:49 Well there are also readability issues with having them all stuck together. For example, having to search for the enum you want to edit. 00:37:58 * floatingatoll nods 00:39:40 <|amethyst> floatingatoll: re chaos_type, I think it's worse to put it in attack.h 00:39:45 * floatingatoll ? 00:39:50 <|amethyst> floatingatoll: since it's only ever used inside attack.cc 00:40:12 is the TODO outdated? 00:40:32 floatingatoll: https://pastebin.mozilla.org/5223961 is the (void)/()/ patch, it would be nice if you checked it for warnings. 00:40:39 <|amethyst> I think "somewhere sane" would be something like chaos.cc instead of attack::chaos_affects_defender() :) 00:40:46 -!- schistosomatic has quit [Quit: Leaving] 00:40:49 heh 00:41:39 More (void) s than I thought. 00:41:39 there were some grumpy TODOs about calling out to melee_attack from non-attack code for the chaos stuff 00:41:45 it breeds 00:41:53 it breeds chaos as you dig into it further, amusingly 00:42:14 * floatingatoll builds 00:42:27 floatingatoll: I have no idea why people even put TODOs or XXXs in Crawl code, we almost never actually try to fix them :D 00:42:31 heh 00:42:44 i mean, i *agree* with the whole "chaos effects shouldn't mandate an attack of some sort" 00:43:02 it's just beyond my simple ken 00:43:24 i couldn't make any sense of areas.cc: // TODO: update sanctuary here. 00:43:38 -!- Furril has quit [Ping timeout: 276 seconds] 00:43:48 sanctuary code is awful so it make sense it has a todo. 00:43:48 -!- tabstorm has quit [Ping timeout: 240 seconds] 00:43:58 or at least the bits in mon-abil.cc are. 00:44:31 -!- Bcadren has quit [Ping timeout: 252 seconds] 00:45:35 flooatingatoll: That function updates things like Disjunction, it would make sense for it to handle Sanctuary too. 00:45:37 reaverb: nothing new at -Wall, it'll take longer to run -Weverything twice 00:45:38 <|amethyst> 20 TODOs added so far in 0.14, 15 removed 00:45:52 <|amethyst> err, in 0.15 00:45:52 floatingatoll: Hmm 00:45:57 <|amethyst> 33 added in 0.14, 36 removed 00:46:05 |amethyst: do you have a suggestion for when to use @param[in] versus just @param? 00:46:50 reaverb: and, like, an hour to do two static analysis runs 00:46:56 so i'll at least try a -Weverything pair :) 00:47:02 -!- CSDCScripter has quit [Read error: Connection reset by peer] 00:47:05 floatingatool: Thanks you. 00:47:11 -!- Bcadren has joined ##crawl-dev 00:47:22 <|amethyst> gammafunk: I've been using [in] for non-const pointers/references when there is at least one out-parameter to the function 00:47:37 Not sure a static analysis run is need for this search and replace, is there any particular issue you know of which could be introduced by this? 00:47:38 <|amethyst> gammafunk: I think it's only really necessary for contrast 00:47:43 yeah, makes sense 00:47:44 none at all 00:47:58 i just have three tiers of helpful, and clarifying that i'm only probably going to make it through the first two 00:48:23 * floatingatoll browses random TODOs 00:48:42 floatingatool: Thanks. 00:48:56 Also we use XXX and FIXME inconsistantly. 00:49:05 Hmm, that could be another good search and replace. 00:49:30 dat/dlua/v_layouts.lua:-- TODO: Spirally maze 00:49:32 heh. 00:50:04 Summon Horrible Things is cast frequently in the crawl codebase 00:50:17 the int drain explains the current state of the source 00:50:25 Summon Horrible Incongruous Things, perhaps. 00:50:31 * floatingatoll tries to make a joke 00:50:57 !lg . min=int x=int 00:50:58 1410. [int=-9] gammafunk the Convoker (L20 HEIE of Sif Muna), forgot to breathe caused by summoning horrible things on Abyss:1 on 2013-08-11 03:08:30, with 307483 points after 45834 turns and 12:39:50. 00:51:02 wait 00:51:41 can 'caused by' say 'while' or 'due to' there? 00:51:57 caused by 00:52:06 since it gave the int drain that lead to stat death 00:52:10 -!- Keanan has quit [Quit: Leaving.] 00:52:13 // if someone deletes the db, no message is ok 00:52:19 -!- iFurril has quit [Ping timeout: 240 seconds] 00:52:22 it was an intentional death of course 00:52:26 oh. 00:52:39 gammafunk: Why? Lowest Int? 00:52:45 no, turncount too high 00:52:47 i like that it's resilient to "who needs a string database, anyways" issues 00:52:51 abyss took way too long 00:53:06 Ah, might be good to fix that sometime. 00:53:27 I like the idea of Banishers sending you to different levels of the Abyss based on how powerful they are. 00:53:42 well if you get banished in the abyss, you're sent deeper 00:53:51 <|amethyst> "caused by" appears only for stat death 00:53:59 BEH_FRIENDLY? 00:54:14 BEH_FRIENDLY, // used during creation only 00:54:17 what about BEH_FRIENDLY 00:54:18 <|amethyst> so it's only relevant for viewing old death messages 00:54:31 hrm 00:54:35 a lot of these lack a description of sorts 00:54:42 'creation only' 00:54:42 yeah I guess that's true, now that I think about the AI code 00:55:01 I wonder why that behavior state is necessary 00:55:12 <|amethyst> it's not used as a state 00:55:38 it seems to create it as friendly and then 'player_angers_monsters' when successful 00:55:40 yeah, I recall that in the monster ai code, that doesn't show up 00:55:58 but also MG_AUTOFOE, so .. okay 00:56:09 this is abominable for sure 00:56:15 Would it be worth it to standardize Crawl to using XXX: to mark bad code? 00:56:43 <|amethyst> reaverb: I don't think so, no 00:56:45 libutil.cc: // TODO: if someone defines a "goblin chief", this should be revisited. 00:56:52 <|amethyst> that's my comment :) 00:56:57 <|amethyst> chieves 00:57:01 so libutil depends critically on whether or not goblin chiefs exist 00:57:07 |amethyst: Hmm, to trivial? 00:57:16 -!- CSDCScripter has quit [Read error: Connection reset by peer] 00:57:19 oh! pluralization! 00:57:55 well clearly we've needed a goblin species for a long time now 00:58:17 <|amethyst> reaverb: some people use FIXME and XXX to mean different things 00:58:41 <|amethyst> granted, that's not standardised either 00:58:45 'serves' 00:58:56 |amethyst: hmm. 00:59:45 can't have more than one Yiuf, right 01:00:52 <|amethyst> uniques are invariant under pluralisation 01:01:02 <|amethyst> hm 01:01:03 <|amethyst> or are they 01:01:20 The Royal Jellies? 01:01:35 |amethyst: They are. 01:02:12 Although I think they either shouldn't be, so our bugs can be grammatically correct, or they should trigger and ASSERT(), so that we can catch our bugs faster. 01:02:37 <|amethyst> people complained about "Maras are frozen" 01:02:40 -!- MgDark has quit [Quit: ChatZilla 0.9.90.1 [Firefox 27.0/20131216183647]] 01:03:00 <|amethyst> ohh 01:03:02 <|amethyst> actually 01:03:05 |amethyst: Yes, Mara has a special except the the no plural uniques. 01:03:06 <|amethyst> we do pluralise them 01:03:14 <|amethyst> You feel very cold. Crazy Yiuves are frozen. 01:03:17 yep 01:03:25 <|amethyst> just not in the monster list 01:03:38 'Xom Crazy Yiuf' worries me slightly 01:03:41 Wow that makes not doing it in the monster list even worse. 01:03:49 like, is that a Yiuf with Xom enhancements? 01:03:54 because Yiuf is already kinda wack 01:03:57 er, whack 01:03:57 <|amethyst> I don't think there should be an assert here 01:04:00 (stave, right) 01:04:12 <|amethyst> floatingatoll: you mean in the speech? It's Yiuf when you worship Xom 01:04:26 |amethyst: Hmm, if it should never happen, why not? Because it's something we might want to happen later? 01:04:44 <|amethyst> reaverb: because I don't want wizmode to crash just because I made two of the same unique 01:04:51 well anyways, Crazy Yiuf and Wiglaf are the two that might go awry in monsters.txt afaict 01:04:53 |amethyst: Ah. 01:05:11 since all the ones that SHOULD pluralize are lowercase, you could hilariously just check /^a-z/ 01:05:15 <|amethyst> also, Mauricescumming 01:05:30 wow, a merge of master with no nasty conflicts to fix 01:05:36 I must have done a good deed 01:05:57 reaverb: i couldn't see any at all 01:05:59 -!- UncertainKitten has quit [Quit: Connection reset by pier.] 01:06:04 reaverb: no changes in -Weverything 01:06:10 floatingatoll: Good. 01:06:12 -!- lobf has quit [Quit: lobf] 01:06:23 <|amethyst> let me see if I can find that description of -f pluralisation 01:07:32 // TODO: change to an ASSERT when the underlying crash goes away 01:07:37 really. 01:07:54 <|amethyst> oh, there isn't a rule 01:07:59 <|amethyst> it's just a list of words 01:08:32 -!- CSDCScripter has quit [Read error: Connection reset by peer] 01:09:07 <|amethyst> floatingatoll: that's a relatively recent one 01:09:18 -!- simmarine_ has joined ##crawl-dev 01:09:29 <|amethyst> %git a0261e1 01:09:30 07Grunt02 * 0.15-a0-990-ga0261e1: Paper over a crash (#8587). 10(11 hours ago, 1 file, 8+ 1-) 13http://s-z.org/neil/git/?p=crawl.git;a=commitdiff;h=a0261e132ed5 01:09:42 floatingatoll: Is that Grunt's recent change? We're working on it, but we might as well let people play while we track this down. 01:09:49 interesting 01:09:50 <|amethyst> the underlying bug was fixed in 01:09:54 <|amethyst> %git df56bc9 01:09:55 07Grunt02 * 0.15-a0-993-gdf56bc9: Really fix Gozag gold detection (#8584). 10(10 hours ago, 3 files, 23+ 16-) 13http://s-z.org/neil/git/?p=crawl.git;a=commitdiff;h=df56bc9af60c 01:10:01 oh, sweet 01:10:09 <|amethyst> but there may be people who saved with a buggy version 01:10:17 <|amethyst> s/may be/are/ 01:10:22 |amethyst: Minor tags? 01:11:15 At the very least #ifdef MAJOR_TAG_VERSION == 34 01:12:00 03reaverb02 07* 0.15-a0-1000-g2d06bff: Chance function(void) to function() 10(36 minutes ago, 75 files, 196+ 196-) 13http://s-z.org/neil/git/?p=crawl.git;a=commitdiff;h=2d06bffd5ab5 01:12:02 <|amethyst> yeah, I don't know if it's worth a minor tag, but putting it in TAG_MAJOR makes sense 01:12:11 <|amethyst> reaverb: you made a typo :) 01:12:26 Gah, drat. 01:12:40 I'm sure people won't notice <_< 01:12:45 <|amethyst> I always spot mine five minutes after pushing 01:12:53 -!- simmarine has quit [Ping timeout: 276 seconds] 01:13:00 <|amethyst> a few of my commit messages just cut off mid-sentence 01:13:08 heh 01:13:15 Luckily I haven't had that happen yet. 01:13:38 It's only a matter of time though. I sometimes save commit messages in half finished state and then amend them later. 01:14:02 -!- hurdos has quit [Ping timeout: 245 seconds] 01:15:45 ah, time to see if I can squash merges 01:15:52 since reaver just ruined my merge 01:16:12 gammafunk: weightless? 01:16:16 yeah 01:16:20 gammafunk: I'll merge it for you if you want. 01:16:32 no not weightless into master 01:16:36 master into weightless 01:16:41 i like that the source uses 'gray' and the dats use 'grey' 01:16:42 I had just done a merge 01:16:46 and you commited that 01:16:50 I could ignore it but 01:16:55 rebase ftw! 01:16:57 I want to try it anyhow 01:16:59 floatingatoll: can't! 01:17:05 yeah. 01:17:10 or just cherry-pick his commit into your branch 01:17:12 gammafunk: Good luck! 01:17:20 git cherry-pick -m or something 01:17:22 * floatingatoll shrugs 01:17:33 well my question is, can I just merge again, squash the merge commits? 01:17:35 that works, right? 01:17:43 gammafunk: I don't think so. 01:17:50 Not sure though. 01:17:54 well. er. i can't quite tell. 01:18:01 like, you already have a squashed merge, right 01:18:15 and you want to integrate his into it without altering your existing commit? 01:18:26 gammafunk: If you have git rerere enabled than you can undo the merge, redo it, and only change the parts which you didn't handle last time :D 01:18:36 heh 01:18:40 yeah, well there were no conflicts anyhow 01:18:51 from the first merge I mean 01:18:55 let me see if it works 01:19:11 the squash, I mean 01:19:19 <|amethyst> floatingatoll: the source uses "gray"? other than as an alias for "grey"? 01:19:19 in that situation, since you're *altering* the commit, i'd use git rebase to inject that patch prior to your merge 01:19:27 -!- home has quit [Ping timeout: 245 seconds] 01:19:29 colour.cc: if (colour == "darkgray") 01:19:33 reaverb: you made conflicts dang it! 01:19:40 <|amethyst> floatingatoll: read the next line 01:19:42 a207 gray, 291 grey 01:19:43 <|amethyst> if (colour == "darkgray") 01:19:43 <|amethyst> colour = "darkgrey"; 01:19:45 haaha 01:19:54 i'm just admiring 01:19:55 gammafunk: Sorry, if you had merge weightless earlier today I should have changed the merge commit too :D 01:20:10 <|amethyst> though you did lead me to find one 01:20:21 it's actually very carefully managed, 'gray' never leaks to user-visible anything 01:20:40 -!- CSDCScripter has quit [Read error: Connection reset by peer] 01:20:40 it is always 'grey' from a user perspective 01:20:48 that's rare 01:20:52 consistency++ 01:21:07 03|amethyst02 07* 0.15-a0-1001-gdb52185: Fix a comment. 10(27 seconds ago, 1 file, 1+ 1-) 13http://s-z.org/neil/git/?p=crawl.git;a=commitdiff;h=db5218568966 01:21:15 <|amethyst> floatingatoll: it becomes harder with all these Americans on the devteam now :) 01:21:25 heh :) 01:21:35 you clearly need en-US and en-UK with gray/grey dialects 01:21:41 We should just invent our own convention. 01:21:42 griy 01:21:55 grye! 01:22:03 it even sounds like the two spellings melded. 01:22:19 <|amethyst> %git d17ad6a 01:22:19 07|amethyst02 * 0.11-a0-1866-gd17ad6a: Remove spurious -iy => -ies pluralisation. 10(2 years ago, 1 file, 0+ 3-) 13http://s-z.org/neil/git/?p=crawl.git;a=commitdiff;h=d17ad6a3ddbf 01:22:56 My spelling has slowly become my European thanks to crawl 01:23:03 I'm not quite at "colour" yet though 01:23:14 however! ziggurat misspells 'centered'. 01:23:27 s/my/more/ 01:23:44 I don't think I'll ever get to "colour" 01:23:47 no, wait 01:23:47 local centered = string.find(dgn.tags(map), " centered ") 01:23:50 i.. what 01:23:58 so it scans the dungeon tags for the english spelling 01:24:03 HILARITY ENSUES 01:24:09 * floatingatoll goes back to spellchecking dat/* 01:26:15 -!- category has quit [Quit: Page closed] 01:27:20 <|amethyst> hm 01:27:37 <|amethyst> "centered" there can't be blamed on the Americans 01:27:41 <|amethyst> %Git 376669c 01:27:43 <|amethyst> %git 376669c 01:27:44 07greensnark02 * 0.5-a0-998-g376669c: Add pillars to ziggurat, more utility functions for Lua to handle maps. 10(5 years ago, 9 files, 301+ 51-) 13http://s-z.org/neil/git/?p=crawl.git;a=commitdiff;h=376669c4079c 01:27:49 heh! 01:29:28 will D:{3..10} work as a search string, given that D:[3-10] doesn't, in stash-tracker? 01:29:29 <|amethyst> neither Common, nor Wealth, nor Nations 01:29:44 no, that's wrong 01:29:45 wait. What's the other centered spelling 01:29:50 <|amethyst> reaverb: centred 01:30:04 hmm 01:30:21 -!- CSDCScripter has quit [Read error: Connection reset by peer] 01:30:31 -!- Keanan has joined ##crawl-dev 01:31:36 what's "gridelin"? 01:31:56 wait, that's a *coloUr*! 01:32:21 -!- Kramin has quit [Quit: This computer has gone to sleep] 01:33:59 so Lord Dunsany was american. 01:34:00 ../descript/quotes.txt: -Lord Dunsany, “How Nuth Would Have Practiced His Art Upon the Gnoles”. 01:34:27 -!- Kalir has quit [Quit: CADENZA-CLASS CLOCKWORK KNIGHT--OPERATION TERMINATED] 01:35:01 -!- KurzedMetal has quit [Read error: Connection reset by peer] 01:35:21 <|amethyst> floatingatoll: good catch, that's an incorrect title 01:35:46 <|amethyst> clang --analyze-dialects 01:36:32 03|amethyst02 07* 0.15-a0-1002-gecfd64c: Correct a citation (floatingatoll) 10(20 seconds ago, 1 file, 1+ 1-) 13http://s-z.org/neil/git/?p=crawl.git;a=commitdiff;h=ecfd64cfc761 01:39:39 Well, I figured out that you can squash merges using git rebase --preserve-merges, unfortunately squashing merges also squashes the commits in the merges :( 01:39:48 revert! 01:39:49 so I just reset and will do the merge again 01:40:57 -!- CSDCScripter has quit [Read error: Connection reset by peer] 01:43:57 ??eggplant 01:43:58 eggplant[1/1]: A rotten eggplant which you can throw at the Crawl developers. (It's a bug if you ever see this message in Crawl.) 01:46:07 should i mention 'majour', 'minour' 01:46:13 like, what era of british is this, anyways 01:46:46 er 'european 01:47:22 <|amethyst> fin-de-si�cle Australian English 01:47:42 <|amethyst> but not that si�cle, the other one 01:49:42 |amethyst: AstralHam upgraded his trunk game and had forest, currently in it now; it won't crash should he get to forest:5, will it? 01:50:03 I should say, he upgraded an old game that had forest generated 01:50:23 I assume the end vault won't be there 01:50:57 !lm AstralHam 01:50:58 -!- CSDCScripter has quit [Read error: Connection reset by peer] 01:50:58 75. [2014-05-24 06:25:59] AstralHam the Executioner (L20 HODK of Yredelemnul) entered the Enchanted Forest on turn 76250. (Vaults:2) 01:51:02 !lm AstralHam -log 01:51:03 AstralHam, XL20 HODK, T:76250 (milestone) has no matching game. 01:51:14 &dump AstralHam 01:51:15 http://dobrazupa.org/morgue/AstralHam/AstralHam.txt 01:51:34 ah 01:51:44 forest.des is deleted, but the cached version is being used? 01:51:48 * floatingatoll mumbles about slime moulds 01:51:48 or the db versioni I guess? 01:51:53 so maybe everything will just work 01:51:53 -!- TAS_2012v has quit [Ping timeout: 264 seconds] 01:52:19 Hmm, does turning rot resistance (Gr and VS) into a mutation and giving it to Ds instead of foul stench sound good. 01:52:32 (If anybody's wondering I would not want this to be a random mutation) 01:53:34 gammafunk: Some vaults still have the Forest place. 01:53:37 <|amethyst> you mean instead of the saprovore part of foul stench? 01:53:51 |amethyst: Yes, sorry. 01:53:53 giving it to Ds instead of sapavore. 01:53:58 <|amethyst> gammafunk: there probably won't be an end vault 01:54:01 yeah 01:54:08 reaverb: some do, but may were outright removed 01:54:13 he got the tomb entrance 01:54:17 but that's a branch entrance 01:54:31 gammafunk: Hmm. 01:54:53 Might be worth it to remove the Forest place from those which currenly have it, since it breaks TAG_MAJOR_VERSION 35 01:55:10 and as seen here there might be 3 saves left it could affect. 01:55:20 Yeah, I'm not sure there's any reason to have Forest in Place/depth 01:56:34 ./crawl 01:56:37 <|amethyst> shouldn't be removed from layout_forest 01:56:41 Sorry, wrong window. 01:56:56 -!- CSDCScripter has quit [Read error: Connection reset by peer] 01:58:37 does this mean we can finally have a "Remove Forest" commit? 01:58:38 yeah, I guess technically the balancer vaults in dummy.des should still have it as well 01:58:51 (preferably titled "Burn down Forest") 01:59:17 <|amethyst> reaverb: foul stench already gives rrot 01:59:19 Lightli: I like your commit message but I think it will be quietly removed on the TAG_MAJOR_VERSION bump. 01:59:34 |amethyst: But not as rot immunity, which is what I'm suggesting. 01:59:34 'biassed', really?! 01:59:39 Sorry if that wasn't clear. 01:59:42 reaverb: Ok 02:00:04 <|amethyst> reaverb: the difference between 1 and 3 is irrelevant for players I believe 02:00:19 |amethyst: Oh, hmm. 02:01:29 <|amethyst> yeah, Zin rot is the only thing that cares about the difference 02:01:29 The message says something about Zin, being able to rot with 1, but yes it's probably trivial. 02:01:35 -!- Kramin has quit [Quit: This computer has gone to sleep] 02:01:50 On the other hand, sapavore is also a pretty trivial mutation. 02:02:04 and just giving rot immunity earlier might be fine. 02:02:30 either way, rot immunity should be a mutation for Gr and VS, and I will probably implement that in the next couple days. 02:02:36 !tell Grunt AstralHam Upgraded an old game that had forest, and although he can't get an end vault, he did find a Qazlal altar there :) 02:02:37 gammafunk: OK, I'll let grunt know. 02:02:49 Ha, that's cute. 02:03:06 ProzacElf is also working on a Forest save IIRC, he might have died though. 02:05:12 'lize' reveals a bunch of uk->us including a spell named 'Localized Ignite Poison', and this is where i decide to stop 02:05:23 presumably renaming a spell will destroy save game compat somehow. 02:06:17 !lm ProzacElf place=forest 02:06:17 floatingatoll: Unless it was in a prop(), it wouldn't. 02:06:18 7. [2014-05-24 06:25:59] AstralHam the Executioner (L20 HODK of Yredelemnul) entered the Enchanted Forest on turn 76250. (Vaults:2) 02:06:25 -!- PleasingFungus has quit [Quit: ChatZilla 0.9.90.1 [Firefox 32.0a1/20140520030202]] 02:06:26 yeah, seems so 02:06:34 !lg porzacelf HODK 02:06:34 No games for porzacelf (HODK). 02:06:38 !lg prozacelf HODK 02:06:39 No games for prozacelf (HODK). 02:06:43 huh, okay 02:06:45 looks like he's still alive 02:06:54 floatingatoll: You can change the name of an enum whenever, you just can't move it somewhere else. 02:07:16 !lm prozacelf HODK 02:07:17 52. [2014-05-24 06:25:59] AstralHam the Executioner (L20 HODK of Yredelemnul) entered the Enchanted Forest on turn 76250. (Vaults:2) 02:07:36 <|amethyst> it would mess up peoples' spell_slot option, but otherwise I wouldn't expect a problem 02:08:01 <|amethyst> s/s'/'s/ 02:09:14 feeding the US side of http://www.tysto.com/uk-us-spelling-list.html into git grep -w -f foo reveals +/- 471 instances 02:09:25 half or so comments, lots of *.txt stuff 02:09:25 -!- Yermak has quit [Quit: Page closed] 02:09:49 -!- CSDCScripter has quit [Read error: Connection reset by peer] 02:10:25 88 in dat/**/*.txt 02:10:31 done for now, be well all 02:14:29 03gammafunk02 07[weightless] * 0.15-a0-999-g25b6415: Make corpses and skeletons stationary items 10(2 hours ago, 9 files, 112+ 47-) 13http://s-z.org/neil/git/?p=crawl.git;a=commitdiff;h=25b641589ec2 02:14:29 03gammafunk02 07[weightless] * 0.15-a0-1020-gad9ecc3: Merge branch master into weightless 10(5 minutes ago, files, + -) 13http://s-z.org/neil/git/?p=crawl.git;a=commitdiff;h=ad9ecc39a6aa 02:14:30 -!- ussdefiant has quit [Ping timeout: 258 seconds] 02:14:30 <|amethyst> floatingatoll: night 02:14:30 -!- Arkaniad has quit [Ping timeout: 240 seconds] 02:15:26 -!- wat has quit [Ping timeout: 265 seconds] 02:15:31 -!- Yermak has quit [Client Quit] 02:16:32 -!- Kramin has quit [Ping timeout: 245 seconds] 02:17:24 Unstable branch on crawl.beRotato.org updated to: 0.15-a0-1002-gecfd64c (34) 02:17:33 -!- iFurril has quit [Ping timeout: 252 seconds] 02:17:46 03|amethyst02 07* 0.15-a0-1003-g6415739: Avoid "you." in player:: methods. 10(67 seconds ago, 1 file, 6+ 6-) 13http://s-z.org/neil/git/?p=crawl.git;a=commitdiff;h=64157391c3e0 02:21:39 -!- CSDCScripter has quit [Read error: Connection reset by peer] 02:22:25 Experimental (weightless) branch on crawl.s-z.org updated to: 0.15-a0-1020-gad9ecc3 02:24:48 Experimental (weightless) branch on crawl.beRotato.org updated to: 0.15-a0-1020-gad9ecc3 02:25:17 -!- Keanan has quit [Quit: Leaving.] 02:26:18 -!- conted has quit [Ping timeout: 240 seconds] 02:26:47 -!- Kramin has quit [Client Quit] 02:26:52 Hmm, would a demonspawn Facet which makes them "divine beings" favored by gods be good? 02:27:24 For example: Tier 1: 1/2 effect of amulet of Faith Tier 2: effect of Faith Tier 3: 3/2 effect of Faith 02:27:32 Numbers would probably need tweaking. 02:28:38 -!- Bcadren has quit [Ping timeout: 252 seconds] 02:30:35 -!- rast has quit [Read error: Connection reset by peer] 02:30:54 -!- rast has joined ##crawl-dev 02:31:22 -!- CSDCScripter has quit [Read error: Connection reset by peer] 02:31:55 -!- rchandra has left ##crawl-dev 02:35:44 -!- ProzacElf has quit [Ping timeout: 265 seconds] 02:36:55 -!- ghallberg has joined ##crawl-dev 02:37:15 Hullo, I've been reading the mailing list a bit about the food changes. 02:37:56 Hello! 02:38:07 Hmm, my idea that I had was really bad :p 02:38:13 I just realized it didn't help out at all :D 02:38:20 Ok, on to my second idea. 02:38:28 ghallberg: Well, better to realize it before saying it. 02:38:36 reaverb: faith is uh super crazy awesome good, giving 3/2 faith sounds insane 02:38:40 If too much inventory space is an issue, why not just halve inventory space? 02:38:46 <|amethyst> 5/2 faith sounds even more insane 02:38:49 eb_: "Numbers would probably need tweaking." 02:38:59 so it's [a-z] instead of [a-zA-Z] 02:39:02 |amethyst: Yes, sorry, those numbers would be with stacking. 02:39:27 <|amethyst> with stacking? 02:39:32 ghallberg: I don't think anybody has said there's too much inventory space? 02:40:03 |amethyst: Sorry, rather than focusing on the numbers, is the idea sound? 02:40:28 |amethyst: Ah, I get it, if you also wore an amulet of faith with the mutation. 02:40:33 s 02:41:02 <|amethyst> I don't know, passive faith without the drawback seems a little powerful 02:41:13 fr: piety regen 02:41:28 <|amethyst> on the other hand, by the time you get to the third level of the mutation, it wouldn't matter for many gods 02:41:36 reaverb: People are saying there's no reason to leave strategic resources behind. 02:41:40 |amethyst: There has to be a place where it isn't too powerful but isn't too weak. Also, it could be something like reduced piety costs. 02:41:48 ghallberg: That's a good thing. 02:41:54 -!- CSDCScripter has quit [Remote host closed the connection] 02:42:09 Since you don't have to cntrl-F and run back wherever when you want to use them. 02:42:21 reaverb: Well, some people thought it was an interesting mini game, deciding what stuff you want to have on you at all times. 02:42:37 But yeah, the ^F stuff isn't as fun :) 02:42:43 ghallberg: For strategic resources that is by definition not the case. 02:42:48 -!- Limulus has quit [Ping timeout: 240 seconds] 02:43:19 I was really meaning tactical, silly me... 02:43:46 ghallberg: Oh, well it doesn't actually matter if you have more than 3 or so of a tactical item on you. 02:44:13 When are you going to use 4 ?blink in one fight without teleporting? 02:44:26 That's not what I was getting at. 02:44:47 ghallberg: Ok, keep going. 02:45:25 If you redue the number of inv-slots you'd have to choose to leave one or more of ?tele, ?blink, !rage, !speed, ?fear 02:45:48 if you only have 26 slots instead of 52 02:46:15 <|amethyst> I'd rather just keep item weights 02:46:30 <|amethyst> if you want people to have to shuffle inventory 02:46:32 ghallberg: Yes 26 slots sounds awful to juggle. 02:46:51 ghallberg: 26 is a lot more slots than you gain by not having to carry food. 02:47:18 Yeah, it might be too much, 26 just seemed easy to do. 02:47:40 I also was thinking about some kind of storage for strategic items. 02:47:54 ghallberg: Yes the storage idea has been brought up. 02:48:13 Oh well 02:48:52 Any idea on the interface? 02:49:02 Let it pop up every 5 levels (only in D maybe?) 02:49:18 It's like a shop, but you can drop stuff on it. 02:49:41 ghallberg: and how do you get the stuff back out? 02:49:41 Using it takes a lot of game-time, so you can't use it in combat. 02:49:53 reaverb: Like a shop, but everything is free 02:50:02 -!- eb_ has quit [] 02:50:14 ghallberg: Oh, that was not the storage idea i was thinking of. 02:50:47 The storage idea I was thinking of was the items not being in your inventory, but instead being like runes, gold, or the orb. 02:50:48 It could be on every level or something too, I'm not sure what the flavor is yet :P 02:50:53 Ah ok 02:51:05 I liked the idea of goldifying food. 02:51:23 ghallberg: If that was implemented I would just let the player access their storage shed from anyway. (Time being the limiter) 02:51:30 <|amethyst> goldifying food seems like kind of a half measure to me 02:51:32 s/anyway/anywhere/ 02:52:28 <|amethyst> at the very least, having an 'eat' command 02:52:41 |amethyst: "Half measure" seems like a good way to describe it. However, I think there are intrinsic benefits (like not having to deal with slots so much) 02:53:08 Yeah I dunno :) 02:53:13 I'll be back later 02:53:13 <|amethyst> why not just make it that every piece of food you pick up adds to your nutrition? 02:53:18 Haven't been around here much latelky 02:53:20 <|amethyst> with no cap on the top end 02:53:35 |amethyst: Because then hunger costs lose their tactical effect. 02:53:43 Of not being completely spamable. 02:53:46 -!- CSDCScripter has quit [Remote host closed the connection] 02:53:58 <|amethyst> reaverb: because you can go from engorged to starving in a single fight? 02:55:20 |amethyst: Well, from not-full to starving. 02:55:42 <|amethyst> oh, would you not be able to eat to Engorged anymore? 02:56:31 -!- magicpoints has quit [Ping timeout: 265 seconds] 02:56:38 |amethyst: I'm not sure what you getting at, the reason I proposed it slightly as a "half measure" was to make the minimum possible change. 02:57:19