From: Rodolfo García Peñas (kix) <[email protected]>
The variable clip_icon, that contains the Clip appicon
is moved to the global clip properties.
Now the screen is not needed to know the clip_icon.
---
src/WindowMaker.h | 2 ++
src/balloon.c | 2 +-
src/defaults.c | 2 +-
src/dock.c | 47 +++++++++++++++++++++++------------------------
src/dockedapp.c | 2 +-
src/screen.c | 2 +-
src/screen.h | 1 -
src/workspace.c | 12 ++++++------
8 files changed, 35 insertions(+), 35 deletions(-)
diff --git a/src/WindowMaker.h b/src/WindowMaker.h
index dd5f480..92baf11 100644
--- a/src/WindowMaker.h
+++ b/src/WindowMaker.h
@@ -498,6 +498,8 @@ extern struct wmaker_global_variables {
/* Clip related */
struct {
+ struct WAppIcon *icon; /* The clip main icon, or the dock's,
if they are merged */
+
struct WMenu *menu; /* Menu for clips */
struct WMenu *submenu; /* Workspace list for clips */
struct WMenu *opt_menu; /* Options for Clip */
diff --git a/src/balloon.c b/src/balloon.c
index cd95cd2..6c056e4 100644
--- a/src/balloon.c
+++ b/src/balloon.c
@@ -439,7 +439,7 @@ static void appiconBalloon(WObjDescriptor *object)
char *tmp;
/* Show balloon if it is the Clip and the workspace name is > 5 chars */
- if (object->parent == scr->clip_icon) {
+ if (object->parent == w_global.clip.icon) {
if
(strlen(w_global.workspace.array[w_global.workspace.current]->name) > 5) {
scr->balloon->text =
wstrdup(w_global.workspace.array[w_global.workspace.current]->name);
} else {
diff --git a/src/defaults.c b/src/defaults.c
index 7d20908..0a29a9e 100644
--- a/src/defaults.c
+++ b/src/defaults.c
@@ -1228,7 +1228,7 @@ void wDefaultUpdateIcons(WScreen *scr)
}
if (!wPreferences.flags.noclip ||
wPreferences.flags.clip_merged_in_dock)
- wClipIconPaint(scr->clip_icon);
+ wClipIconPaint(w_global.clip.icon);
for (dc = scr->drawers; dc != NULL; dc = dc->next)
wDrawerIconPaint(dc->adrawer->icon_array[0]);
diff --git a/src/dock.c b/src/dock.c
index 3320f6c..9d691b2 100644
--- a/src/dock.c
+++ b/src/dock.c
@@ -527,7 +527,7 @@ static void keepIconsCallback(WMenu *menu, WMenuEntry
*entry)
selectedIcons = getSelected(dock);
if (!WMGetArrayItemCount(selectedIcons)
- && clickedIcon != dock->screen_ptr->clip_icon) {
+ && clickedIcon != w_global.clip.icon) {
char *command = NULL;
if (!clickedIcon->command && !clickedIcon->editing) {
@@ -816,8 +816,8 @@ static WAppIcon *mainIconCreate(WScreen *scr, int type,
const char *name)
switch(type) {
case WM_CLIP:
- if (scr->clip_icon)
- return scr->clip_icon;
+ if (w_global.clip.icon)
+ return w_global.clip.icon;
btn = wAppIconCreateForDock(scr, NULL, "Logo", "WMClip",
TILE_CLIP);
btn->icon->core->descriptor.handle_expose = clipIconExpose;
@@ -852,7 +852,7 @@ static WAppIcon *mainIconCreate(WScreen *scr, int type,
const char *name)
btn->docked = 1;
if (type == WM_CLIP ||
(type == WM_DOCK && wPreferences.flags.clip_merged_in_dock))
- scr->clip_icon = btn;
+ w_global.clip.icon = btn;
return btn;
}
@@ -860,7 +860,6 @@ static WAppIcon *mainIconCreate(WScreen *scr, int type,
const char *name)
static void switchWSCommand(WMenu *menu, WMenuEntry *entry)
{
WAppIcon *btn, *icon = (WAppIcon *) entry->clientdata;
- WScreen *scr = icon->icon->core->screen_ptr;
WDock *src, *dest;
WMArray *selectedIcons;
int x, y;
@@ -881,7 +880,7 @@ static void switchWSCommand(WMenu *menu, WMenuEntry *entry)
XUnmapWindow(dpy, btn->icon->core->window);
}
}
- } else if (icon != scr->clip_icon) {
+ } else if (icon != w_global.clip.icon) {
if (wDockFindFreeSlot(dest, &x, &y)) {
wDockMoveIconBetweenDocks(src, dest, icon, x, y);
XUnmapWindow(dpy, icon->icon->core->window);
@@ -976,7 +975,7 @@ static WMenu *makeWorkspaceMenu(WScreen *scr)
if (!menu)
wwarning(_("could not create workspace submenu for Clip menu"));
- wMenuAddCallback(menu, "", switchWSCommand, (void *)scr->clip_icon);
+ wMenuAddCallback(menu, "", switchWSCommand, (void *)w_global.clip.icon);
menu->flags.realized = 0;
wMenuRealize(menu);
@@ -1414,7 +1413,7 @@ static void clipIconExpose(WObjDescriptor *desc, XEvent
*event)
static void dockIconPaint(WAppIcon *btn)
{
- if (btn == btn->icon->core->screen_ptr->clip_icon) {
+ if (btn == w_global.clip.icon) {
wClipIconPaint(btn);
} else if (wIsADrawer(btn)) {
wDrawerIconPaint(btn);
@@ -1452,7 +1451,7 @@ static WMPropList *make_icon_state(WAppIcon *btn)
buggy = btn->buggy_app ? dYes : dNo;
- if (!wPreferences.flags.clip_merged_in_dock && btn ==
btn->icon->core->screen_ptr->clip_icon)
+ if (!wPreferences.flags.clip_merged_in_dock && btn ==
w_global.clip.icon)
snprintf(buffer, sizeof(buffer), "%i,%i", btn->x_pos,
btn->y_pos);
else
snprintf(buffer, sizeof(buffer), "%hi,%hi",
btn->xindex, btn->yindex);
@@ -1584,7 +1583,7 @@ void wClipSaveState(WScreen *scr)
{
WMPropList *clip_state;
- clip_state = make_icon_state(scr->clip_icon);
+ clip_state = make_icon_state(w_global.clip.icon);
WMPutInPLDictionary(scr->session_state, dClip, clip_state);
@@ -1963,8 +1962,8 @@ WDock *wDockRestoreState(WScreen *scr, WMPropList
*dock_state, int type)
* incremented in the loop above.
*/
} else if (old_top != dock->icon_array[0]) {
- if (old_top == scr->clip_icon) // TODO dande: understand the
logic
- scr->clip_icon = dock->icon_array[0];
+ if (old_top == w_global.clip.icon) /* TODO dande: understand
the logic */
+ w_global.clip.icon = dock->icon_array[0];
wAppIconDestroy(old_top);
}
@@ -3272,7 +3271,7 @@ void wDockTrackWindowLaunch(WDock *dock, Window window)
void wClipUpdateForWorkspaceChange(WScreen *scr, int workspace)
{
if (!wPreferences.flags.noclip) {
- scr->clip_icon->dock =
w_global.workspace.array[workspace]->clip;
+ w_global.clip.icon->dock =
w_global.workspace.array[workspace]->clip;
if (w_global.workspace.current != workspace) {
WDock *old_clip =
w_global.workspace.array[w_global.workspace.current]->clip;
WAppIconChain *chain = scr->global_icons;
@@ -3428,7 +3427,7 @@ static void openDockMenu(WDock *dock, WAppIcon *aicon,
XEvent *event)
if (dock->type == WM_CLIP) {
/* Rename Workspace */
entry = dock->menu->entries[++index];
- if (aicon == scr->clip_icon) {
+ if (aicon == w_global.clip.icon) {
entry->callback = renameCallback;
entry->clientdata = dock;
entry->flags.indicator = 0;
@@ -3452,7 +3451,7 @@ static void openDockMenu(WDock *dock, WAppIcon *aicon,
XEvent *event)
entry = dock->menu->entries[++index];
entry->clientdata = aicon;
entry->flags.indicator_on = aicon->icon->selected;
- wMenuSetEnabled(dock->menu, index, aicon != scr->clip_icon &&
!wIsADrawer(aicon));
+ wMenuSetEnabled(dock->menu, index, aicon != w_global.clip.icon
&& !wIsADrawer(aicon));
/* select/unselect all icons */
entry = dock->menu->entries[++index];
@@ -3605,7 +3604,7 @@ static void iconDblClick(WObjDescriptor *desc, XEvent
*event)
if (event->xbutton.state & MOD_MASK) {
/* raise/lower dock */
toggleLowered(dock);
- } else if (btn == dock->screen_ptr->clip_icon) {
+ } else if (btn == w_global.clip.icon) {
if (getClipButton(event->xbutton.x,
event->xbutton.y) != CLIP_IDLE)
handleClipChangeWorkspace(dock->screen_ptr, event);
else if
(wPreferences.flags.clip_merged_in_dock) {
@@ -3855,14 +3854,14 @@ static void handleClipChangeWorkspace(WScreen *scr,
XEvent *event)
XEvent ev;
int done, direction, new_ws;
int new_dir;
- WDock *clip = scr->clip_icon->dock;
+ WDock *clip = w_global.clip.icon->dock;
direction = getClipButton(event->xbutton.x, event->xbutton.y);
clip->lclip_button_pushed = direction == CLIP_REWIND;
clip->rclip_button_pushed = direction == CLIP_FORWARD;
- wClipIconPaint(scr->clip_icon);
+ wClipIconPaint(w_global.clip.icon);
done = 0;
while (!done) {
WMMaskEvent(dpy, ExposureMask | ButtonMotionMask |
ButtonReleaseMask | ButtonPressMask, &ev);
@@ -3877,7 +3876,7 @@ static void handleClipChangeWorkspace(WScreen *scr,
XEvent *event)
direction = new_dir;
clip->lclip_button_pushed = direction ==
CLIP_REWIND;
clip->rclip_button_pushed = direction ==
CLIP_FORWARD;
- wClipIconPaint(scr->clip_icon);
+ wClipIconPaint(w_global.clip.icon);
}
break;
@@ -3909,7 +3908,7 @@ static void handleClipChangeWorkspace(WScreen *scr,
XEvent *event)
wWorkspaceChange(scr, w_global.workspace.count - 1);
}
- wClipIconPaint(scr->clip_icon);
+ wClipIconPaint(w_global.clip.icon);
}
static void iconMouseDown(WObjDescriptor *desc, XEvent *event)
@@ -3941,7 +3940,7 @@ static void iconMouseDown(WObjDescriptor *desc, XEvent
*event)
else
wDockRaise(dock);
- if ((event->xbutton.state & ShiftMask) && aicon !=
scr->clip_icon && dock->type != WM_DOCK) {
+ if ((event->xbutton.state & ShiftMask) && aicon !=
w_global.clip.icon && dock->type != WM_DOCK) {
wIconSelect(aicon->icon);
return;
}
@@ -3957,7 +3956,7 @@ static void iconMouseDown(WObjDescriptor *desc, XEvent
*event)
if (wPreferences.single_click && !hasMoved)
iconDblClick(desc, event);
}
- } else if (event->xbutton.button == Button2 && aicon == scr->clip_icon)
{
+ } else if (event->xbutton.button == Button2 && aicon ==
w_global.clip.icon) {
if (!w_global.clip.ws_menu)
w_global.clip.ws_menu = wWorkspaceMenuMake(scr, False);
@@ -3980,7 +3979,7 @@ static void iconMouseDown(WObjDescriptor *desc, XEvent
*event)
(*desc->handle_mousedown) (desc, event);
}
} else if (event->xbutton.button == Button2 && dock->type == WM_CLIP &&
- (event->xbutton.state & ShiftMask) && aicon !=
scr->clip_icon) {
+ (event->xbutton.state & ShiftMask) && aicon !=
w_global.clip.icon) {
wClipMakeIconOmnipresent(aicon, !aicon->omnipresent);
} else if (event->xbutton.button == Button3) {
if (event->xbutton.send_event &&
@@ -4174,7 +4173,7 @@ int wClipMakeIconOmnipresent(WAppIcon *aicon, int
omnipresent)
WAppIconChain *new_entry, *tmp, *tmp1;
int status = WO_SUCCESS;
- if ((scr->dock && aicon->dock == scr->dock) || aicon == scr->clip_icon)
+ if ((scr->dock && aicon->dock == scr->dock) || aicon ==
w_global.clip.icon)
return WO_NOT_APPLICABLE;
if (aicon->omnipresent == omnipresent)
diff --git a/src/dockedapp.c b/src/dockedapp.c
index 4c77cfa..adccd5b 100644
--- a/src/dockedapp.c
+++ b/src/dockedapp.c
@@ -177,7 +177,7 @@ static void panelBtnCallback(WMWidget * self, void *data)
WAppIcon *aicon = panel->editedIcon;
// Cf dock.c:dockIconPaint(WAppIcon *aicon)?
- if (aicon == aicon->icon->core->screen_ptr->clip_icon)
+ if (aicon == w_global.clip.icon)
wClipIconPaint(aicon);
else if (wIsADrawer(aicon))
wDrawerIconPaint(aicon);
diff --git a/src/screen.c b/src/screen.c
index 88821bd..710da30 100644
--- a/src/screen.c
+++ b/src/screen.c
@@ -829,7 +829,7 @@ void wScreenRestoreState(WScreen * scr)
if (!wPreferences.flags.noclip) {
state = WMGetFromPLDictionary(scr->session_state, dClip);
- scr->clip_icon = wClipRestoreState(scr, state);
+ w_global.clip.icon = wClipRestoreState(scr, state);
}
if (!wPreferences.flags.nodrawer)
diff --git a/src/screen.h b/src/screen.h
index 0d667a5..4387ead 100644
--- a/src/screen.h
+++ b/src/screen.h
@@ -213,7 +213,6 @@ typedef struct _WScreen {
struct WDock *dock; /* the application dock */
struct WPixmap *dock_dots; /* 3 dots for the Dock */
Window dock_shadow; /* shadow for dock buttons */
- struct WAppIcon *clip_icon; /* The clip main icon, or the dock's,
if they are merged */
struct WDock *last_dock;
WAppIconChain *global_icons; /* for omnipresent icons chain in clip
*/
int global_icon_count; /* How many global icons do we have */
diff --git a/src/workspace.c b/src/workspace.c
index 4d1b301..3894852 100644
--- a/src/workspace.c
+++ b/src/workspace.c
@@ -620,11 +620,11 @@ void wWorkspaceForceChange(WScreen * scr, int workspace)
if (!wPreferences.flags.noclip &&
(w_global.workspace.array[workspace]->clip->auto_collapse ||
w_global.workspace.array[workspace]->clip->auto_raise_lower)) {
/* to handle enter notify. This will also */
- XUnmapWindow(dpy, scr->clip_icon->icon->core->window);
- XMapWindow(dpy, scr->clip_icon->icon->core->window);
+ XUnmapWindow(dpy, w_global.clip.icon->icon->core->window);
+ XMapWindow(dpy, w_global.clip.icon->icon->core->window);
}
- else if (scr->clip_icon != NULL) {
- wClipIconPaint(scr->clip_icon);
+ else if (w_global.clip.icon != NULL) {
+ wClipIconPaint(w_global.clip.icon);
}
wScreenUpdateUsableArea(scr);
wNETWMUpdateDesktop(scr);
@@ -699,8 +699,8 @@ void wWorkspaceRename(WScreen *scr, int workspace, const
char *name)
}
}
- if (scr->clip_icon)
- wClipIconPaint(scr->clip_icon);
+ if (w_global.clip.icon)
+ wClipIconPaint(w_global.clip.icon);
WMPostNotificationName(WMNWorkspaceNameChanged, scr, (void
*)(uintptr_t) workspace);
}
--
1.8.4.rc3
--
To unsubscribe, send mail to [email protected].
