[PATCH] randr: stop clients from deleting immutable output properties
Luc Verhaegen
libv at skynet.be
Wed Aug 10 06:53:03 PDT 2011
On Wed, Aug 10, 2011 at 03:48:30PM +0200, Luc Verhaegen wrote:
> On Wed, Aug 10, 2011 at 01:44:57PM +0100, Daniel Stone wrote:
> > On Wed, Aug 10, 2011 at 01:06:33PM +0200, Luc Verhaegen wrote:
> > > Immutable in randr means that clients are not able to alter the
> > > property itself, they are only allowed to alter the property value.
> > > This logically means that the property then should not be deleted
> > > by the client either.
> >
> > Instead of inlining RRDeleteOutputProperty here, you could do something
> > like this.
> >
> > Cheers,
> > Daniel
>
> This was not done for a few reasons:
> 1) it clearly splits between clientside and server internal usage. The
> comment that i added to RRDeleteOutputProperty in the very first version
> of the patch underlined that this was only for server internal use due
> to immutable (this was the code tested on the N9, but was never
> committed).
> 2) ValidAtom is only ever called in the client side functions, while
> the check for immutable is once in general and once in client side,
> putting the balance client side, so i picked client side.
> 3) it is much more invasive.
>
> Luc Verhaegen.
Not that it really matters either way, i just felt that the other had a
higher probability of being accepted as is, as any of the above three
reasons could've been cited for going for the smaller patch.
Luc Verhaegen.
More information about the xorg-devel
mailing list