Static-ify rsaenh/mpi?
Andrew Talbot
Andrew.Talbot at talbotville.com
Sat May 23 11:29:55 CDT 2009
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.
--
Andy.
More information about the wine-devel
mailing list