Hello,

mysql++ 2.1.1

I have found 4 (!) bugs in 2 (!) minutes in platform.h.

1. mysql.h is included AFTER platform.h in all .cpp, therefore
MYSQL_VERSION_ID is NOT defined in platform.h at all, but:

# if ((MYSQL_VERSION_ID > 40103).......

Workaround: include <mysql_version.h> before that directive.

2. The same place:

// The shutdown_level argument was added in MySQL 4.1.3 and in 5.0.1.

OK, but what we can see in realisation of this statement:

#if ((MYSQL_VERSION_ID > 40103) && (MYSQL_VERSION_ID < 49999)) || (MYSQL_VERSION_ID > 50001)
(bigger than 4.1.3 and NOT 4.1.3) and (less than 4.99.99 and NOT 4.99.99)
OR
(bigger than 5.0.1 and NOT 5.0.1)

Workaround:
#if ((MYSQL_VERSION_ID >= 40103) && (MYSQL_VERSION_ID <= 49999)) || (MYSQL_VERSION_ID >= 50001)


Too inaccurate work

--
Best regards,
Skipy mailto:skipyrich (AT) mail (DOT) ru


--
MySQL++ Mailing List
For list archives: http://lists.mysql.com/plusplus
To unsubscribe: http://lists.mysql.com/plusplus?unsu...ie.nctu.edu.tw

Skipy Rich wrote:
Applied.

Applied.

Sigh...your efforts are appreciated, but you're not going to ingratiate
yourself with the community on effort alone. A good attitude helps, too.

If you find a bug, just explain the problem and provide a fix. As
you've seen, if it's a legitimate fix and I agree with how it's done,
it'll get applied. It's as simple as that. There's no need to insult
me and the rest of us who have worked on MySQL++ for free for your
benefit when you report the bug.

--
MySQL++ Mailing List
For list archives: http://lists.mysql.com/plusplus
To unsubscribe: http://lists.mysql.com/plusplus?unsu...ie.nctu.edu.tw