On 8 March 2013 11:29, Wacha Gábor <[email protected]> wrote: > > I am using git master. The strange thing is that this code snippet I sent > you does not seem to reproduce the problem, I will try to dive into it and > write a test case. The failing code is a bit too complex to upload, I will > try to simplify it.
Sounds good. If you'd like to suggest new test cases, feel free to submit pull request (from topic branch you create, not from master, please, ). > Thanks for your answer. This "double conversion" still seems to be a "flaw" > in the design, We're open for suggestion how to improve it. > when one has a database with a lots of fields then it can > slow down the database insert process. That is common controversy related to this issue, indeed. After SOCI 3.2.0, I'm going to actually profile it and collect some hard numbers first. Absolute time spent on this "double conversion" may turn to be irrelevant, in comparison to time spend on database communication at large. Then, I'd rather stick to current implementation as simplest correct solution with less maintenance overhead, thus less error prone. Adding more complexity in core almost always adds complexity in backends. But, as I said, we're open for suggestions and improvements are strongly welcome. It's just that we can't look from perspective of just one backend or one SQL feature (i.e. fix SELECT and break stored procedures). That is always a tough nut to crack. Long story short, we will welcome most of additions and changes in the Core as long as combination of *all* the tests for current SOCI codebase pass. Best regards, -- Mateusz Loskot, http://mateusz.loskot.net ------------------------------------------------------------------------------ Symantec Endpoint Protection 12 positioned as A LEADER in The Forrester Wave(TM): Endpoint Security, Q1 2013 and "remains a good choice" in the endpoint security space. For insight on selecting the right partner to tackle endpoint security challenges, access the full report. http://p.sf.net/sfu/symantec-dev2dev _______________________________________________ soci-users mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/soci-users
