On 31/03/2011 14:48, Maciej Sobczak wrote: > I will certainly have a look at it, as now there is a time for closing > the release.
OK. The majority of this patch contains whitespace and formatting modifications, which made it more difficult to figure out its essential parts - but here is what I understand: - you have unconditionally assumed that Boost is available for compilation; this is not necessarily the case and one of the important design assumptions in SOCI is that Boost is optional, not required - you have introduced the binding type parameterization in terms of templated backend type implementations - this is technically valid, but inconsistent with the existing code and now there are two partially decoupled parameterization approaches: one based on enums and switch statements and one based on templates; I would rather incorporate the new type uniformly with the existing code, which will make it easier to maintain the whole in long term Again - thank you for this patch, but I will not include it in the coming release in its current form. I will try to extract the essential parts (the timestamp-specific OCI binding routines) and incorporate them with the rest of the codebase for one of the future releases instead, with the hope that the result will be more rigid and maintainable. Best regards, -- Maciej Sobczak * www.msobczak.com * www.inspirel.com ------------------------------------------------------------------------------ Create and publish websites with WebMatrix Use the most popular FREE web apps or write code yourself; WebMatrix provides all the features you need to develop and publish your website. http://p.sf.net/sfu/ms-webmatrix-sf _______________________________________________ Soci-users mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/soci-users
