Hi Przemek,

On Thu, 25 Sep 2008, Szak�ts Viktor wrote:
I've already did it, but it's the best if you take a second look
(I can make a mistake virtually anywhere :)

You made exactly what I did with the byte precision ;-)
so after svn update I do not have any conflicts and hbdefs.h
is not marked as modified in my code. I have to say that people
working on MinGW port are really funny sumtimes. MinGW supports
LL suffix for long long int numbers which is even strictly necessary
but does not support 'll' in printf() format and use MS extension.
Who is defining it :-)

I have no idea, anyhow lucky we have standards in the computing field :)

BTW I've just found that in my MinGW builds PRG_USR was overwritten
by other settings and this build was created with default -gc
switch when I was making testing it.
In summary it will be good to implement our own *printf() formatting
somewhere. It's the most expensive function we are using from C RTL
and quite often written in two versions for MT and ST mode. The ST
one simply uses static variables.

Seems like a good idea. Considering the few CRTL stuff we use,
maybe we could just get rid of them all. At least as far as core is
concerned. [ Clipper did it, too. ]

BTW, I've just looked into the build results and there are 4 new
warnings after the hbdefs.h change :(

../../genc.c:1585: warning: format '%I64i' expects type 'int', but argument 3 has type 'long long int' ../../gencc.c:1239: warning: format '%I64i' expects type 'int', but argument 3 has type 'long long int' ../../estack.c:1019: warning: format '%I64i' expects type 'int', but argument 2 has type 'HB_LONG' ../../hbsqlit3.c:1136: warning: format '%I64d' expects type 'int', but argument 4 has type 'sqlite3_uint64'

Brgds,
Viktor

_______________________________________________
Harbour mailing list
Harbour@harbour-project.org
http://lists.harbour-project.org/mailman/listinfo/harbour

Reply via email to