aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorUlrich Sibiller <uli42@gmx.de>2019-07-23 21:25:41 +0200
committerMike Gabriel <mike.gabriel@das-netzwerkteam.de>2019-09-29 17:10:05 +0200
commit1f71a75f583c54b07a62f181d7301ffa1ade55e8 (patch)
treeeaf05389a0d94f20fc0ef1071468b8a92c49e670
parenta61102b6208cd61ef8a19f242aedb579799e4473 (diff)
downloadnx-libs-1f71a75f583c54b07a62f181d7301ffa1ade55e8.tar.gz
nx-libs-1f71a75f583c54b07a62f181d7301ffa1ade55e8.tar.bz2
nx-libs-1f71a75f583c54b07a62f181d7301ffa1ade55e8.zip
Atoms.c: code cleanup
no functional change
-rw-r--r--nx-X11/programs/Xserver/hw/nxagent/Atoms.c128
1 files changed, 54 insertions, 74 deletions
diff --git a/nx-X11/programs/Xserver/hw/nxagent/Atoms.c b/nx-X11/programs/Xserver/hw/nxagent/Atoms.c
index fa4c45b9a..a52cb9658 100644
--- a/nx-X11/programs/Xserver/hw/nxagent/Atoms.c
+++ b/nx-X11/programs/Xserver/hw/nxagent/Atoms.c
@@ -179,8 +179,6 @@ void nxagentWMDetect()
int nxagentInitAtoms(WindowPtr pWin)
{
- Atom atom;
-
/*
* Value of nxagentAtoms[8] is "NX_AGENT_SIGNATURE".
*
@@ -189,39 +187,43 @@ int nxagentInitAtoms(WindowPtr pWin)
* run nested.
*/
- atom = MakeAtom(nxagentAtomNames[8], strlen(nxagentAtomNames[8]), 1);
+ Atom atom = MakeAtom(nxagentAtomNames[8], strlen(nxagentAtomNames[8]), 1);
if (atom == None)
{
#ifdef PANIC
- fprintf(stderr, "nxagentInitAtoms: PANIC! Could not create [%s] atom.\n",
+ fprintf(stderr, "%s: PANIC! Could not create [%s] atom.\n", __func__,
nxagentAtomNames[8]);
#endif
return -1;
}
-
- return 1;
+ else
+ {
+ #ifdef TEST
+ fprintf(stderr, "nxagentInitAtoms: atom [%s] created with value [%d].\n",
+ nxagentAtomNames[8], atom);
+ #endif
+ return 1;
+ }
}
int nxagentQueryAtoms(ScreenPtr pScreen)
{
- int i;
static unsigned long atomGeneration = 1;
int num_of_atoms = NXAGENT_NUMBER_OF_ATOMS;
char *names[NXAGENT_NUMBER_OF_ATOMS];
#ifdef TEST
- fprintf(stderr, "nxagentQueryAtoms: Going to create the intern atoms on real display.\n");
+ fprintf(stderr, "%s: Going to create the intern atoms on real display.\n", __func__);
#endif
nxagentPrintAtomMapInfo("nxagentQueryAtoms: Entering");
- for (i = 0; i < num_of_atoms; i++)
+ for (int i = 0; i < num_of_atoms; i++)
{
names[i] = nxagentAtomNames[i];
-
nxagentAtoms[i] = None;
}
@@ -248,10 +250,10 @@ int nxagentQueryAtoms(ScreenPtr pScreen)
if (atomGeneration != serverGeneration)
{
#ifdef WARNING
- fprintf(stderr, "nxagentQueryAtoms: The nxagent has been reset with server %ld atom %ld.\n",
+ fprintf(stderr, "%s: The nxagent has been reset with server %ld atom %ld.\n", __func__,
serverGeneration, atomGeneration);
- fprintf(stderr, "nxagentQueryAtoms: Forcing a sync to detect the window manager.\n");
+ fprintf(stderr, "%s: Forcing a sync to detect the window manager.\n", __func__);
#endif
atomGeneration = serverGeneration;
@@ -309,9 +311,9 @@ int nxagentQueryAtoms(ScreenPtr pScreen)
#ifdef TEST
- for (i = 0; i < num_of_atoms; i++)
+ for (int i = 0; i < num_of_atoms; i++)
{
- fprintf(stderr, "nxagentQueryAtoms: Created intern atom [%s] with id [%ld].\n",
+ fprintf(stderr, "%s: Created intern atom [%s] with id [%ld].\n", __func__,
names[i], nxagentAtoms[i]);
}
#endif
@@ -353,9 +355,8 @@ static void nxagentExpandCache(void)
}
/*
- * Check if there is space left on the map
- * and manage the possible consequent allocation,
- * then cache the atom-couple.
+ * Check if there is space left on the map and manage the possible
+ * consequent allocation, then cache the atom-couple.
*/
static void nxagentWriteAtom(Atom local, Atom remote, const char *string, Bool duplicate)
@@ -363,10 +364,9 @@ static void nxagentWriteAtom(Atom local, Atom remote, const char *string, Bool d
const char *s;
/*
- * We could remove this string duplication if
- * we know for sure that the server will not
- * reset, since only at reset the dix layer
- * free all the atom names.
+ * We could remove this string duplication if we knew for sure that
+ * the server will not reset, since only at reset the dix layer
+ * frees all the atom names.
*/
if (duplicate)
@@ -399,19 +399,16 @@ static void nxagentWriteAtom(Atom local, Atom remote, const char *string, Bool d
}
/*
- * FIXME: We should clean up the atom map
- * at nxagent reset, in order to cancel
- * all the local atoms but still maintaining
- * the Xserver values and the atom names.
+ * FIXME: We should clean up the atom map at nxagent reset, in order
+ * to cancel all the local atoms but still maintaining the Xserver
+ * values and the atom names.
*/
void nxagentResetAtomMap(void)
{
- unsigned i;
-
nxagentPrintAtomMapInfo("nxagentResetAtomMap: Entering");
- for (i = 0; i < privLastAtom; i++)
+ for (unsigned int i = 0; i < privLastAtom; i++)
{
privAtomMap[i].local = None;
}
@@ -426,16 +423,13 @@ void nxagentResetAtomMap(void)
static int nxagentInitAtomMap(char **atomNameList, int count, Atom *atomsRet)
{
- XlibAtom *atom_list;
- char **name_list;
unsigned int i;
- int ret_value = 0;
int list_size = count + privLastAtom;
nxagentPrintAtomMapInfo("nxagentInitAtomMap: Entering");
- atom_list = malloc((list_size) * sizeof(*atom_list));
- name_list = malloc((list_size) * sizeof(char*));
+ XlibAtom *atom_list = malloc((list_size) * sizeof(*atom_list));
+ char **name_list = malloc((list_size) * sizeof(char*));
if ((atom_list == NULL) || (name_list == NULL))
{
@@ -459,7 +453,7 @@ static int nxagentInitAtomMap(char **atomNameList, int count, Atom *atomsRet)
* ... if successful cache them too.
*/
- ret_value = XInternAtoms(nxagentDisplay, name_list, list_size, False, atom_list);
+ int ret_value = XInternAtoms(nxagentDisplay, name_list, list_size, False, atom_list);
if (ret_value == 0)
{
@@ -517,22 +511,19 @@ static int nxagentInitAtomMap(char **atomNameList, int count, Atom *atomsRet)
}
/*
- * If the nxagent has been reset,
- * the local value of the atoms stored
- * in cache could have the value None,
- * do not call this function with None.
+ * If the nxagent has been reset, the local value of the atoms stored
+ * in cache could have the value None, do not call this function with
+ * None.
*/
static AtomMap* nxagentFindAtomByLocalValue(Atom local)
{
- unsigned i;
-
if (!ValidAtom(local))
{
return NULL;
}
- for (i = 0; i < privLastAtom; i++)
+ for (unsigned int i = 0; i < privLastAtom; i++)
{
if (local == privAtomMap[i].local)
{
@@ -545,14 +536,12 @@ static AtomMap* nxagentFindAtomByLocalValue(Atom local)
static AtomMap* nxagentFindAtomByRemoteValue(Atom remote)
{
- unsigned i;
-
if (remote == None || remote == BAD_RESOURCE)
{
return NULL;
}
- for (i = 0; i < privLastAtom; i++)
+ for (unsigned int i = 0; i < privLastAtom; i++)
{
if (remote == privAtomMap[i].remote)
{
@@ -565,9 +554,7 @@ static AtomMap* nxagentFindAtomByRemoteValue(Atom remote)
static AtomMap* nxagentFindAtomByName(char *string, unsigned int length)
{
- unsigned i;
-
- for (i = 0; i < privLastAtom; i++)
+ for (unsigned int i = 0; i < privLastAtom; i++)
{
if ((length == privAtomMap[i].length) &&
(strcmp(string, privAtomMap[i].string) == 0))
@@ -591,7 +578,6 @@ static AtomMap* nxagentFindAtomByName(char *string, unsigned int length)
Atom nxagentMakeAtom(char *string, unsigned int length, Bool Makeit)
{
- Atom local;
AtomMap *current;
/*
@@ -599,7 +585,7 @@ Atom nxagentMakeAtom(char *string, unsigned int length, Bool Makeit)
* our nxagentFindAtomByName.
*/
- local = MakeAtom(string, length, Makeit);
+ Atom local = MakeAtom(string, length, Makeit);
if (!ValidAtom(local))
{
@@ -623,7 +609,7 @@ Atom nxagentMakeAtom(char *string, unsigned int length, Bool Makeit)
if ((current = nxagentFindAtomByName(string, length)))
{
/*
- * Found Cached by name.
+ * Found cached by name.
* It means that nxagent has been reset,
* but not the xserver so we still have cached its atoms.
*/
@@ -634,13 +620,11 @@ Atom nxagentMakeAtom(char *string, unsigned int length, Bool Makeit)
}
/*
- * We really have to ask Xserver for it.
+ * We really have to ask the Xserver for it.
*/
{
- Atom remote;
-
- remote = XInternAtom(nxagentDisplay, string, !Makeit);
+ Atom remote = XInternAtom(nxagentDisplay, string, !Makeit);
if (remote == None)
{
@@ -659,10 +643,6 @@ Atom nxagentMakeAtom(char *string, unsigned int length, Bool Makeit)
Atom nxagentLocalToRemoteAtom(Atom local)
{
- AtomMap *current;
- const char *string;
- Atom remote;
-
#ifdef TEST
fprintf(stderr, "%s: entering\n", __func__);
#endif
@@ -683,7 +663,9 @@ Atom nxagentLocalToRemoteAtom(Atom local)
return local;
}
- if ((current = nxagentFindAtomByLocalValue(local)))
+ AtomMap *current = nxagentFindAtomByLocalValue(local);
+
+ if (current)
{
#ifdef TEST
fprintf(stderr, "%s: local [%d] -> remote [%d]\n", __func__, local, current->remote);
@@ -691,9 +673,9 @@ Atom nxagentLocalToRemoteAtom(Atom local)
return current->remote;
}
- string = NameForAtom(local);
+ const char *string = NameForAtom(local);
- remote = XInternAtom(nxagentDisplay, string, False);
+ Atom remote = XInternAtom(nxagentDisplay, string, False);
if (remote == None)
{
@@ -715,10 +697,6 @@ Atom nxagentLocalToRemoteAtom(Atom local)
Atom nxagentRemoteToLocalAtom(Atom remote)
{
- AtomMap *current;
- char *string;
- Atom local;
-
if (remote == None || remote == BAD_RESOURCE)
{
#ifdef DEBUG
@@ -735,11 +713,13 @@ Atom nxagentRemoteToLocalAtom(Atom remote)
return remote;
}
- if ((current = nxagentFindAtomByRemoteValue(remote)))
+ AtomMap *current = nxagentFindAtomByRemoteValue(remote);
+
+ if (current)
{
if (!ValidAtom(current->local))
{
- local = MakeAtom(current->string, current->length, True);
+ Atom local = MakeAtom(current->string, current->length, True);
if (ValidAtom(local))
{
@@ -761,14 +741,16 @@ Atom nxagentRemoteToLocalAtom(Atom remote)
return current->local;
}
- if ((string = XGetAtomName(nxagentDisplay, remote)))
+ char *string = XGetAtomName(nxagentDisplay, remote);
+
+ if (string)
{
- local = MakeAtom(string, strlen(string), True);
+ Atom local = MakeAtom(string, strlen(string), True);
if (!ValidAtom(local))
{
#ifdef WARNING
- fprintf(stderr, "nxagentRemoteToLocalAtom: WARNING MakeAtom failed.\n");
+ fprintf(stderr, "%s: WARNING MakeAtom failed.\n", __func__);
#endif
local = None;
@@ -785,7 +767,7 @@ Atom nxagentRemoteToLocalAtom(Atom remote)
}
#ifdef WARNING
- fprintf(stderr, "nxagentRemoteToLocalAtom: WARNING failed to get name from remote atom.\n");
+ fprintf(stderr, "%s: WARNING failed to get name from remote atom.\n", __func__);
#endif
return None;
@@ -795,13 +777,11 @@ Atom nxagentRemoteToLocalAtom(Atom remote)
static void nxagentPrintAtomMapInfo(char *message)
{
- unsigned i;
-
fprintf(stderr, "--------------- Atom map in context [%s] ----------------------\n", message);
fprintf(stderr, "nxagentPrintAtomMapInfo: Map at [%p] size [%d] number of entry [%d] auto increment [%d].\n",
(void*) privAtomMap, privLastAtom, privAtomMapSize, NXAGENT_ATOM_MAP_SIZE_INCREMENT);
- for (i = 0; i < privLastAtom; i++)
+ for (unsigned int i = 0; i < privLastAtom; i++)
{
fprintf(stderr, "[%5.1d] local: %6.1u - remote: %6.1u - [%p] %s\n", i,
privAtomMap[i].local,