Remove spurious %s markers
Move Python version check to beginning of install.py

Before moving this block, pre-2.7 Python versions would
choke on the "import argparse" line a bit futher down, and
hence the users never got to see the error message that
explains that their Python version is too old.
Local variable "old_type" is not used, removing it
Use _global_ variable "rclinks_added"
Move json import to after python version check

This allows Python 2.5 users to see the "insufficient python version"
error message instead of choking on the import.
Merge pull request #7 from mo/python26_error_and_minor_cleanups

Fix Python26 error msg and also some minor cleanups
Add Critic user FAQ

Explains some functionality which could be easily missed otherwise.
Merge pull request #9 from mo/tips_and_tricks

Add Critic user FAQ
Avoid adding duplicated review filters on /createreview

If the user used the "Add Reviewer" or "Add Watcher" buttons to make
the same user a reviewer or watcher of the same path multiple times,
duplicate entries would be submitted to the server when submitting the
review, which caused the submit to fail.

Instead, just silently replace conflicting entries when new are added.

(The review filters UI on /createreview is rather bad and could use a
thorough redesign.)
Make /createreview a bit more flexible

Support specifying an arbitrary commit as the upstream to use when
selecting commits to include in the review instead of requiring it
to be a branch in the remote repository.  There might not be a
suitable branch in the remote repository, and it's silly to require
that one is created there just to create a review.

If the upstream is on the form "refs/*" it is fetched from the
remote repository, otherwise it is simply rev-parse:d in Critic's
repository after the work branch has been fetched.  This means an
abbreviated SHA-1 will work as long as the specified commit is
reachable from the head of the work branch and thus was fetched
when along with the work branch.
Improve /newrepository slightly

Make it more obvious that the remote repository is optional, and that
the branch name is only relevant if a remote repository is specified.

Also adjust error messages on invalid name and path input to match
the actual check done.
README: Fix broken link to user FAQ
Merge pull request #10 from Velmont/fix_broken_link

README: Fix broken link to user FAQ
Escape HTML characters in operation failure title and message

They are not escaped client-side, and although the title and message
is unlikely to ever contain content directly provided by the user,
it's typically still not supposed to be interpreted by HTML.

Also optionally disable the escaping of the message, in case the code
generating the message actually does generate HTML code.
Give useful error message when raising issue in closed review

The code just raised an exception, which got translated into a
standard "an unexpected error occurred" message being presented
to the user.  This is not an unexpected error, after all.
Give useful error message when creating comment via /showfile

If the commented file has been changed in the review (in any way
other than just being added or removed) the code for creating
comments can't deal with creating a comment in a single version
of the file rather than in the context of a diff.  Give a useful
error message when this limitation is hit, including a link to
a diff via which the comment could be created instead.

This is of course really just a stupid limitation that should be
fixed, but until then...
Improve client-side handling of failed asynchronous operations

If the client requests an asynchronous operation (i.e. provides a
callback function) and the operation fails, then make sure to call
the callback function after the error dialog is closed rather than
directly.  This makes it possible for the client code to try again
without risking trying again over and over while opening more and
more error dialogs on top of each other.
Fix installation of non-live extension versions

The path to 'tar' is spelled "configuration.executables.TAR", not
just "TAR".
Add <title> elements on some more pages

Was missing on /config, /branches and /tutorial (when not used to
show a specific tutorial, in which case the turorial's title was
set as the document's title automatically.)
Kill some unnecessary code duplication