[Development] Adding support for version number comparisons

Keith Gardner kreios4004 at gmail.com
Thu Jul 10 14:53:29 CEST 2014


On Thu, Jul 10, 2014 at 7:31 AM, Olivier Goffart <olivier at woboq.com> wrote:

> On Monday 02 June 2014 13:24:55 Richard Moore wrote:
> > On 2 June 2014 13:12, Keith Gardner <kreios4004 at gmail.com> wrote:
> > > On Mon, Jun 2, 2014 at 2:36 AM, Simon Hausmann <
> simon.hausmann at digia.com>
> > >
> > > wrote:
> > >> I suggest a name that is more centric towards the _function_ of the
> > >> class,
> > >> comparison of different software versions.
> > >
> > > QVersionInformation was also proposed as a name in the code review.  I
> > > don't have a problem with changing the name other than it is really
> long
> > > for a simple class.
> >
> > How about QVersionComparator?
>
> How about qVersionCompare(const QString&, const QString&)?
> Yes, that's a function. Is there really the need for a class to hold just a
> version string?
>
> And there could be an overload with a SuffixComparator.
>
> (or is qCompareVersion an even better name?)
>
> Currently, the QVersion::compare has an overload to pass a functor that
performs the suffix comparison.  Are you suggesting having a "default" in
the operators that can be overwritten?
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.qt-project.org/pipermail/development/attachments/20140710/1b025df4/attachment.html>


More information about the Development mailing list