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 » Developing U++ » UppHub » [Funtions4U] Double definition of MIN
[Funtions4U] Double definition of MIN [message #43306] Fri, 27 June 2014 10:39 Go to next message
jibe is currently offline  jibe
Messages: 294
Registered: February 2007
Location: France
Experienced Member
Hi, Koldo,

In bsdiff.cpp, there is a
#define MIN(x,y) (((x)<(y)) ? (x) : (y))

but this was already defined in /usr/include/glib-2.0/glib/gmacros.h (I'm using Linux Mint Maya, and have the same warning with Scientific Linux and CentOS. Probably the same with all Linux distros).

This is not a problem : the definition is similar. It's just causing a warning.

Probably, it's bad to remove this define, as in some case or other OS gmacros could not be included or not have this define. But probably you could add a #indef before ?
Re: [Funtions4U] Double definition of MIN [message #43310 is a reply to message #43306] Fri, 27 June 2014 13:07 Go to previous messageGo to next message
koldo is currently offline  koldo
Messages: 3268
Registered: August 2008
Senior Veteran
Hell Jibe

No problem. I have just changed it. It is only used in that .cpp file. And license permits it.

Thank you.


Best regards
Iñaki
Re: [Funtions4U] Double definition of MIN [message #43326 is a reply to message #43310] Mon, 30 June 2014 19:28 Go to previous messageGo to next message
jibe is currently offline  jibe
Messages: 294
Registered: February 2007
Location: France
Experienced Member
Hi, Koldo,

koldo wrote on Fri, 27 June 2014 13:07
And license permits it.

Ooops ! Sorry : I didn't payed attention and I was thinking that it was your own work.

koldo wrote on Fri, 27 June 2014 13:07
I have just changed it.

Yes, good ! No more warning Smile

Thanks.
Re: [Funtions4U] Double definition of MIN [message #43332 is a reply to message #43326] Tue, 01 July 2014 08:05 Go to previous message
koldo is currently offline  koldo
Messages: 3268
Registered: August 2008
Senior Veteran
Quote:
Ooops ! Sorry : I didn't payed attention and I was thinking that it was your own work.
Of course yes Smile. But in this case specially because bsdiff is an external library, but with a BSD license.


Best regards
Iñaki
Previous Topic: Time for a little cleanup ?
Next Topic: Zint Barcode Generator
Goto Forum:
  


Current Time: Thu Oct 21 06:16:03 CEST 2021

Total time taken to generate the page: 0.01410 seconds