> On Monday 13 February 2012 23:25:13 Timothy Pearson wrote:
>
>> > The second commit I pointed out was 9cc1e2c1: I think others already
>> > commented
>> > in my blog comments why this one is rather bad from a users point of
>> view
>> > (introducing new config options without removing the obsoleted ones).
>> But
>> > well
>> > the main issue from my point of view is that it modified an enum in a
>> > public
>> > header by not appending to the end, but in the middle. I think you can
>> > imagine
>> > what happens to 3rd party offerings compiled against the previous
>> version.
>>
>> We are not too concerned about ABI compatibility here, but since the
>> requested change is so trivial I suppose I can push it through.
>
> This is completely useless and broken patch because all the same
> functionality already existed in KDE3 (i.e. the shadows).
> And the newly introduced shadows have less functionality than those
> existed originally (i.e. they are not displayed when moving a window).
>
> I suspect this patch was introduced before the final KDE 3.5.10 was
> released, when KDE still had no shadow function in kwin.

I do agree that if shadow code has been duplicated, the broken code should
be removed.  I seem to remember KDE 3.5.10 having badly broken shadows,
though at this distance I could be mistaken.

Actually, with all the repairs I made to kompmgr, and given the inherent
bugginess of the non-composited kwin shadow code, I'd like to see the
non-composited shadows removed altogether.  Non-composited fading has a
similar problem IIRC, although I have not tested it as I run kompmgr
exclusively on my systems.

Tim