However, I'd like to know why you put those includes in the *header* file? Isn't it sufficient to leave them in the file CompareArgs.cpp?
It's generally good-practice to avoid pulling in extra headers inside a project header file if you can put them in the implementation file instead. (This reduces dependencies and total compile time.)
When is gcc4.3 likely to be out?
~Chris.
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
PerceptualDiff-1.0.1-gcc43
PerceptualDiff-1.0.1-gcc43-v2
Logged In: YES
user_id=1683413
Originator: YES
File Added: PerceptualDiff-1.0.1-gcc43.patch
Logged In: YES
user_id=1086736
Originator: NO
Hi kwizart, thanks a lot for the patch!
However, I'd like to know why you put those includes in the *header* file? Isn't it sufficient to leave them in the file CompareArgs.cpp?
It's generally good-practice to avoid pulling in extra headers inside a project header file if you can put them in the implementation file instead. (This reduces dependencies and total compile time.)
When is gcc4.3 likely to be out?
~Chris.