Fix a bunch of little things in sitedoc.module

If you couldn't tell, the way I get to know the code of new modules is go through line-by-line and fix a bunch of little inconsistencies with them. ;)

So there are a few kind of odd-ball things in the .module file as well:

- The help hook has HTML entities in it; Drupal supports unicode, so it should just use the proper characters for the curly quotes. But let's just make them straight quotes like throughout the rest of the module, hm? :)
- The help hook also returns $output ."\n";.. nah. Just $ouput is fine.
- The module implements hook_access, but is not a node module. Therefore this hook will never fire. Removed it.
- There were calls to "theme_table" ... this should be "theme('table' ...)" so that themes can override the default output.
- Has a closing ?>. We keep those off so people don't have weird "header already sent" problems saving their module through web-based control panels and such.
- A bunch of minor coding-standards related changes. I gave up half way through... They mostly boiled down to:
- Indentation is all over the map.
- Inline comments should use // and not /* */ .. make commenting out a block of code for debugging purposes easier.
- Inline comment should also be capitalized and end in a period.

Here's the start of a patch, anyway... not sure if I'll be able to finish the rest off.