On Tue, Mar 26, 2013 at 11:04 AM, Thiago Macieira <[email protected]> wrote: > On segunda-feira, 25 de março de 2013 10.42.27, Alan Alpert wrote: >> > This is not good enough a reason. Any new feature can only be accepted in >> > *dev* once it's complete: that is, it is there, works, unit-tested, >> > documented, with examples if necessary, and an initial API review has been >> > done. If your feature is still discussing the class name, it indicates >> > that >> > the initial API review is not concluded. >> >> I got the impression that the initial API review was done. Then >> someone brought up that they didn't like the name... > > Ok, that sounds more like nitpicking. We can always rename a class before it's > released. That doesn't stop the integration, though, if everything else is > fine. > > Unfortunately, the rest of the email thread doesn't lead me to believe that it > is. > >> >> 2) What to call it? >> >> Oswald suggested "QPathSelector. or QPathSwitch[er]. or >> >> QPathMultiplexer." I'm still leaning towards QFileSelectors, but I'd >> >> also be happy with Q[File|Path]Switcher or QPathMultiplexer. I know >> >> that everyone has an opinion on naming matters, is there something >> >> even better that we haven't thought of yet? >> > >> > Explain here in a few words what the class is meant to do and how it does >> > that. That's two things I'm asking for: what and how. The name should come >> > from one or both. >> >> Applies selectors to file paths. Hence QFileSelectors (QPathSelectors >> maybe). > > What is a selector? Maybe I should just read the documentation... > > " QFileSelectors provides a convenient way of selecting file variants based > on platform or device characteristics." > > (BTW, the docs are missing the \brief) > > "Multiplexer" is too complex a term. "Switcher" gives the impression that you > can switch it -- as in switching from Android to iOS to QNX. That doesn't look > right to me. So "Selector" is better -- singular since it's what the class > does, not that it applies "selectors" to the name.
Okay, maybe I'm too close to the issue. I'll rename it QFileSelector (and add the \brief). -- Alan Alpert _______________________________________________ Development mailing list [email protected] http://lists.qt-project.org/mailman/listinfo/development
