Quote:Original post by Antheus
Since you mention DWORDS, it's likely you're talking about C++.

*** Source Snippet Removed ***

Quote:
You can also change the types of individual A,B,C,D to be different from 8-bit int, perhaps 16-bit for D, to allow more builds.

Quote:But the best thing is, there is no overhead, there is no storage, and no #defines.

Quote:You can optionally make all methods static.

Quote:Excercises left for the reader:
- overload << operator
- overload comparison operators
etc...

Quote:Original post by Konfusius

Quote:Original post by Antheus
Since you mention DWORDS, it's likely you're talking about C++.

*** Source Snippet Removed ***

Nice solution, but...

Quote:
You can also change the types of individual A,B,C,D to be different from 8-bit int, perhaps 16-bit for D, to allow more builds.

And watch it blow up?

Quote:

Quote:But the best thing is, there is no overhead, there is no storage, and no #defines.

Maybe 4 times as much storage than the macro solution.

Quote:

Quote:You can optionally make all methods static.

To what effect?

Quote:

Quote:Excercises left for the reader:
- overload << operator
- overload comparison operators
etc...

Comparison operators. Forgot that you have to define a different type for every version?

For DWORDS, comparison operators are built in.

Quote:Original post by Bregma

Quote:Original post by Konfusius

Quote:Original post by Antheus
Since you mention DWORDS, it's likely you're talking about C++.

*** Source Snippet Removed ***

Nice solution, but...

Quote:
You can also change the types of individual A,B,C,D to be different from 8-bit int, perhaps 16-bit for D, to allow more builds.

And watch it blow up?

Why would it blow up?

// Fixed operator()// Fixed print using type_traits to avoid the static cast which would // cause errors if sizeof(ver_t) > sizeof(int)typedef unsigned char ver_t;template < ver_t A, ver_t B, ver_t C, ver_t D, typename CompactType = DWORD >struct version{  CompactType operator()() const  {    BOOST_STATIC_ASSERT(4 * sizeof(ver_t) <= sizeof(CompactType));    std::size_t bits = sizeof(ver_t) * CHAR_BITS;    return (A << 3 * bits) | (B << 2 * bits) | (C << bits) | (D);  }  ver_t Version() const { return A; }  ver_t Major() const { return B; }  ver_t Minor() const { return C; }  ver_t Build() const { return D; }private:  template<typename T>  struct print_traits  {     T convert(t)     { return t; }  };  template<>  struct print_traits<char>  {    int convert(char c)    { return static_cast<int>(c); }  };  template<>  struct print_traits<unsigned char>  {    int convert(unsigned char c)    { return static_cast<int>(c); }  };  template<>  struct print_traits<signed char>  {    int convert(signed char c)    { return static_cast<int>(c); }  };   void print() const  {    std::cout << print_traits<ver_t>::convert(A) << '.'               << print_traits<ver_t>::convert(B) << '.'               << print_traits<ver_t>::convert(C) << '.'               << print_traits<ver_t>::convert(D);  }};

Quote:Original post by Bregma
Why would it blow up?

Quote:Or maybe no storage. Oh, it's true that the values would appear in the DATA segment, just like with the C preprocessor macro, but there's no need for the additional non-portable DWORD with the template.

Quote:

Quote:

Quote:You can optionally make all methods static.

To what effect?

So you don't need an instance variable.

Quote:So you can compare apple.verision and orange.version and get meaningful results, yes?