reformat, whitespace, code simplification, FIXMEs

... to match it as closely as possible to the upstream version.

The autograb stuff had been forgotten.

Fixes: ArcticaProject/nx-libs#635

This is a string that contains the address, the index, the PID and the
process name of the client. The string can be used in debugging
messages to identify the client.

The only place where this was referenced outside Client.c was a
commented section in Handlers.c. For now lets mask all that by an
(unset) define.

no functional change

Backport from this commit:

  commit 8dfd98245d2c44a1eb4c8b7c275e6cfc10fe40f1
  Author: Chase Douglas <chase.douglas@canonical.com>
  Date:   Wed Apr 11 11:14:51 2012 -0700

    Fix copy/paste error from before git history in UpdateCurrentTimeIf()

    See UpdateCurrentTime() for reference. I don't know what bug this might
    trigger, but it wouldn't hurt to fix this.
Signed-off-by: Chase Douglas <chase.douglas@canonical.com>
Reviewed-by: Peter Hutterer <peter.hutterer@who-t.net>

Looks like a copy'n'paste error.

to make them all fit in the same naming scheme that better describes their purpose.

drop the nxagent prefix to emphasize they are not called from the
outside. Also make them static and add prototypes, if missing.

For now deactivate nxagentResetSelectionOwner. Must do some research
before removing it completely.

There were two location where the windowPtr was not cleared after
calling ClearSelectionOwner(). One was nxagentInitClipboard where it
was set to NULL before and the second one was nxagentClearSelection()
where it really should be reset.

So we can safely move this to where it really belongs.

and correctly print its value in debugging output

ensure same order for internal and external Atoms

Fixes ArcticaProject/nx-libs#907 (3/3)