Vincent van Ravesteijn - TNW wrote:
Vincent van Ravesteijn <[EMAIL PROTECTED]> writes:
I have processed your comments. I also removed als Change::UNCHANGED
d friends from the code. I had to work with objects after all,
due
to the Change::color function. If you like it, we might change it in
other parts of the code.
What s the status of the patch (wrt change_running initialization,
in particular)? I like it in general, although I am not sure we need
so many accessors for Change. OTOH, I do not know which one should be
removed :)
Jmarc
The status is that I was awaiting your comments and I'm refreshing my
C++ OO knowledge a bit. It seems I just can replace these two
initializations by "Change change_running;" and everything is OK.
Because this always uses the default constructor. Note that you can also
do things like:
Change change_running(arg1, arg2);
if you want to use a non-default constructor. Indeed,
Change change_running;
is really just shorthand for:
Change change_running();
which is why it works.
rh
