On 03/17/2011 02:19 AM, Mads Kiilerich wrote: > On 03/16/2011 06:14 PM, Vic Lee wrote: > I would like to merge that with crypto_openssl.c, but I agree with the > direction of the change ;-)
I don't think it's a good idea to merge them. tls.c is more readable as a separated file. It implements very specific TLS streaming. crypto_* is just a set of crypto algorithms. > Before that I would like to have all references to openssl removed from > credssp.c and encapsulated in *_openssl.c. credssp.c is already done (actually the only function left in it was credssp_nonce and I moved it to crypto_nonce). However ntlmssp.c still has a lot of stuff that depends on openssl and it seems not trivial to move. > libfreerdp is "just" a library, and I don't think it should have its own > configuration files. The configuration belongs to the UI. Adding a > "known_hosts_file" parameter would make the functions more general - as > long as the UI don't want to store its configuration in xml or registry > or gconf or whatever ... Right, that's a good idea. > > /Mads > Vic ------------------------------------------------------------------------------ Colocation vs. Managed Hosting A question and answer guide to determining the best fit for your organization - today and in the future. http://p.sf.net/sfu/internap-sfd2d _______________________________________________ Freerdp-devel mailing list Freerdp-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/freerdp-devel
