A good little module that does its job well.
I have a few niggles with it, but don't take these too negatively, the module works just fine already.
1. The clock update is triggered every second, even when the seconds aren't being shown. This causes a slight page flicker which is a bit distracting. My rough fix is just to change the setTimeout() interval to 60000 when no seconds are being used.
2. The Javascript is all inline which the HTML rather than being in a separate .js file. Not really a problem, it just looks a bit messy.
3. The styles are also inline rather than using the template CSS file. From other comments it looks like this is deliberate, but it's a bit of a pain to have CSS styles spread around the place.
Owner's reply:
Changing the timeout will affect the accuracy. Currently it locks the time to the pc and the server. The time should be the server time not the time on your pc. If you change the timeout the clocks will drift, condiderably if you leave them up on the page for a long time. As it is written you should not be more than a couple of seconds off for any amount of time up.
The javascript is written using php to do all of the calculations updating an include file is problematic on many installations.