One comment: you should not put window_title member into rdpSet. This 
has nothing to do with the core, but X11 specific. Please put it in 
xfInfo (X11/xf_types.h).

Thanks,

Vic

On 04/29/2011 11:20 PM, Tim Gouma wrote:
> 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.



------------------------------------------------------------------------------
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