<div dir="ltr">Hi Nicolas,<div><br><div class="gmail_quote"><div dir="ltr">On Tue, Dec 11, 2018 at 1:38 AM Nicolas Cellier <<a href="mailto:nicolas.cellier.aka.nice@gmail.com">nicolas.cellier.aka.nice@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex"> <div dir="ltr"><div dir="ltr"><div dir="ltr"><div>Hi,</div><div>Compiler warnings on 64bits for B3DAccelerator Plugin makes me think it can't work:<br></div><div><br></div><div>clang-5.0: warning: argument unused during compilation: '-mdll' [-Wunused-command-line-argument]<br>../../platforms/Cross/plugins/B3DAcceleratorPlugin/sqOpenGLRenderer.c:103:19: warning: implicit declaration of function 'getImageName' is invalid in C99 [-Wimplicit-function-declaration]<br>                strcpy(fileName,getImageName());<br>                                ^<br>../../platforms/Cross/plugins/B3DAcceleratorPlugin/sqOpenGLRenderer.c:103:19: warning: incompatible integer to pointer conversion passing 'int' to parameter of type 'const char *' [-Wint-conversion]<br>                strcpy(fileName,getImageName());<br>                                ^~~~~~~~~~~~~~<br>/usr/x86_64-w64-mingw32/sys-root/mingw/include/string.h:61:77: note: passing argument to parameter '_Source' here<br>  char * __cdecl strcpy(char * __restrict__ _Dest,const char * __restrict__ _Source);<br></div><div><br></div><div>We are going to pass an address on 32bits because 
getImageName() has not been declared and will be considered as returning an int...</div><div>If the address were on low end of address space it could eventually work, but we can't depend on such property.</div></div></div></div></blockquote><div><br></div><div>Agreed.  Ugh; this is ugly code (and if I'm responsible I apologize).  What it's trying to do is find the image file's directory and write the Squeak3D.log file there-in.  I think there are a few other places that do this.  So instead of replicating this ugly code (that won't work on Windows) we should be using e.g. something called getImageFileDirectory, which may have to be written.  I'll try and look at this soon.  Maybe raise an issue on open smalltalk-vm?</div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div><br></div><div>It's vital that we properly declare function prototypes (preferably by using ad hoc header) and all  [-Wimplicit-function-declaration] reports deserve greater attention.<br></div></div></div></div>
</blockquote></div><br clear="all"><div><br></div>-- <br><div dir="ltr" class="gmail_signature"><div dir="ltr"><div><span style="font-size:small;border-collapse:separate"><div>_,,,^..^,,,_<br></div><div>best, Eliot</div></span></div></div></div></div></div>