Francois Gouget wrote:
So dlls/rsaenh/mpi.c defines 13 functions that are only used there. So they could be made static by tweaking mpi.c and tomcrypt.h.
However my understanding is that this files has been imported in Wine from another project so maybe it's not a good idea to diverge much from them.
What do you think? Should we just leave them as is?
Hi Francois,
I made a few of the functions in mpi.c static a while back, but stopped when there was a bug report (http://bugs.winehq.org/show_bug.cgi?id=17406) that seemed consequential but turned out to be apparently due to a bug gcc.
Also one of my patches to this file was rejected, which I didn't follow up, but it may have been due to the fact that there are comments to each function in both tomcrypt.h and mpi.c, and I tried to form combined versions in the latter.