Hi Warren,
Unfortunately I ran out of time helping with this. I'll get back to it as
soon as possible, I think this project provides something very useful for
people. However, it would be much easier helping if there was a cvs
repository to check out from.
thanks
-totte
"Warren Young" <mysqlpp@stripped> wrote in message
news:412674EB.8060107@ style="color:#666">stripped...
> Totte Karlsson wrote:
>
> > There is a .zip file at
> > http://www.codeschack.com/public/
>
> Thank you for your efforts. I managed to boil all of your changes down
> to simply #including <winsock.h> conditionally where necessary and
> removing those bogus Windows32 header includes. And, of course, the
> addition of the BCB project files.
>
> I didn't agree with several of the changes you made in your version.
> Would you please download the following IN-DEVELOPMENT version of the
> library and fix up any missing bits?
>
> http://tangentsoft.net/mysql++/releases/mysql++-1.7.13-test1.tar.gz
>
> In particular, you'll notice these changes from the version you've made
> available on your site:
>
> - One of the files #included windows.h. I don't believe that to be
> absolutely necessary. At most, you should conditionally include
> winsock.h as you did in so many of the other modules. Furthermore, I
> think the module you had this fix in was the wrong place; it goes in one
> of the .hh files, not the .cc file you had it in. (connection.cc, I
> believe.) Please try this fix again, keeping these guidelines in mind.
>
> - One of the project files was named very generically: ProjectGroup1 or
> something like that. I renamed it to mysqlpp.bp? to match the others.
> You may need to fix up some things to make it build again.
>
> - Your ZIP file contains directory structure and code formatting
> changes. Please don't do this. Keep the change set to that which is
> absolutely necessary to make the program build. When I have to
> disentangle your style changes from the subtantive ones, it slows me
> down. If you want to talk about style changes, let's argue about that
> separately.
>
> > Any Borland user has to define macros (under IDE user variables) for the
> > "main" sql installation directory, e.g. MySQL = c:\mysql
> > and also change where the .lib file is being created.
>
> Can you be more explicit? I need something detailed that I can put in
> the project README file. Since I don't have BCB, I can't infer details.
>
> --
> MySQL++ Mailing List
> For list archives: http://lists.mysql.com/plusplus
> To unsubscribe:
http://lists.mysql.com/plusplus?unsub=1
>
>