Overview
Examples
Screenshots
Comparisons
Applications
Download
Documentation
Tutorials
Bazaar
Status & Roadmap
FAQ
Authors & License
Forums
Funding Ultimate++
Search on this site
Search in forums












SourceForge.net Logo
Home » U++ Library support » U++ Core » bug in http.cpp
bug in http.cpp [message #35911] Mon, 02 April 2012 16:32 Go to next message
borbek is currently offline  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

Re: bug in http.cpp [message #35913 is a reply to message #35911] Mon, 02 April 2012 18:44 Go to previous messageGo to next message
mirek is currently offline  mirek
Messages: 13975
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



Re: bug in http.cpp [message #35915 is a reply to message #35913] Mon, 02 April 2012 20:52 Go to previous messageGo to next message
unodgs is currently offline  unodgs
Messages: 1366
Registered: November 2005
Location: Poland
Ultimate Contributor

mirek wrote on Mon, 02 April 2012 12:44

Apologies

Too much Spartacus ? Wink
Re: bug in http.cpp [message #35931 is a reply to message #35915] Thu, 05 April 2012 08:15 Go to previous messageGo to next message
mirek is currently offline  mirek
Messages: 13975
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



Wink
Re: bug in http.cpp [message #35943 is a reply to message #35911] Sat, 07 April 2012 18:55 Go to previous messageGo to next message
dolik.rce is currently offline  dolik.rce
Messages: 1789
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
Re: bug in http.cpp [message #35944 is a reply to message #35943] Sat, 07 April 2012 20:32 Go to previous message
dolik.rce is currently offline  dolik.rce
Messages: 1789
Registered: August 2008
Location: Czech Republic
Ultimate Contributor

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

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


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...
Previous Topic: how to check specific end tag in a xml file?
Next Topic: LaunchWebBrowser() problem and perhaps not the best solution
Goto Forum:
  


Current Time: Fri Mar 29 08:44:23 CET 2024

Total time taken to generate the page: 0.01401 seconds