This reaplace the easier usage of vtkMutexLock with std::mutex
Actual deprecation of vtkMutexLock will need more work and design.

attempt to fix issues from
vtk/vtk!4916

Traceback (most recent call last):
  File "...\wslink\websocket.py", line 275, in onMessage
    results = func(obj, *args, **kwargs)
  File "...\vtk\web\protocols.py", line 379, in addRenderObserver
    self.publish('viewport.image.push.subscription', pushRender())
  File "...\vtk\web\protocols.py", line 337, in pushRender
    reply = self.stillRender({ "view": realViewId, "mtime": self.trackingViews[vId]["mtime"] })
  File "...\vtk\web\protocols.py", line 309, in stillRender
    reply["memsize"] = reply_image.GetDataSize() if reply_image else 0
AttributeError: 'str' object has no attribute 'GetDataSize'

Found via `codespell`

Kitware Robot authored 6 years ago and Brad King committed 6 years ago

Populate a script with `map_<md5>=<sha512>` variable settings and
then run the following to convert the content link files:

    git ls-files -- '*.md5' | while read f; do
      md5="$(cat "$f")"
      sha512="$(eval echo \${map_$md5})"
      rm "$f"
      echo $sha512 > ${f%.md5}.sha512
    done

This required some clean up, because the automatic changes borked a couple of files, but mostly resulted in stupid whitespacing. A few find/replace fixed things up.

There are now some duplicate semis because it did changes like:

  ~AMRIndexIterator() override{};
to
  ~AMRIndexIterator() override= default;;

Note there was a pointless semi before, which clang-tidy didn’t expect / account for.

My next commit will remove them.

Removed unneed null checks before delete.

found via `codespell` 

Split `vtk` package into `vtkmodules` and `vtk`. `vtkmodules` doesn't
import any of the modules included in its `__init__.py`, instead
provides a `all.py` module that imports all modules in the old `vtk`
package. `vtk` is now a module that imports `vtkmodules/all.py` thus
behaving like the old `vtk` package.

The property isn't used anymore by the wrapping tools.

Now that Tcl wrapping is gone, all wrapping tools now use
`WRAP_EXCLUDE_PYTHON` instead.

Use `vtk.${vtk-module}` rather than `${vtk-module}Python` when importing
VTK modules.

Some VTK source code still had tabs (even some of the .py files!).
Also, several comments used utf-8 quotes, which are hard to tell
from regular quotes by eye and can lead to confusing mistakes.

This topic catches another batch of NULL conversions that
clang did not handle for various reasons. It also includes
a number of changes to comments and strings where NULL was
used. It also catches some recent code changes where NULL
was used instead of nullptr.

Now that VTK requires a C++11 compiler we can ditch using custom typedefs to
wrap =delete;