[00:28:25] brion: do you have time to look at https://bugzilla.wikimedia.org/show_bug.cgi?id=34539 ? [00:28:40] AaronSchulz started looking at it, but some swift stuff took priority [00:29:04] *AaronSchulz isn't looking at swift atm [00:29:35] hm [00:29:52] brion: maybe Aaron's got it [00:30:06] 'global oversight' must be new in the last couple years, don't remember that feature. ic an take a peek through if you want though :D [00:35:56] robla: I've been meaning to ask about the 1.19 schedule, but I guess it can wait till TL;DR in the morning [01:26:52] Nikerabbit: still there? [01:28:19] Reedy: You there? [02:05:33] Krinkle: who would you recommend for mysql tests? I'm thinking chad... [02:05:47] Ask him :P [02:05:49] ^demon: [02:05:51] I have no clue [02:05:55] What kind of tests? [02:05:59] huh [02:06:04] I didn't see him [02:06:07] *hexmode is blind [02:06:11] <^demon> Why do I feel like I'm getting volunteered for something? [02:06:19] hexmode: don't worry, he joined while you were typing [02:06:24] https://bugzilla.wikimedia.org/34762 [02:06:27] like 1 sec before [02:06:39] ^demon: could you look at that tiny patch [02:07:01] looks like it would be a good one for 1.19 tarball [02:07:29] Krinkle: yeah I got annoyed with join/part a long time ago and hid them [02:07:37] <^demon> I'm not entirely sure why you'd call close() twice anyway ;-) [02:07:37] aha [02:08:11] <^demon> $dbw->close(); // close the connection [02:08:16] ^demon: neither am I... so if you do should you get the warning? [02:08:16] <^demon> $dbw->close(); // making sure [02:08:25] <^demon> Well yeah [02:08:33] <^demon> I'm more curious why close() is getting called twice. [02:08:45] <^demon> Probably just a hypothetical, or some awful extension. [02:09:16] yeah, I'll ask on the bug [02:09:28] unless you've already typed something up? [02:10:38] <^demon> I'm about to commit. [02:10:43] ^demon: Well, it shouldn't give a low-level PHP warning for msyql_close [02:10:51] <^demon> I don't think it's worth backporting, unless he can point to something silly that's already doing this. [02:10:58] no matter what, and the documentation says it should be cleared [02:11:02] <^demon> If it's just a hypothetical, it can wait. [02:11:28] k, so the question was worth it! [02:11:33] it's a minor case, shouldn't be in production code. (or actually hardly can be since it emits a PHP warning) [02:11:34] <^demon> whoops, no git commit. [02:13:37] <^demon> Actually, all the database subclasses suffer from this. [02:15:05] ^demon: https://wikitech.wikimedia.org/view/Formey , can you confirm that 'will be' is 'is' now ? [02:15:14] "which will be the new server for our public subversion repository, replacing mayflower" [02:15:54] <^demon> Oh yeah, mayflower was decommissioned like a year ago [02:16:02] ok, thx [02:16:19] <^demon> And actually we'll be moving gerrit off formey to some new box. [02:17:39] ok [02:19:00] ^demon: I'm trying to confirm my email for my wikitech account but can't. Something is probably misconfigured in the smtp ? [02:19:27] <^demon> No clue. I don't have access to that. [02:20:16] k [02:20:24] who does? [02:27:51] <^demon> someone from ops. [02:28:03] <^demon> wikitech's hosted offsite, so probably a root user. [02:28:05] <^demon> Roan might know [02:28:21] Someone who's root on wikitech [02:28:23] i.e. not me [02:28:25] Krinkle: try asking on #wikimedia-operations [02:28:33] I'm probably root there [02:28:54] ...or just wait for Tim here :) [02:30:04] 2012-02-28 02:19:08 1S2Ceh-0003AL-Uw ** ttijfhof@wikimedia.org R=dnslookup T=remote_smtp: SMTP error from remote mail server after RCPT TO:: host mchenry.wikimedia.org [208.80.152.186]: 550 Address ttijfhof@wikimedia.org does not exist [02:30:47] TimStarling: tijhof not tijfhof [02:31:00] tell Krinkle not me [02:31:09] lol [02:31:20] you'd think he'd be able to spell his own name [02:31:25] You'd think so yeah [02:35:37] TimStarling: are you the same guy that thinks people should be able to count to 12? [02:35:49] TimStarling: Thanks! [02:36:50] TimStarling: mail fixed, account confirmed [02:36:58] some day we're going to have bounce processing in MediaWiki, right? [02:37:12] :) [02:38:23] RoanKattouw: thanks for the bug fix [02:38:39] maybe the lack of dollar signs fooled some part of my brain into thinking I was writing C++ [02:38:55] heh [02:39:01] I figured that's where that came from [02:39:10] The way global variables work in JS is really kind of evil [02:40:07] in C++ you have a hugely complex name resolution process, [02:40:35] if it was an interpreted language it'd probably be as slow as JS [02:40:57] it's just the fact that it's done at compile time that saves it [02:41:21] Name resolution is probably equally bad in both languages [02:41:36] JS doesn't have as much OOP-related complexity, but it has closure scope-related complexity instead [02:42:18] closure scope and OOP are kind of mixed [02:42:27] actually I bet that was where that bug came from [02:42:29] In practice they often are yes [02:42:44] I was assuming isReady was a private variable from the enclosing scope [02:42:47] If that class were written with a different pattern, what you did would have been right [02:42:49] Exactly [02:43:51] aren't you glad PHP has a "class" keyword and doesn't require you to construct OOM from primitives? [02:43:58] OOP [02:44:08] finger memory [02:44:42] Yeah [02:44:47] I have mixed feelings about JavaScript [02:44:55] It's both the most awesome and most terrible language ever [02:45:08] closures are super awesome [02:45:17] which is why they've been imported into PHP I guess [02:45:38] (BTW the reason a global variable reference is slightly slower is that it's really doing window.jQuery, so there's a hashtable key lookup involved) [02:46:03] Or window['jQuery'] is closer to how it's actually implemented [02:46:28] Krinkle tells me that a global variable reference has to check the local symbol table of every enclosing lexical scope individually [02:46:45] so it can be 4 or 5 hashtable lookups if you have a lot of enclosing scopes [02:47:04] Oh, yeah, there's that too [02:47:09] You have to make sure it's not in some other scope [02:48:02] yeah, and to make it more complicated: there is no global scope in JavaScript [02:48:28] browsers create a variable var window = new DOMWindow(); in their initial scope [02:48:47] TimStarling: where does wmerrors log it's stuff? [02:48:48] and every