Fixed workspace-indicator issue triggered by IntelliJ IDEA

Normally the workspace_indicator_timeout config option creates an indicator with
the name of the current workspace in the corder of the screen. I was observing
an issue where switching workspaces away from an IntelliJ IDEA window causes the
indicator to not be displayed. This was due to logic in notion that removes the
indicator when an NULL or unnamed screen or mplex are focussed. Removing this
logic fixes the issue, and I can find no reason this was needed in the first
place. Things work well without.
mod_notionflux Makefile improvement

more re-use, 'make realclean' now cleans up better.
Use C-style comments rather than C++-style

so /* */ instead of //
Declaration on top
Revert "Declaration on top"

This reverts commit dd019523c8996503df609f299b839ce5966becf8.
Declare local variable at start of method
Don't double-free the rootwin

When initializing fails, it is the responsibility of CREATOBJ_IMP to
free the memory. Thanks to Coverity.
Code Cleanup: indentation (tabs -> spaces)
Code cleanup: remove unused vars

also comment in cases where it looks like they could be useful
Remove commented-out variables
Include string.h for strlen and friends
More cleanups of dead code
Add macro to mark parameters as unused
Mark callback parameters as 'intentionally unused'

.. and enable more warnings for others ;)
Remove unused variables
Another unused parameter
Remove unused mod_query parameters
Use new UNUSED macro in dock.c