https://github.com/FreeRDP/FreeRDP/pull/33

Moving libfreerdp-core/constants_**.h to include/freerdp/constants/*.h
Moving include/freerdp/types_***.h to include/freerdp/types/**.h
Moving debug.h to include/freerdp
Moving libfreerdp-core/crypto_**.h to libfreerdp-core/crypto/**.h
Merging tls_openssl.c with crypto_openssl.c
Merging orderstypes.h with orders.h

Here is another pull request for a second round of refactoring + cleanup.

I would see two different things that would need renaming for consistency:

libfreerdp-core/frdp.h: isn't this header file corresponding to freerdp.c?
It could stay internal to libfreerdp-core, but renamed to freerdp.h. There
is another freerdp.h in the include folder, but if we stick to proper
includes (#include <freerdp/freerdp.h> versus #include "freerdp.h") there
shouldn't be a conflict.

Regarding virtual channel stuff, in some places "vchan" is used, while in
other places "chan" is used. Maybe we should simply stick to "channel"?
dynamic virtual channels are named dvc or drdynvc anyway, so there is no
conflict between the name used for dynamic virtual channels and static
virtual channels.
------------------------------------------------------------------------------
Simplify data backup and recovery for your virtual environment with vRanger.
Installation's a snap, and flexible recovery options mean your data is safe,
secure and there when you need it. Discover what all the cheering's about.
Get your free trial download today. 
http://p.sf.net/sfu/quest-dev2dev2 
_______________________________________________
Freerdp-devel mailing list
Freerdp-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/freerdp-devel

Reply via email to