[PATCH] randr: stop clients from deleting immutable output properties
Luc Verhaegen
libv at skynet.be
Wed Aug 10 07:34:23 PDT 2011
On Wed, Aug 10, 2011 at 03:28:37PM +0100, Daniel Stone wrote:
> On Wed, Aug 10, 2011 at 03:53:03PM +0200, Luc Verhaegen wrote:
> > On Wed, Aug 10, 2011 at 03:48:30PM +0200, Luc Verhaegen wrote:
> > > 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.
> >
> > 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.
>
> *shrug*, I'm not the RandR maintainer, but it just seemed a bit daft to
> inline an entire function just to add one check in the middle.
In the first approximation, sure.
Luc Verhaegen.
More information about the xorg-devel
mailing list