On 12/07/2011, at 8:13 PM, Benson Margulies wrote: > Brett, > > I'm not working on a problem -- well, to be exact, there was an NPE in > the code in question, but that's fixed. I happened to see this code > while working on 'aggregate' and I wondered if it was entirely > satisfactory.
Cool - it's the same pattern used in other plugins, so it's probably been replicated. The "getLanguage" field hasn't been widely used so I don't think it was put in place here (or maybe was in the past and taken out again for some limitation). I'd say it's fine. - Brett -- Brett Porter [email protected] http://brettporter.wordpress.com/ http://au.linkedin.com/in/brettporter --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
