Tim Gouma wrote, On 04/29/2011 05:20 PM: > Hi! > > I made an patch for FreeRDP to support alternative window titles > configurable via a command line option. > > I don't know if this is the right way to deliver the patch, following > your code standards (although I tried to adapt the style of the other > code) or if this is the best way to do it. This is my first time > coding anything in this language (C?). > > So see for yourself what you are going to do with this patch, I > thought it would be useful to post here.
We unfortunately have some code duplication between xfreerdp and wfreerdp. I think it would be great if it could be kept as aligned as possible so it would be easier to consolidate eventually, so I would prefer if you could update the stuff in win/wfreerdp as well. 'printf("komt ie:' should probably be something else if it should be there at all. /Mads ------------------------------------------------------------------------------ WhatsUp Gold - Download Free Network Management Software The most intuitive, comprehensive, and cost-effective network management toolset available today. Delivers lowest initial acquisition cost and overall TCO of any competing solution. http://p.sf.net/sfu/whatsupgold-sd _______________________________________________ Freerdp-devel mailing list Freerdp-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/freerdp-devel