U++ forum: Welcome to the forum

Search on this site

Search in forums

Home » U++ Library support » U++ Core » bug in http.cpp

Show: Today's Messages :: Show Polls :: Message Navigator
E-mail to friend

bug in http.cpp [message #35911]

Mon, 02 April 2012 16:32

borbek
Messages: 31
Registered: November 2010
Location: Russia

Member

";" missing in #define LLOG(x) LOG(x)

#include "Core.h"

NAMESPACE_UPP

bool HttpRequest_Trace__;

#ifdef _DEBUG
#define LLOG(x)      LOG(x)   
#else
#define LLOG(x)      if(HttpRequest_Trace__) RLOG(x); else;
#endif

Report message to a moderator

Re: bug in http.cpp [message #35913 is a reply to message #35911]

Mon, 02 April 2012 18:44

mirek
Messages: 14255
Registered: November 2005

Ultimate Member

Apologies, I guess I should not have added totally development version to Core yet..

Mirek

borbek wrote on Mon, 02 April 2012 10:32

";" missing in #define LLOG(x) LOG(x)

#include "Core.h"

NAMESPACE_UPP

bool HttpRequest_Trace__;

#ifdef _DEBUG
#define LLOG(x)      LOG(x)   
#else
#define LLOG(x)      if(HttpRequest_Trace__) RLOG(x); else;
#endif

Report message to a moderator

Re: bug in http.cpp [message #35915 is a reply to message #35913]

Mon, 02 April 2012 20:52

unodgs
Messages: 1367
Registered: November 2005
Location: Poland

Ultimate Contributor

mirek wrote on Mon, 02 April 2012 12:44

Apologies

Too much Spartacus ? Wink

Report message to a moderator

Re: bug in http.cpp [message #35931 is a reply to message #35915]

Thu, 05 April 2012 08:15

mirek
Messages: 14255
Registered: November 2005

Ultimate Member

unodgs wrote on Mon, 02 April 2012 14:52

mirek wrote on Mon, 02 April 2012 12:44

Apologies

Too much Spartacus ? Wink

Report message to a moderator

Re: bug in http.cpp [message #35943 is a reply to message #35911]

Sat, 07 April 2012 18:55

dolik.rce
Messages: 1791
Registered: August 2008
Location: Czech Republic

Ultimate Contributor

I think I found one more bug in the new socket stuff... Since revision 4752 there is a __stdcall attribute in Core/Web.h (line 33), which breaks compilation on Linux (it is MSVC only attribute, right?).

Fix is just to remove it or make it #ifdef PLATFORM_WIN32

Best regards,
Honza

Report message to a moderator

Re: bug in http.cpp [message #35944 is a reply to message #35943]

Sat, 07 April 2012 20:32

dolik.rce
Messages: 1791
Registered: August 2008
Location: Czech Republic

Ultimate Contributor

dolik.rce wrote on Sat, 07 April 2012 18:55

PS: Unfortunately, I found also bunch of similar and other bugs in Socket.cpp. The latest changes probably don't compile on any other platforms than win32 Sad

Maybe it is time to set up some automated testing? It would sure caught a lot of mistakes like this...

Report message to a moderator

Previous Topic:	how to check specific end tag in a xml file?
Next Topic:	LaunchWebBrowser() problem and perhaps not the best solution

Goto Forum:

-=] Back to Top [=-

[ Syndicate this forum (XML) ] [

] [

]

Current Time: Tue Apr 29 08:30:04 CEST 2025

Total time taken to generate the page: 0.00667 seconds