On Sat, 23 Mar 2013 at 14:22:26 +0100, Rodolfo García Peñas wrote: > On Fri, 22 Mar 2013, [email protected] escribió: > > > From: Daniel Déchelotte <[email protected]> > > > > --- > > src/dock.c | 2 ++ > > 1 file changed, 2 insertions(+) > > > > diff --git a/src/dock.c b/src/dock.c > > index 21d6aee..f9da29f 100644 > > --- a/src/dock.c > > +++ b/src/dock.c > > @@ -3796,6 +3796,8 @@ static void clipEnterNotify(WObjDescriptor *desc, > > XEvent *event) > > return; > > > > scr = btn->icon->core->screen_ptr; > > + /* Useless test: if btn->omnipresent, then btn->dock is > > + precisely scr->workspaces[scr->current_workspace]->clip */ > > if (!btn->omnipresent) > > dock = btn->dock; > > else > > I cannot understand it :-( can you explain it more.
He is removing a code comment, but superficially the comment seems to be relevant since it is providing information which might not be obvious from just reading the code at that point. If I'm not crazy, I guess the patch should keep the information and simply delete the "Useless test:" warning. -- To unsubscribe, send mail to [email protected].
