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

Reply via email to