Bugs item #3163428, was opened at 2011-01-21 16:52
Message generated for change (Comment added) made by mhoenicka
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=512945&aid=3163428&group_id=65979

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: None
Group: None
>Status: Closed
>Resolution: Fixed
Priority: 5
Private: No
Submitted By: Gergely Nagy (algernon)
>Assigned to: Markus Hoenicka (mhoenicka)
Summary: [PATCH] Allow setting the mysql port as string or int

Initial Comment:
The MySQL driver only accepts a port set via dbi_conn_set_option_numeric, while 
other drivers (namely Postgres) accept both string or numeric ports. This 
inconsistency can lead to problems, when one tries to set the SQL server's port 
from a program built with libdbi: one driver expects a numeric port, another a 
string, a third accepts either. Setting either overrides the other, thus, the 
program would need to decide based on the database driver, which is 
unfortunate. If we only set one, for example the string option, 
dbi_conn_get_option_numeric will return 0, and thus breaking the connection 
attempt.

It would be better, if all the drivers worked like the Postgres driver, and 
used whichever is set. A patch by Viktor Tusa for the MySQL driver is attached.

----------------------------------------------------------------------

>Comment By: Markus Hoenicka (mhoenicka)
Date: 2011-02-19 14:14

Message:
Unfortunately I wasn't able to access your patch as Sourceforge still seems
to have some issues. In any case I've implemented the changes you
requested. Feel free to test the latest CVS revision.

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=512945&aid=3163428&group_id=65979

------------------------------------------------------------------------------
The ultimate all-in-one performance toolkit: Intel(R) Parallel Studio XE:
Pinpoint memory and threading errors before they happen.
Find and fix more than 250 security defects in the development cycle.
Locate bottlenecks in serial and parallel code that limit performance.
http://p.sf.net/sfu/intel-dev2devfeb
_______________________________________________
Libdbi-drivers-devel mailing list
Libdbi-drivers-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/libdbi-drivers-devel

Reply via email to