U++ forum: Welcome to the forum

Re: Various fixes to uppsrc [message #31100 is a reply to message #31099]

Fri, 04 February 2011 15:53

chickenk
Messages: 169
Registered: May 2007
Location: Grenoble, France

Experienced Member

mirek wrote on Fri, 04 February 2011 15:51

chickenk wrote on Fri, 04 February 2011 09:43

mirek wrote on Fri, 04 February 2011 15:07

chickenk wrote on Thu, 03 February 2011 07:27

[*] CtrlLib/Splitter.cpp : move some initialisations from the ctor body to its initialization list, and set chstyle to NULL initially

Is this supposed to fix anything? (except maybe chstyle).

Indeed, does not fix anything apart from chstyle.

Quote:

[*] plugin/png/... : make the path to the headers uniform, also avoids confusion with the system png headers

Wrong. If you would want to put the longer path there, you should have '<' instead of '"'.

So in that case I believe we should have <plugin/png/png.h>. Do you agree?

Well, maybe, however #include ".." is AFAIK meant exactly to tell compiler to take 'local' header file (in the same dir) and it is used in this context U++-wide...

Mirek

So leave it as-is, I agree with you. It must be an error from my build system at that time (it was some time ago). I will try to reproduce it and correct my build system instead.

Report message to a moderator