On Sat, Apr 02, 2011 at 09:59:30PM +0200, Aurélien Aptel wrote:
> On Sat, Apr 2, 2011 at 6:11 PM, Petr Sabata <psab...@redhat.com> wrote:
> > This patch adds UTF8 clipboard support via UTF8_STRING, if it's available. 
> > This
> > is needed to correctly exchange UTF8 data with other applications.
> 
> ok

Thanks.

> 
> > -       } else if(xsre->target == XA_STRING) {
> > +       } else if(xsre->target == seltarget || xsre->target == XA_STRING) {
> 
> Why is this necessary? Can't we just do:
> 
> } else if(xsre->target == seltarget) {

I included this to be compatible with applications requesting XA_STRING. I'm
not sure if there's a real benefit in that, so I guess you can drop it.

-- 
Petr 'contyk' Sabata, Red Hat

Attachment: pgpbuNF4Xt3PE.pgp
Description: PGP signature

Reply via email to