2025-04-28 21:07:59 +03:00
106 changed files with 827 additions and 1518 deletions
--- a/.clang-tidy
+++ b/.clang-tidy
@ -1,15 +1,18 @@
 Checks: >
  -*,
+  boost-*,
  portability-*,
  clang-analyzer-*,
-  -clang-analyzer-optin.*,
+  -clang-analyzer-optin*,
  -clang-analyzer-cplusplus.NewDeleteLeaks,
-  -clang-analyzer-cplusplus.NewDelete,
  -clang-analyzer-core.CallAndMessage,
-  modernize-avoid-bind,
-  readability-identifier-naming
-WarningsAsErrors: '*'
-HeaderFilterRegex: '(apps|components)/'
-CheckOptions:
- key: readability-identifier-naming.ConceptCase
-  value: CamelCase
+  -modernize-avoid-bind
+WarningsAsErrors: >
+  -*,
+  boost-*,
+  portability-*,
+  clang-analyzer-*,
+  -clang-analyzer-optin*,
+  -clang-analyzer-cplusplus.NewDeleteLeaks,
+  -clang-analyzer-core.CallAndMessage
+HeaderFilterRegex: '^(apps|components)'
--- a/.gitlab-ci.yml
+++ b/.gitlab-ci.yml
@ -632,16 +632,12 @@ macOS14_Xcode15_arm64:
    - |
      if (Get-ChildItem -Recurse *.pdb) {
        7z a -tzip "..\..\$(Make-SafeFileName("OpenMW_MSVC2022_64_${config}_${CI_COMMIT_REF_NAME}_${CI_JOB_ID}_symbols.zip"))" '*.pdb' CI-ID.txt
-        if(!$?) { Exit $LASTEXITCODE }
        if (Test-Path env:AWS_ACCESS_KEY_ID) {
          aws --endpoint-url https://rgw.ctrl-c.liu.se s3 cp "..\..\$(Make-SafeFileName("OpenMW_MSVC2022_64_${config}_${CI_COMMIT_REF_NAME}_${CI_JOB_ID}_symbols.zip"))" s3://openmw-artifacts/${artifactDirectory}
-          if(!$?) { Exit $LASTEXITCODE }
        }
        Push-Location ..
        ..\CI\Store-Symbols.ps1 -SkipCompress
-        if(!$?) { Exit $LASTEXITCODE }
        7z a -tzip "..\$(Make-SafeFileName("OpenMW_MSVC2022_64_${config}_${CI_COMMIT_REF_NAME}_${CI_JOB_ID}_sym_store.zip"))" '.\SymStore\*' $config\CI-ID.txt
-        if(!$?) { Exit $LASTEXITCODE }
        Pop-Location
        Get-ChildItem -Recurse *.pdb | Remove-Item
      }
@ -649,20 +645,13 @@ macOS14_Xcode15_arm64:
    - |
      if (Test-Path env:AWS_ACCESS_KEY_ID) {
        aws --endpoint-url https://rgw.ctrl-c.liu.se s3 cp "..\..\$(Make-SafeFileName("OpenMW_MSVC2022_64_${config}_${CI_COMMIT_REF_NAME}.zip"))" s3://openmw-artifacts/${artifactDirectory}
-        if(!$?) { Exit $LASTEXITCODE }
-      }
-    - |
-      if ($executables) {
-        foreach ($exe in $executables.Split(',')) {
-          & .\$exe
-          if(!$?) { Exit $LASTEXITCODE }
-        }
      }
+    - if ($executables) { foreach ($exe in $executables.Split(',')) { & .\$exe } }
  after_script:
    - Get-Volume
    - Copy-Item C:\ProgramData\chocolatey\logs\chocolatey.log
  cache:
-    key: ninja-2022-v12
+    key: ninja-2022-v11
    paths:
    - ccache
    - deps
@ -790,16 +779,12 @@ macOS14_Xcode15_arm64:
    - |
      if (Get-ChildItem -Recurse *.pdb) {
        7z a -tzip "..\..\$(Make-SafeFileName("OpenMW_MSVC2022_64_${config}_${CI_COMMIT_REF_NAME}_${CI_JOB_ID}_symbols.zip"))" '*.pdb' CI-ID.txt
-        if(!$?) { Exit $LASTEXITCODE }
        if (Test-Path env:AWS_ACCESS_KEY_ID) {
          aws --endpoint-url https://rgw.ctrl-c.liu.se s3 cp "..\..\$(Make-SafeFileName("OpenMW_MSVC2022_64_${config}_${CI_COMMIT_REF_NAME}_${CI_JOB_ID}_symbols.zip"))" s3://openmw-artifacts/${artifactDirectory}
-          if(!$?) { Exit $LASTEXITCODE }
        }
        Push-Location ..
        ..\CI\Store-Symbols.ps1 -SkipCompress
-        if(!$?) { Exit $LASTEXITCODE }
        7z a -tzip "..\$(Make-SafeFileName("OpenMW_MSVC2022_64_${config}_${CI_COMMIT_REF_NAME}_${CI_JOB_ID}_sym_store.zip"))" '.\SymStore\*' $config\CI-ID.txt
-        if(!$?) { Exit $LASTEXITCODE }
        Pop-Location
        Get-ChildItem -Recurse *.pdb | Remove-Item
      }
@ -807,20 +792,13 @@ macOS14_Xcode15_arm64:
    - |
      if (Test-Path env:AWS_ACCESS_KEY_ID) {
        aws --endpoint-url https://rgw.ctrl-c.liu.se s3 cp "..\..\$(Make-SafeFileName("OpenMW_MSVC2022_64_${config}_${CI_COMMIT_REF_NAME}.zip"))" s3://openmw-artifacts/${artifactDirectory}
-        if(!$?) { Exit $LASTEXITCODE }
-      }
-    - |
-      if ($executables) {
-        foreach ($exe in $executables.Split(',')) {
-          & .\$exe
-          if(!$?) { Exit $LASTEXITCODE }
-        }
      }
+    - if ($executables) { foreach ($exe in $executables.Split(',')) { & .\$exe } }
  after_script:
    - Get-Volume
    - Copy-Item C:\ProgramData\chocolatey\logs\chocolatey.log
  cache:
-    key: msbuild-2022-v12
+    key: msbuild-2022-v11
    paths:
    - deps
    - MSVC2022_64/deps/Qt
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@ -229,9 +229,6 @@
    Bug #8299: Crash while smoothing landscape
    Bug #8364: Crash when clicking scrollbar without handle (divide by zero)
    Bug #8378: Korean bitmap fonts are unusable
-    Bug #8439: Creatures without models can crash the game
-    Bug #8441: Freeze when using video main menu replacers
-    Bug #8462: Crashes when resizing the window on macOS
    Feature #1415: Infinite fall failsafe
    Feature #2566: Handle NAM9 records for manual cell references
    Feature #3501: OpenMW-CS: Instance Editing - Shortcuts for axial locking
--- a/CI/before_script.linux.sh
+++ b/CI/before_script.linux.sh
@ -38,7 +38,7 @@ fi

 if [[ $CI_CLANG_TIDY ]]; then
    CMAKE_CONF_OPTS+=(
-        -DCMAKE_CXX_CLANG_TIDY=clang-tidy
+        -DCMAKE_CXX_CLANG_TIDY="clang-tidy;--warnings-as-errors=*"
        -DBUILD_COMPONENTS_TESTS=ON
        -DBUILD_OPENMW_TESTS=ON
        -DBUILD_OPENCS_TESTS=ON
--- a/CI/install_debian_deps.sh
+++ b/CI/install_debian_deps.sh
@ -95,7 +95,7 @@ declare -rA GROUPED_DEPS=(
  [libasan6]="libasan6"

  [android]="binutils build-essential cmake ccache curl unzip git pkg-config"
-
+  
  [openmw-clang-format]="
    clang-format-14
    git-core
@ -126,24 +126,10 @@ export APT_CACHE_DIR="${PWD}/apt-cache"
 export DEBIAN_FRONTEND=noninteractive
 set -x
 mkdir -pv "$APT_CACHE_DIR"
-
-while true; do
-  apt-get update -yqq && break
-done
-
+apt-get update -yqq
 apt-get -qq -o dir::cache::archives="$APT_CACHE_DIR" install -y --no-install-recommends software-properties-common gnupg >/dev/null
-
-while true; do
-  add-apt-repository -y ppa:openmw/openmw && break
-done
-
-while true; do
-  add-apt-repository -y ppa:openmw/openmw-daily && break
-done
-
-while true; do
-  add-apt-repository -y ppa:openmw/staging && break
-done
-
+add-apt-repository -y ppa:openmw/openmw
+add-apt-repository -y ppa:openmw/openmw-daily
+add-apt-repository -y ppa:openmw/staging
 apt-get -qq -o dir::cache::archives="$APT_CACHE_DIR" install -y --no-install-recommends "${deps[@]}" >/dev/null
 apt list --installed
--- a/CI/run_integration_tests.sh
+++ b/CI/run_integration_tests.sh
@ -9,7 +9,7 @@ git checkout FETCH_HEAD
 cd ..

 xvfb-run --auto-servernum --server-args='-screen 0 640x480x24x60' \
-    scripts/integration_tests.py --verbose --omw build/install/bin/openmw --workdir integration_tests_output example-suite/
+    scripts/integration_tests.py --omw build/install/bin/openmw --workdir integration_tests_output example-suite/

 ls integration_tests_output/*.osg_stats.log | while read v; do
    scripts/osg_stats.py --stats '.*' --regexp_match < "${v}"
--- a/CMakeLists.txt
+++ b/CMakeLists.txt
@ -82,7 +82,7 @@ message(STATUS "Configuring OpenMW...")
 set(OPENMW_VERSION_MAJOR 0)
 set(OPENMW_VERSION_MINOR 49)
 set(OPENMW_VERSION_RELEASE 0)
-set(OPENMW_LUA_API_REVISION 72)
+set(OPENMW_LUA_API_REVISION 71)
 set(OPENMW_POSTPROCESSING_API_REVISION 2)

 set(OPENMW_VERSION_COMMITHASH "")
@ -860,6 +860,7 @@ if (OPENMW_OSX_DEPLOYMENT AND APPLE)
        set(BU_CHMOD_BUNDLE_ITEMS ON)
        set(CMAKE_MODULE_PATH ${CMAKE_MODULE_PATH})
        include(BundleUtilities)
+        cmake_minimum_required(VERSION 3.1)
    " COMPONENT Runtime)

    set(ABSOLUTE_PLUGINS "")
--- a/apps/benchmarks/detournavigator/navmeshtilescache.cpp
+++ b/apps/benchmarks/detournavigator/navmeshtilescache.cpp
@ -179,7 +179,7 @@ namespace
        generateKeys(std::back_inserter(keys), keys.size() * (100 - hitPercentage) / 100, random);
        std::size_t n = 0;

-        for ([[maybe_unused]] auto _ : state)
+        for (auto _ : state)
        {
            const auto& key = keys[n++ % keys.size()];
            auto result = cache.get(key.mAgentBounds, key.mTilePosition, key.mRecastMesh);
--- a/apps/benchmarks/esm/benchrefid.cpp
+++ b/apps/benchmarks/esm/benchrefid.cpp
@ -104,7 +104,7 @@ namespace
        std::minstd_rand random;
        std::vector<ESM::RefId> refIds = generateStringRefIds(state.range(0), random);
        std::size_t i = 0;
-        for ([[maybe_unused]] auto _ : state)
+        for (auto _ : state)
        {
            benchmark::DoNotOptimize(refIds[i].serialize());
            if (++i >= refIds.size())
@ -118,7 +118,7 @@ namespace
        std::vector<std::string> serializedRefIds
            = generateSerializedStringRefIds(state.range(0), random, [](ESM::RefId v) { return v.serialize(); });
        std::size_t i = 0;
-        for ([[maybe_unused]] auto _ : state)
+        for (auto _ : state)
        {
            benchmark::DoNotOptimize(ESM::RefId::deserialize(serializedRefIds[i]));
            if (++i >= serializedRefIds.size())
@ -131,7 +131,7 @@ namespace
        std::minstd_rand random;
        std::vector<ESM::RefId> refIds = generateStringRefIds(state.range(0), random);
        std::size_t i = 0;
-        for ([[maybe_unused]] auto _ : state)
+        for (auto _ : state)
        {
            benchmark::DoNotOptimize(refIds[i].serializeText());
            if (++i >= refIds.size())
@ -145,7 +145,7 @@ namespace
        std::vector<std::string> serializedRefIds
            = generateSerializedStringRefIds(state.range(0), random, [](ESM::RefId v) { return v.serializeText(); });
        std::size_t i = 0;
-        for ([[maybe_unused]] auto _ : state)
+        for (auto _ : state)
        {
            benchmark::DoNotOptimize(ESM::RefId::deserializeText(serializedRefIds[i]));
            if (++i >= serializedRefIds.size())
@ -158,7 +158,7 @@ namespace
        std::minstd_rand random;
        std::vector<ESM::RefId> refIds = generateGeneratedRefIds(random);
        std::size_t i = 0;
-        for ([[maybe_unused]] auto _ : state)
+        for (auto _ : state)
        {
            benchmark::DoNotOptimize(refIds[i].serializeText());
            if (++i >= refIds.size())
@ -172,7 +172,7 @@ namespace
        std::vector<std::string> serializedRefIds
            = generateSerializedGeneratedRefIds(random, [](ESM::RefId v) { return v.serializeText(); });
        std::size_t i = 0;
-        for ([[maybe_unused]] auto _ : state)
+        for (auto _ : state)
        {
            benchmark::DoNotOptimize(ESM::RefId::deserializeText(serializedRefIds[i]));
            if (++i >= serializedRefIds.size())
@ -185,7 +185,7 @@ namespace
        std::minstd_rand random;
        std::vector<ESM::RefId> refIds = generateIndexRefIds(random);
        std::size_t i = 0;
-        for ([[maybe_unused]] auto _ : state)
+        for (auto _ : state)
        {
            benchmark::DoNotOptimize(refIds[i].serializeText());
            if (++i >= refIds.size())
@ -199,7 +199,7 @@ namespace
        std::vector<std::string> serializedRefIds
            = generateSerializedIndexRefIds(random, [](ESM::RefId v) { return v.serializeText(); });
        std::size_t i = 0;
-        for ([[maybe_unused]] auto _ : state)
+        for (auto _ : state)
        {
            benchmark::DoNotOptimize(ESM::RefId::deserializeText(serializedRefIds[i]));
            if (++i >= serializedRefIds.size())
@ -212,7 +212,7 @@ namespace
        std::minstd_rand random;
        std::vector<ESM::RefId> refIds = generateESM3ExteriorCellRefIds(random);
        std::size_t i = 0;
-        for ([[maybe_unused]] auto _ : state)
+        for (auto _ : state)
        {
            benchmark::DoNotOptimize(refIds[i].serializeText());
            if (++i >= refIds.size())
@ -226,7 +226,7 @@ namespace
        std::vector<std::string> serializedRefIds
            = generateSerializedESM3ExteriorCellRefIds(random, [](ESM::RefId v) { return v.serializeText(); });
        std::size_t i = 0;
-        for ([[maybe_unused]] auto _ : state)
+        for (auto _ : state)
        {
            benchmark::DoNotOptimize(ESM::RefId::deserializeText(serializedRefIds[i]));
            if (++i >= serializedRefIds.size())
--- a/apps/benchmarks/settings/access.cpp
+++ b/apps/benchmarks/settings/access.cpp
@ -9,7 +9,7 @@ namespace
 {
    void settingsManager(benchmark::State& state)
    {
-        for ([[maybe_unused]] auto _ : state)
+        for (auto _ : state)
        {
            benchmark::DoNotOptimize(Settings::Manager::getFloat("sky blending start", "Fog"));
        }
@ -17,7 +17,7 @@ namespace

    void settingsManager2(benchmark::State& state)
    {
-        for ([[maybe_unused]] auto _ : state)
+        for (auto _ : state)
        {
            benchmark::DoNotOptimize(Settings::Manager::getFloat("near clip", "Camera"));
            benchmark::DoNotOptimize(Settings::Manager::getBool("transparent postpass", "Post Processing"));
@ -26,7 +26,7 @@ namespace

    void settingsManager3(benchmark::State& state)
    {
-        for ([[maybe_unused]] auto _ : state)
+        for (auto _ : state)
        {
            benchmark::DoNotOptimize(Settings::Manager::getFloat("near clip", "Camera"));
            benchmark::DoNotOptimize(Settings::Manager::getBool("transparent postpass", "Post Processing"));
@ -36,7 +36,7 @@ namespace

    void localStatic(benchmark::State& state)
    {
-        for ([[maybe_unused]] auto _ : state)
+        for (auto _ : state)
        {
            static float v = Settings::Manager::getFloat("sky blending start", "Fog");
            benchmark::DoNotOptimize(v);
@ -45,7 +45,7 @@ namespace

    void localStatic2(benchmark::State& state)
    {
-        for ([[maybe_unused]] auto _ : state)
+        for (auto _ : state)
        {
            static float v1 = Settings::Manager::getFloat("near clip", "Camera");
            static bool v2 = Settings::Manager::getBool("transparent postpass", "Post Processing");
@ -56,7 +56,7 @@ namespace

    void localStatic3(benchmark::State& state)
    {
-        for ([[maybe_unused]] auto _ : state)
+        for (auto _ : state)
        {
            static float v1 = Settings::Manager::getFloat("near clip", "Camera");
            static bool v2 = Settings::Manager::getBool("transparent postpass", "Post Processing");
@ -69,7 +69,7 @@ namespace

    void settingsStorage(benchmark::State& state)
    {
-        for ([[maybe_unused]] auto _ : state)
+        for (auto _ : state)
        {
            float v = Settings::fog().mSkyBlendingStart.get();
            benchmark::DoNotOptimize(v);
@ -78,7 +78,7 @@ namespace

    void settingsStorage2(benchmark::State& state)
    {
-        for ([[maybe_unused]] auto _ : state)
+        for (auto _ : state)
        {
            bool v1 = Settings::postProcessing().mTransparentPostpass.get();
            float v2 = Settings::camera().mNearClip.get();
@ -89,7 +89,7 @@ namespace

    void settingsStorage3(benchmark::State& state)
    {
-        for ([[maybe_unused]] auto _ : state)
+        for (auto _ : state)
        {
            bool v1 = Settings::postProcessing().mTransparentPostpass.get();
            float v2 = Settings::camera().mNearClip.get();
@ -102,7 +102,7 @@ namespace

    void settingsStorageGet(benchmark::State& state)
    {
-        for ([[maybe_unused]] auto _ : state)
+        for (auto _ : state)
        {
            benchmark::DoNotOptimize(Settings::get<float>("Fog", "sky blending start"));
        }
@ -110,7 +110,7 @@ namespace

    void settingsStorageGet2(benchmark::State& state)
    {
-        for ([[maybe_unused]] auto _ : state)
+        for (auto _ : state)
        {
            benchmark::DoNotOptimize(Settings::get<bool>("Post Processing", "transparent postpass"));
            benchmark::DoNotOptimize(Settings::get<float>("Camera", "near clip"));
@ -119,7 +119,7 @@ namespace

    void settingsStorageGet3(benchmark::State& state)
    {
-        for ([[maybe_unused]] auto _ : state)
+        for (auto _ : state)
        {
            benchmark::DoNotOptimize(Settings::get<bool>("Post Processing", "transparent postpass"));
            benchmark::DoNotOptimize(Settings::get<float>("Camera", "near clip"));
--- a/apps/bulletobjecttool/main.cpp
+++ b/apps/bulletobjecttool/main.cpp
@ -53,6 +53,8 @@ namespace

    bpo::options_description makeOptionsDescription()
    {
+        using Fallback::FallbackMap;
+
        bpo::options_description result;
        auto addOption = result.add_options();
        addOption("help", "print help message");
@ -85,8 +87,7 @@ namespace
            "\n\twin1251 - Cyrillic alphabet such as Russian, Bulgarian, Serbian Cyrillic and other languages\n"
            "\n\twin1252 - Western European (Latin) alphabet, used by default");

-        addOption("fallback",
-            bpo::value<Fallback::FallbackMap>()->default_value(Fallback::FallbackMap(), "")->multitoken()->composing(),
+        addOption("fallback", bpo::value<FallbackMap>()->default_value(FallbackMap(), "")->multitoken()->composing(),
            "fallback values");

        Files::ConfigurationManager::addCommonOptions(result);
--- a/apps/components_tests/detournavigator/asyncnavmeshupdater.cpp
+++ b/apps/components_tests/detournavigator/asyncnavmeshupdater.cpp
@ -5,9 +5,7 @@
 #include <components/detournavigator/makenavmesh.hpp>
 #include <components/detournavigator/navmeshdbutils.hpp>
 #include <components/detournavigator/serialization.hpp>
-#include <components/files/conversion.hpp>
 #include <components/loadinglistener/loadinglistener.hpp>
-#include <components/testing/util.hpp>

 #include <BulletCollision/CollisionShapes/btBoxShape.h>

@ -374,106 +372,6 @@ namespace
        }
    }

-    TEST_F(DetourNavigatorAsyncNavMeshUpdaterTest, should_write_debug_recast_mesh)
-    {
-        mRecastMeshManager.setWorldspace(mWorldspace, nullptr);
-        addHeightFieldPlane(mRecastMeshManager);
-        mSettings.mEnableWriteRecastMeshToFile = true;
-        const std::filesystem::path dir = TestingOpenMW::outputDirPath("DetourNavigatorAsyncNavMeshUpdaterTest");
-        mSettings.mRecastMeshPathPrefix = Files::pathToUnicodeString(dir) + "/";
-        Log(Debug::Verbose) << mSettings.mRecastMeshPathPrefix;
-        AsyncNavMeshUpdater updater(mSettings, mRecastMeshManager, mOffMeshConnectionsManager, nullptr);
-        const auto navMeshCacheItem = std::make_shared<GuardedNavMeshCacheItem>(1, mSettings);
-        const std::map<TilePosition, ChangeType> changedTiles{ { TilePosition{ 0, 0 }, ChangeType::add } };
-        updater.post(mAgentBounds, navMeshCacheItem, mPlayerTile, mWorldspace, changedTiles);
-        updater.wait(WaitConditionType::allJobsDone, &mListener);
-        EXPECT_TRUE(std::filesystem::exists(dir / "0.0.recastmesh.obj"));
-    }
-
-    TEST_F(DetourNavigatorAsyncNavMeshUpdaterTest, should_write_debug_recast_mesh_with_revision)
-    {
-        mRecastMeshManager.setWorldspace(mWorldspace, nullptr);
-        addHeightFieldPlane(mRecastMeshManager);
-        mSettings.mEnableWriteRecastMeshToFile = true;
-        mSettings.mEnableRecastMeshFileNameRevision = true;
-        const std::filesystem::path dir = TestingOpenMW::outputDirPath("DetourNavigatorAsyncNavMeshUpdaterTest");
-        mSettings.mRecastMeshPathPrefix = Files::pathToUnicodeString(dir) + "/";
-        Log(Debug::Verbose) << mSettings.mRecastMeshPathPrefix;
-        AsyncNavMeshUpdater updater(mSettings, mRecastMeshManager, mOffMeshConnectionsManager, nullptr);
-        const auto navMeshCacheItem = std::make_shared<GuardedNavMeshCacheItem>(1, mSettings);
-        const std::map<TilePosition, ChangeType> changedTiles{ { TilePosition{ 0, 0 }, ChangeType::add } };
-        updater.post(mAgentBounds, navMeshCacheItem, mPlayerTile, mWorldspace, changedTiles);
-        updater.wait(WaitConditionType::allJobsDone, &mListener);
-        EXPECT_TRUE(std::filesystem::exists(dir / "0.0.recastmesh.1.2.obj"));
-    }
-
-    TEST_F(DetourNavigatorAsyncNavMeshUpdaterTest, writing_recast_mesh_to_absent_file_should_not_fail_tile_generation)
-    {
-        mRecastMeshManager.setWorldspace(mWorldspace, nullptr);
-        addHeightFieldPlane(mRecastMeshManager);
-        mSettings.mEnableWriteRecastMeshToFile = true;
-        const std::filesystem::path dir = TestingOpenMW::outputDir() / "absent";
-        mSettings.mRecastMeshPathPrefix = Files::pathToUnicodeString(dir) + "/";
-        Log(Debug::Verbose) << mSettings.mRecastMeshPathPrefix;
-        AsyncNavMeshUpdater updater(mSettings, mRecastMeshManager, mOffMeshConnectionsManager, nullptr);
-        const auto navMeshCacheItem = std::make_shared<GuardedNavMeshCacheItem>(1, mSettings);
-        const std::map<TilePosition, ChangeType> changedTiles{ { TilePosition{ 0, 0 }, ChangeType::add } };
-        updater.post(mAgentBounds, navMeshCacheItem, mPlayerTile, mWorldspace, changedTiles);
-        updater.wait(WaitConditionType::allJobsDone, &mListener);
-        EXPECT_NE(navMeshCacheItem->lockConst()->getImpl().getTileRefAt(0, 0, 0), 0u);
-        EXPECT_FALSE(std::filesystem::exists(dir));
-    }
-
-    TEST_F(DetourNavigatorAsyncNavMeshUpdaterTest, should_write_debug_navmesh)
-    {
-        mRecastMeshManager.setWorldspace(mWorldspace, nullptr);
-        addHeightFieldPlane(mRecastMeshManager);
-        mSettings.mEnableWriteNavMeshToFile = true;
-        const std::filesystem::path dir = TestingOpenMW::outputDirPath("DetourNavigatorAsyncNavMeshUpdaterTest");
-        mSettings.mNavMeshPathPrefix = Files::pathToUnicodeString(dir) + "/";
-        Log(Debug::Verbose) << mSettings.mRecastMeshPathPrefix;
-        AsyncNavMeshUpdater updater(mSettings, mRecastMeshManager, mOffMeshConnectionsManager, nullptr);
-        const auto navMeshCacheItem = std::make_shared<GuardedNavMeshCacheItem>(1, mSettings);
-        const std::map<TilePosition, ChangeType> changedTiles{ { TilePosition{ 0, 0 }, ChangeType::add } };
-        updater.post(mAgentBounds, navMeshCacheItem, mPlayerTile, mWorldspace, changedTiles);
-        updater.wait(WaitConditionType::allJobsDone, &mListener);
-        EXPECT_TRUE(std::filesystem::exists(dir / "all_tiles_navmesh.bin"));
-    }
-
-    TEST_F(DetourNavigatorAsyncNavMeshUpdaterTest, should_write_debug_navmesh_with_revision)
-    {
-        mRecastMeshManager.setWorldspace(mWorldspace, nullptr);
-        addHeightFieldPlane(mRecastMeshManager);
-        mSettings.mEnableWriteNavMeshToFile = true;
-        mSettings.mEnableNavMeshFileNameRevision = true;
-        const std::filesystem::path dir = TestingOpenMW::outputDirPath("DetourNavigatorAsyncNavMeshUpdaterTest");
-        mSettings.mNavMeshPathPrefix = Files::pathToUnicodeString(dir) + "/";
-        Log(Debug::Verbose) << mSettings.mRecastMeshPathPrefix;
-        AsyncNavMeshUpdater updater(mSettings, mRecastMeshManager, mOffMeshConnectionsManager, nullptr);
-        const auto navMeshCacheItem = std::make_shared<GuardedNavMeshCacheItem>(1, mSettings);
-        const std::map<TilePosition, ChangeType> changedTiles{ { TilePosition{ 0, 0 }, ChangeType::add } };
-        updater.post(mAgentBounds, navMeshCacheItem, mPlayerTile, mWorldspace, changedTiles);
-        updater.wait(WaitConditionType::allJobsDone, &mListener);
-        EXPECT_TRUE(std::filesystem::exists(dir / "all_tiles_navmesh.1.1.bin"));
-    }
-
-    TEST_F(DetourNavigatorAsyncNavMeshUpdaterTest, writing_navmesh_to_absent_file_should_not_fail_tile_generation)
-    {
-        mRecastMeshManager.setWorldspace(mWorldspace, nullptr);
-        addHeightFieldPlane(mRecastMeshManager);
-        mSettings.mEnableWriteNavMeshToFile = true;
-        const std::filesystem::path dir = TestingOpenMW::outputDir() / "absent";
-        mSettings.mNavMeshPathPrefix = Files::pathToUnicodeString(dir) + "/";
-        Log(Debug::Verbose) << mSettings.mRecastMeshPathPrefix;
-        AsyncNavMeshUpdater updater(mSettings, mRecastMeshManager, mOffMeshConnectionsManager, nullptr);
-        const auto navMeshCacheItem = std::make_shared<GuardedNavMeshCacheItem>(1, mSettings);
-        const std::map<TilePosition, ChangeType> changedTiles{ { TilePosition{ 0, 0 }, ChangeType::add } };
-        updater.post(mAgentBounds, navMeshCacheItem, mPlayerTile, mWorldspace, changedTiles);
-        updater.wait(WaitConditionType::allJobsDone, &mListener);
-        EXPECT_NE(navMeshCacheItem->lockConst()->getImpl().getTileRefAt(0, 0, 0), 0u);
-        EXPECT_FALSE(std::filesystem::exists(dir));
-    }
-
    struct DetourNavigatorSpatialJobQueueTest : Test
    {
        const AgentBounds mAgentBounds{ CollisionShapeType::Aabb, osg::Vec3f(1, 1, 1) };
--- a/apps/components_tests/esm3/readerscache.cpp
+++ b/apps/components_tests/esm3/readerscache.cpp
@ -88,31 +88,4 @@ namespace
            EXPECT_EQ(reader->getFileOffset(), sInitialOffset);
        }
    }
-
-    TEST_F(ESM3ReadersCacheWithContentFile, CachedSizeAndName)
-    {
-        ESM::ReadersCache readers(2);
-        {
-            readers.get(0)->openRaw(std::make_unique<std::istringstream>("123"), "closed0.omwaddon");
-            readers.get(1)->openRaw(std::make_unique<std::istringstream>("12345"), "closed1.omwaddon");
-            readers.get(2)->openRaw(std::make_unique<std::istringstream>("1234567"), "free.omwaddon");
-        }
-        auto busy = readers.get(3);
-        busy->openRaw(std::make_unique<std::istringstream>("123456789"), "busy.omwaddon");
-
-        EXPECT_EQ(readers.getFileSize(0), 3);
-        EXPECT_EQ(readers.getName(0), "closed0.omwaddon");
-
-        EXPECT_EQ(readers.getFileSize(1), 5);
-        EXPECT_EQ(readers.getName(1), "closed1.omwaddon");
-
-        EXPECT_EQ(readers.getFileSize(2), 7);
-        EXPECT_EQ(readers.getName(2), "free.omwaddon");
-
-        EXPECT_EQ(readers.getFileSize(3), 9);
-        EXPECT_EQ(readers.getName(3), "busy.omwaddon");
-
-        // not-yet-seen indices give zero for their size
-        EXPECT_EQ(readers.getFileSize(4), 0);
-    }
 }
--- a/apps/components_tests/files/hash.cpp
+++ b/apps/components_tests/files/hash.cpp
@ -30,7 +30,7 @@ namespace

    TEST(FilesGetHash, shouldClearErrors)
    {
-        const auto fileName = outputFilePath("fileName");
+        const auto fileName = temporaryFilePath("fileName");
        std::string content;
        std::fill_n(std::back_inserter(content), 1, 'a');
        std::istringstream stream(content);
@ -41,7 +41,7 @@ namespace

    TEST_P(FilesGetHash, shouldReturnHashForStringStream)
    {
-        const auto fileName = outputFilePath("fileName");
+        const auto fileName = temporaryFilePath("fileName");
        std::string content;
        std::fill_n(std::back_inserter(content), GetParam().mSize, 'a');
        std::istringstream stream(content);
--- a/apps/components_tests/main.cpp
+++ b/apps/components_tests/main.cpp
@ -2,7 +2,6 @@
 #include <components/misc/strings/conversion.hpp>
 #include <components/settings/parser.hpp>
 #include <components/settings/values.hpp>
-#include <components/testing/util.hpp>

 #include <gtest/gtest.h>

@ -25,9 +24,5 @@ int main(int argc, char** argv)
    Settings::StaticValues::init();

    testing::InitGoogleTest(&argc, argv);
-
-    const int result = RUN_ALL_TESTS();
-    if (result == 0)
-        std::filesystem::remove_all(TestingOpenMW::outputDir());
-    return result;
+    return RUN_ALL_TESTS();
 }
--- a/apps/components_tests/shader/shadermanager.cpp
+++ b/apps/components_tests/shader/shadermanager.cpp
@ -16,7 +16,7 @@ namespace
        ShaderManager mManager;
        ShaderManager::DefineMap mDefines;

-        ShaderManagerTest() { mManager.setShaderPath(TestingOpenMW::outputDir()); }
+        ShaderManagerTest() { mManager.setShaderPath("tests_output"); }

        template <class F>
        void withShaderFile(const std::string& content, F&& f)
--- a/apps/launcher/datafilespage.cpp
+++ b/apps/launcher/datafilespage.cpp
@ -8,7 +8,6 @@
 #include <QList>
 #include <QMessageBox>
 #include <QPair>
-#include <QProgressDialog>
 #include <QPushButton>

 #include <algorithm>
@ -352,17 +351,9 @@ void Launcher::DataFilesPage::populateFileViews(const QString& contentModelName)
    if (!resourcesVfs.isEmpty())
        directories.insert(0, { resourcesVfs });

-    QIcon containsDataIcon(":/images/openmw-plugin.png");
-
-    QProgressDialog progressBar("Adding data directories", {}, 0, directories.count(), this);
-    progressBar.setWindowModality(Qt::WindowModal);
-    progressBar.setValue(0);
-
    std::unordered_set<QString> visitedDirectories;
    for (const Config::SettingValue& currentDir : directories)
    {
-        progressBar.setValue(progressBar.value() + 1);
-
        if (!visitedDirectories.insert(currentDir.value).second)
            continue;

@ -411,7 +402,7 @@ void Launcher::DataFilesPage::populateFileViews(const QString& contentModelName)
        // Add a "data file" icon if the directory contains a content file
        if (mSelector->containsDataFiles(currentDir.value))
        {
-            item->setIcon(containsDataIcon);
+            item->setIcon(QIcon(":/images/openmw-plugin.png"));

            tooltip << tr("Contains content file(s)");
        }
@ -774,7 +765,7 @@ void Launcher::DataFilesPage::addSubdirectories(bool append)
        return;

    QString rootPath = QFileDialog::getExistingDirectory(
-        this, tr("Select Directory"), {}, QFileDialog::ShowDirsOnly | QFileDialog::Option::ReadOnly);
+        this, tr("Select Directory"), QDir::homePath(), QFileDialog::ShowDirsOnly | QFileDialog::Option::ReadOnly);

    if (rootPath.isEmpty())
        return;
--- a/apps/navmeshtool/CMakeLists.txt
+++ b/apps/navmeshtool/CMakeLists.txt
@ -1,16 +1,20 @@
-set(NAVMESHTOOL_LIB
+set(NAVMESHTOOL
    worldspacedata.cpp
    navmesh.cpp
+    main.cpp
+)
+source_group(apps\\navmeshtool FILES ${NAVMESHTOOL})
+
+add_library(openmw-navmeshtool-lib STATIC
+    ${NAVMESHTOOL}
 )

-source_group(apps\\navmeshtool FILES ${NAVMESHTOOL_LIB} main.cpp)
-
-add_library(openmw-navmeshtool-lib STATIC ${NAVMESHTOOL_LIB})
-
 if (ANDROID)
-    add_library(openmw-navmeshtool SHARED main.cpp)
+    add_library(openmw-navmeshtool SHARED
+        main.cpp
+    )
 else()
-    openmw_add_executable(openmw-navmeshtool main.cpp)
+    openmw_add_executable(openmw-navmeshtool ${NAVMESHTOOL})
 endif()

 target_link_libraries(openmw-navmeshtool openmw-navmeshtool-lib)
--- a/apps/navmeshtool/main.cpp
+++ b/apps/navmeshtool/main.cpp
@ -62,6 +62,8 @@ namespace NavMeshTool

        bpo::options_description makeOptionsDescription()
        {
+            using Fallback::FallbackMap;
+
            bpo::options_description result;
            auto addOption = result.add_options();
            addOption("help", "print help message");
@ -223,8 +225,7 @@ namespace NavMeshTool
            Resource::SceneManager sceneManager(&vfs, &imageManager, &nifFileManager, &bgsmFileManager, expiryDelay);
            Resource::BulletShapeManager bulletShapeManager(&vfs, &sceneManager, &nifFileManager, expiryDelay);
            DetourNavigator::RecastGlobalAllocator::init();
-            DetourNavigator::Settings navigatorSettings
-                = DetourNavigator::makeSettingsFromSettingsManager(Debug::getRecastMaxLogLevel());
+            DetourNavigator::Settings navigatorSettings = DetourNavigator::makeSettingsFromSettingsManager();
            navigatorSettings.mRecast.mSwimHeightScale
                = EsmLoader::getGameSetting(esmData.mGameSettings, "fSwimHeightScale").getFloat();

--- a/apps/opencs/editor.cpp
+++ b/apps/opencs/editor.cpp
@ -38,6 +38,8 @@

 #include "view/doc/viewmanager.hpp"

+using namespace Fallback;
+
 CS::Editor::Editor(int argc, char** argv)
    : mConfigVariables(readConfiguration())
    , mSettingsState(mCfgMgr)
@ -122,10 +124,7 @@ boost::program_options::variables_map CS::Editor::readConfiguration()
            ->default_value(std::vector<std::string>(), "fallback-archive")
            ->multitoken());
    addOption("fallback",
-        boost::program_options::value<Fallback::FallbackMap>()
-            ->default_value(Fallback::FallbackMap(), "")
-            ->multitoken()
-            ->composing(),
+        boost::program_options::value<FallbackMap>()->default_value(FallbackMap(), "")->multitoken()->composing(),
        "fallback values");
    Files::ConfigurationManager::addCommonOptions(desc);

@ -142,7 +141,7 @@ std::pair<Files::PathContainer, std::vector<std::string>> CS::Editor::readConfig
 {
    boost::program_options::variables_map& variables = mConfigVariables;

-    Fallback::Map::init(variables["fallback"].as<Fallback::FallbackMap>().mMap);
+    Fallback::Map::init(variables["fallback"].as<FallbackMap>().mMap);

    mEncodingName = variables["encoding"].as<std::string>();
    mDocumentManager.setEncoding(ToUTF8::calculateEncoding(mEncodingName));
--- a/apps/opencs_tests/main.cpp
+++ b/apps/opencs_tests/main.cpp
@ -1,5 +1,4 @@
 #include <components/debug/debugging.hpp>
-#include <components/testing/util.hpp>

 #include <gtest/gtest.h>

@ -8,9 +7,5 @@ int main(int argc, char* argv[])
    Log::sMinDebugLevel = Debug::getDebugLevel();

    testing::InitGoogleTest(&argc, argv);
-
-    const int result = RUN_ALL_TESTS();
-    if (result == 0)
-        std::filesystem::remove_all(TestingOpenMW::outputDir());
-    return result;
+    return RUN_ALL_TESTS();
 }
--- a/apps/openmw/engine.cpp
+++ b/apps/openmw/engine.cpp
@ -373,15 +373,11 @@ OMW::Engine::Engine(Files::ConfigurationManager& configurationManager)
    , mScriptConsoleMode(false)
    , mActivationDistanceOverride(-1)
    , mGrab(true)
-    , mExportFonts(false)
    , mRandomSeed(0)
    , mNewGame(false)
    , mCfgMgr(configurationManager)
    , mGlMaxTextureImageUnits(0)
 {
-#if SDL_VERSION_ATLEAST(2, 24, 0)
-    SDL_SetHint(SDL_HINT_MAC_OPENGL_ASYNC_DISPATCH, "1");
-#endif
    SDL_SetHint(SDL_HINT_ACCELEROMETER_AS_JOYSTICK, "0"); // We use only gamepads

    Uint32 flags
@ -811,7 +807,7 @@ void OMW::Engine::prepareEngine()
    rootNode->addChild(guiRoot);

    mWindowManager = std::make_unique<MWGui::WindowManager>(mWindow, mViewer, guiRoot, mResourceSystem.get(),
-        mWorkQueue.get(), mCfgMgr.getLogPath(), mScriptConsoleMode, mTranslationDataStorage, mEncoding, mExportFonts,
+        mWorkQueue.get(), mCfgMgr.getLogPath(), mScriptConsoleMode, mTranslationDataStorage, mEncoding,
        Version::getOpenmwVersionDescription(), shadersSupported, mCfgMgr);
    mEnvironment.setWindowManager(*mWindowManager);

@ -851,7 +847,7 @@ void OMW::Engine::prepareEngine()
    }
    listener->loadingOff();

-    mWorld->init(mMaxRecastLogLevel, mViewer, std::move(rootNode), mWorkQueue.get(), *mUnrefQueue);
+    mWorld->init(mViewer, std::move(rootNode), mWorkQueue.get(), *mUnrefQueue);
    mEnvironment.setWorldScene(mWorld->getWorldScene());
    mWorld->setupPlayer();
    mWorld->setRandomSeed(mRandomSeed);
@ -1113,11 +1109,6 @@ void OMW::Engine::setWarningsMode(int mode)
    mWarningsMode = mode;
 }

-void OMW::Engine::enableFontExport(bool exportFonts)
-{
-    mExportFonts = exportFonts;
-}
-
 void OMW::Engine::setSaveGameFile(const std::filesystem::path& savegame)
 {
    mSaveGameFile = savegame;
--- a/apps/openmw/engine.hpp
+++ b/apps/openmw/engine.hpp
@ -4,7 +4,6 @@
 #include <filesystem>

 #include <components/compiler/extensions.hpp>
-#include <components/debug/debuglog.hpp>
 #include <components/esm/refid.hpp>
 #include <components/files/collections.hpp>
 #include <components/settings/settings.hpp>
@ -172,9 +171,7 @@ namespace OMW
        // Grab mouse?
        bool mGrab;

-        bool mExportFonts;
        unsigned int mRandomSeed;
-        Debug::Level mMaxRecastLogLevel = Debug::Error;

        Compiler::Extensions mExtensions;
        std::unique_ptr<Compiler::Context> mScriptContext;
@ -183,9 +180,6 @@ namespace OMW
        Translation::Storage mTranslationDataStorage;
        bool mNewGame;

-        Files::ConfigurationManager& mCfgMgr;
-        int mGlMaxTextureImageUnits;
-
        // not implemented
        Engine(const Engine&);
        Engine& operator=(const Engine&);
@ -257,14 +251,14 @@ namespace OMW

        void setWarningsMode(int mode);

-        void enableFontExport(bool exportFonts);
-
        /// Set the save game file to load after initialising the engine.
        void setSaveGameFile(const std::filesystem::path& savegame);

        void setRandomSeed(unsigned int seed);

-        void setRecastMaxLogLevel(Debug::Level value) { mMaxRecastLogLevel = value; }
+    private:
+        Files::ConfigurationManager& mCfgMgr;
+        int mGlMaxTextureImageUnits;
    };
 }

--- a/apps/openmw/main.cpp
+++ b/apps/openmw/main.cpp
@ -28,6 +28,8 @@ extern "C" __declspec(dllexport) DWORD AmdPowerXpressRequestHighPerformance = 0x
 #include <unistd.h>
 #endif

+using namespace Fallback;
+
 /**
 * \brief Parses application command line and calls \ref Cfg::ConfigurationManager
 * to parse configuration files.
@ -150,10 +152,9 @@ bool parseOptions(int argc, char** argv, OMW::Engine& engine, Files::Configurati
    engine.setSaveGameFile(variables["load-savegame"].as<Files::MaybeQuotedPath>().u8string());

    // other settings
-    Fallback::Map::init(variables["fallback"].as<Fallback::FallbackMap>().mMap);
+    Fallback::Map::init(variables["fallback"].as<FallbackMap>().mMap);
    engine.setSoundUsage(!variables["no-sound"].as<bool>());
    engine.setActivationDistanceOverride(variables["activate-dist"].as<int>());
-    engine.enableFontExport(variables["export-fonts"].as<bool>());
    engine.setRandomSeed(variables["random-seed"].as<unsigned int>());

    return true;
@ -219,8 +220,6 @@ int runApplication(int argc, char* argv[])
    Files::ConfigurationManager cfgMgr;
    std::unique_ptr<OMW::Engine> engine = std::make_unique<OMW::Engine>(cfgMgr);

-    engine->setRecastMaxLogLevel(Debug::getRecastMaxLogLevel());
-
    if (parseOptions(argc, argv, *engine, cfgMgr))
    {
        if (!Misc::checkRequiredOSGPluginsArePresent())
--- a/apps/openmw/mwbase/mechanicsmanager.hpp
+++ b/apps/openmw/mwbase/mechanicsmanager.hpp
@ -200,7 +200,7 @@ namespace MWBase
        ///< Skip the animation for the given MW-reference for one frame. Calls to this function for
        /// references that are currently not in the scene should be ignored.

-        virtual bool checkAnimationPlaying(const MWWorld::Ptr& ptr, std::string_view groupName) = 0;
+        virtual bool checkAnimationPlaying(const MWWorld::Ptr& ptr, const std::string& groupName) = 0;

        virtual bool checkScriptedAnimationPlaying(const MWWorld::Ptr& ptr) const = 0;

--- a/apps/openmw/mwbase/windowmanager.hpp
+++ b/apps/openmw/mwbase/windowmanager.hpp
@ -363,7 +363,7 @@ namespace MWBase
        void windowVisibilityChange(bool visible) override = 0;
        void windowResized(int x, int y) override = 0;
        void windowClosed() override = 0;
-        virtual bool isWindowVisible() const = 0;
+        virtual bool isWindowVisible() = 0;

        virtual void watchActor(const MWWorld::Ptr& ptr) = 0;
        virtual MWWorld::Ptr getWatchedActor() const = 0;
--- a/apps/openmw/mwclass/container.cpp
+++ b/apps/openmw/mwclass/container.cpp
@ -237,12 +237,7 @@ namespace MWClass
    bool Container::hasToolTip(const MWWorld::ConstPtr& ptr) const
    {
        if (const MWWorld::CustomData* data = ptr.getRefData().getCustomData())
-        {
-            if (!canBeHarvested(ptr))
-                return true;
-            const MWWorld::ContainerStore& store = data->asContainerCustomData().mStore;
-            return !store.isResolved() || store.hasVisibleItems();
-        }
+            return !canBeHarvested(ptr) || data->asContainerCustomData().mStore.hasVisibleItems();
        return true;
    }

--- a/apps/openmw/mwgui/bookpage.cpp
+++ b/apps/openmw/mwgui/bookpage.cpp
@ -1343,13 +1343,12 @@ namespace MWGui
        return codePoint == '\r';
    }

-    // Normal no-break space (0x00A0) is ignored here
-    // because Morrowind compatibility requires us to render its glyph
    static bool ucsSpace(int codePoint)
    {
        switch (codePoint)
        {
            case 0x0020: // SPACE
+            case 0x00A0: // NO-BREAK SPACE
            case 0x1680: // OGHAM SPACE MARK
            case 0x180E: // MONGOLIAN VOWEL SEPARATOR
            case 0x2000: // EN QUAD
@ -1374,14 +1373,12 @@ namespace MWGui
        }
    }

-    // No-break spaces (0x00A0, 0x202F, 0xFEFF - normal, narrow, zero width)
-    // are ignored here for obvious reasons
-    // Figure space (0x2007) is not a breaking space either
    static bool ucsBreakingSpace(int codePoint)
    {
        switch (codePoint)
        {
            case 0x0020: // SPACE
+                         // case 0x00A0: // NO-BREAK SPACE
            case 0x1680: // OGHAM SPACE MARK
            case 0x180E: // MONGOLIAN VOWEL SEPARATOR
            case 0x2000: // EN QUAD
@ -1391,12 +1388,15 @@ namespace MWGui
            case 0x2004: // THREE-PER-EM SPACE
            case 0x2005: // FOUR-PER-EM SPACE
            case 0x2006: // SIX-PER-EM SPACE
+            case 0x2007: // FIGURE SPACE
            case 0x2008: // PUNCTUATION SPACE
            case 0x2009: // THIN SPACE
            case 0x200A: // HAIR SPACE
            case 0x200B: // ZERO WIDTH SPACE
+            case 0x202F: // NARROW NO-BREAK SPACE
            case 0x205F: // MEDIUM MATHEMATICAL SPACE
            case 0x3000: // IDEOGRAPHIC SPACE
+                // case 0xFEFF: // ZERO WIDTH NO-BREAK SPACE
                return true;
            default:
                return false;
--- a/apps/openmw/mwgui/mainmenu.cpp
+++ b/apps/openmw/mwgui/mainmenu.cpp
@ -29,26 +29,11 @@ namespace MWGui
    {
        Misc::FrameRateLimiter frameRateLimiter
            = Misc::makeFrameRateLimiter(MWBase::Environment::get().getFrameRateLimit());
-        const MWBase::WindowManager& windowManager = *MWBase::Environment::get().getWindowManager();
-        bool paused = false;
        while (mRunning)
        {
-            if (windowManager.isWindowVisible())
-            {
-                if (paused)
-                {
-                    mVideo->resume();
-                    paused = false;
-                }
-                // If finished playing, start again
-                if (!mVideo->update())
-                    mVideo->playVideo("video\\menu_background.bik");
-            }
-            else if (!paused)
-            {
-                paused = true;
-                mVideo->pause();
-            }
+            // If finished playing, start again
+            if (!mVideo->update())
+                mVideo->playVideo("video\\menu_background.bik");
            frameRateLimiter.limit();
        }
    }
--- a/apps/openmw/mwgui/windowmanagerimp.cpp
+++ b/apps/openmw/mwgui/windowmanagerimp.cpp
@ -146,7 +146,7 @@ namespace MWGui
    WindowManager::WindowManager(SDL_Window* window, osgViewer::Viewer* viewer, osg::Group* guiRoot,
        Resource::ResourceSystem* resourceSystem, SceneUtil::WorkQueue* workQueue, const std::filesystem::path& logpath,
        bool consoleOnlyScripts, Translation::Storage& translationDataStorage, ToUTF8::FromType encoding,
-        bool exportFonts, const std::string& versionDescription, bool useShaders, Files::ConfigurationManager& cfgMgr)
+        const std::string& versionDescription, bool useShaders, Files::ConfigurationManager& cfgMgr)
        : mOldUpdateMask(0)
        , mOldCullMask(0)
        , mStore(nullptr)
@ -215,8 +215,7 @@ namespace MWGui
        MyGUI::LanguageManager::getInstance().eventRequestTag = MyGUI::newDelegate(this, &WindowManager::onRetrieveTag);

        // Load fonts
-        mFontLoader
-            = std::make_unique<Gui::FontLoader>(encoding, resourceSystem->getVFS(), mScalingFactor, exportFonts);
+        mFontLoader = std::make_unique<Gui::FontLoader>(encoding, resourceSystem->getVFS(), mScalingFactor);

        // Register own widgets with MyGUI
        MyGUI::FactoryManager::getInstance().registerFactory<MWGui::Widgets::MWSkill>("Widget");
@ -730,9 +729,6 @@ namespace MWGui
            return;
        }

-        if (mGuiModes.empty())
-            return;
-
        GuiModeState& state = mGuiModeStates[mGuiModes.back()];
        for (const auto& window : state.mWindows)
        {
@ -1218,7 +1214,7 @@ namespace MWGui
        // TODO: check if any windows are now off-screen and move them back if so
    }

-    bool WindowManager::isWindowVisible() const
+    bool WindowManager::isWindowVisible()
    {
        return mWindowVisible;
    }
--- a/apps/openmw/mwgui/windowmanagerimp.hpp
+++ b/apps/openmw/mwgui/windowmanagerimp.hpp
@ -128,7 +128,7 @@ namespace MWGui
        WindowManager(SDL_Window* window, osgViewer::Viewer* viewer, osg::Group* guiRoot,
            Resource::ResourceSystem* resourceSystem, SceneUtil::WorkQueue* workQueue,
            const std::filesystem::path& logpath, bool consoleOnlyScripts, Translation::Storage& translationDataStorage,
-            ToUTF8::FromType encoding, bool exportFonts, const std::string& versionDescription, bool useShaders,
+            ToUTF8::FromType encoding, const std::string& versionDescription, bool useShaders,
            Files::ConfigurationManager& cfgMgr);
        virtual ~WindowManager();

@ -290,7 +290,7 @@ namespace MWGui
        void windowVisibilityChange(bool visible) override;
        void windowResized(int x, int y) override;
        void windowClosed() override;
-        bool isWindowVisible() const override;
+        bool isWindowVisible() override;

        void watchActor(const MWWorld::Ptr& ptr) override;
        MWWorld::Ptr getWatchedActor() const override;
--- a/apps/openmw/mwlua/nearbybindings.cpp
+++ b/apps/openmw/mwlua/nearbybindings.cpp
@ -213,7 +213,6 @@ namespace MWLua
                    { "NavMeshNotFound", DetourNavigator::Status::NavMeshNotFound },
                    { "StartPolygonNotFound", DetourNavigator::Status::StartPolygonNotFound },
                    { "EndPolygonNotFound", DetourNavigator::Status::EndPolygonNotFound },
-                    { "TargetPolygonNotFound", DetourNavigator::Status::TargetPolygonNotFound },
                    { "MoveAlongSurfaceFailed", DetourNavigator::Status::MoveAlongSurfaceFailed },
                    { "FindPathOverPolygonsFailed", DetourNavigator::Status::FindPathOverPolygonsFailed },
                    { "InitNavMeshQueryFailed", DetourNavigator::Status::InitNavMeshQueryFailed },
--- a/apps/openmw/mwmechanics/activespells.cpp
+++ b/apps/openmw/mwmechanics/activespells.cpp
@ -289,9 +289,8 @@ namespace MWMechanics
                    const ESM::RefId& enchantmentId = slot->getClass().getEnchantment(*slot);
                    if (enchantmentId.empty())
                        continue;
-                    const ESM::Enchantment* enchantment
-                        = world->getStore().get<ESM::Enchantment>().search(enchantmentId);
-                    if (enchantment == nullptr || enchantment->mData.mType != ESM::Enchantment::ConstantEffect)
+                    const ESM::Enchantment* enchantment = world->getStore().get<ESM::Enchantment>().find(enchantmentId);
+                    if (enchantment->mData.mType != ESM::Enchantment::ConstantEffect)
                        continue;
                    if (std::find_if(mSpells.begin(), mSpells.end(),
                            [&](const ActiveSpellParams& params) {
--- a/apps/openmw/mwmechanics/actors.cpp
+++ b/apps/openmw/mwmechanics/actors.cpp
@ -2028,7 +2028,7 @@ namespace MWMechanics
            iter->second->getCharacterController().skipAnim();
    }

-    bool Actors::checkAnimationPlaying(const MWWorld::Ptr& ptr, std::string_view groupName) const
+    bool Actors::checkAnimationPlaying(const MWWorld::Ptr& ptr, const std::string& groupName) const
    {
        const auto iter = mIndex.find(ptr.mRef);
        if (iter != mIndex.end())
--- a/apps/openmw/mwmechanics/actors.hpp
+++ b/apps/openmw/mwmechanics/actors.hpp
@ -119,7 +119,7 @@ namespace MWMechanics
            std::string_view startKey, std::string_view stopKey, bool forceLoop);
        void enableLuaAnimations(const MWWorld::Ptr& ptr, bool enable);
        void skipAnimation(const MWWorld::Ptr& ptr) const;
-        bool checkAnimationPlaying(const MWWorld::Ptr& ptr, std::string_view groupName) const;
+        bool checkAnimationPlaying(const MWWorld::Ptr& ptr, const std::string& groupName) const;
        bool checkScriptedAnimationPlaying(const MWWorld::Ptr& ptr) const;
        void persistAnimationStates() const;
        void clearAnimationQueue(const MWWorld::Ptr& ptr, bool clearScripted);
--- a/apps/openmw/mwmechanics/aiwander.cpp
+++ b/apps/openmw/mwmechanics/aiwander.cpp
@ -40,15 +40,15 @@ namespace MWMechanics

    static const std::size_t MAX_IDLE_SIZE = 8;

-    const std::string_view AiWander::sIdleSelectToGroupName[GroupIndex_MaxIdle - GroupIndex_MinIdle + 1] = {
-        "idle2",
-        "idle3",
-        "idle4",
-        "idle5",
-        "idle6",
-        "idle7",
-        "idle8",
-        "idle9",
+    const std::string AiWander::sIdleSelectToGroupName[GroupIndex_MaxIdle - GroupIndex_MinIdle + 1] = {
+        std::string("idle2"),
+        std::string("idle3"),
+        std::string("idle4"),
+        std::string("idle5"),
+        std::string("idle6"),
+        std::string("idle7"),
+        std::string("idle8"),
+        std::string("idle9"),
    };

    namespace
@ -680,7 +680,7 @@ namespace MWMechanics
    {
        if ((GroupIndex_MinIdle <= idleSelect) && (idleSelect <= GroupIndex_MaxIdle))
        {
-            const std::string_view groupName = sIdleSelectToGroupName[idleSelect - GroupIndex_MinIdle];
+            const std::string& groupName = sIdleSelectToGroupName[idleSelect - GroupIndex_MinIdle];
            return MWBase::Environment::get().getMechanicsManager()->playAnimationGroup(actor, groupName, 0, 1);
        }
        else
@ -695,7 +695,7 @@ namespace MWMechanics
    {
        if ((GroupIndex_MinIdle <= idleSelect) && (idleSelect <= GroupIndex_MaxIdle))
        {
-            const std::string_view groupName = sIdleSelectToGroupName[idleSelect - GroupIndex_MinIdle];
+            const std::string& groupName = sIdleSelectToGroupName[idleSelect - GroupIndex_MinIdle];
            return MWBase::Environment::get().getMechanicsManager()->checkAnimationPlaying(actor, groupName);
        }
        else
--- a/apps/openmw/mwmechanics/aiwander.hpp
+++ b/apps/openmw/mwmechanics/aiwander.hpp
@ -3,7 +3,6 @@

 #include "typedaipackage.hpp"

-#include <string_view>
 #include <vector>

 #include "aitemporarybase.hpp"
@ -182,7 +181,9 @@ namespace MWMechanics
            const ESM::Pathgrid::Point& start, const ESM::Pathgrid::Point& end, AiWanderStorage& storage);

        /// lookup table for converting idleSelect value to groupName
-        static const std::string_view sIdleSelectToGroupName[GroupIndex_MaxIdle - GroupIndex_MinIdle + 1];
+        static const std::string sIdleSelectToGroupName[GroupIndex_MaxIdle - GroupIndex_MinIdle + 1];
+
+        static int OffsetToPreventOvercrowding();
    };
 }

--- a/apps/openmw/mwmechanics/mechanicsmanagerimp.cpp
+++ b/apps/openmw/mwmechanics/mechanicsmanagerimp.cpp
@ -778,8 +778,7 @@ namespace MWMechanics
        else
            mObjects.skipAnimation(ptr);
    }
-
-    bool MechanicsManager::checkAnimationPlaying(const MWWorld::Ptr& ptr, std::string_view groupName)
+    bool MechanicsManager::checkAnimationPlaying(const MWWorld::Ptr& ptr, const std::string& groupName)
    {
        if (ptr.getClass().isActor())
            return mActors.checkAnimationPlaying(ptr, groupName);
--- a/apps/openmw/mwmechanics/mechanicsmanagerimp.hpp
+++ b/apps/openmw/mwmechanics/mechanicsmanagerimp.hpp
@ -146,7 +146,7 @@ namespace MWMechanics
            std::string_view startKey, std::string_view stopKey, bool forceLoop) override;
        void enableLuaAnimations(const MWWorld::Ptr& ptr, bool enable) override;
        void skipAnimation(const MWWorld::Ptr& ptr) override;
-        bool checkAnimationPlaying(const MWWorld::Ptr& ptr, std::string_view groupName) override;
+        bool checkAnimationPlaying(const MWWorld::Ptr& ptr, const std::string& groupName) override;
        bool checkScriptedAnimationPlaying(const MWWorld::Ptr& ptr) const override;
        void persistAnimationStates() override;
        void clearAnimationQueue(const MWWorld::Ptr& ptr, bool clearScripted) override;
--- a/apps/openmw/mwrender/actoranimation.cpp
+++ b/apps/openmw/mwrender/actoranimation.cpp
@ -161,7 +161,7 @@ namespace MWRender

    bool ActorAnimation::updateCarriedLeftVisible(const int weaptype) const
    {
-        if (Settings::game().mShieldSheathing && mObjectRoot)
+        if (Settings::game().mShieldSheathing)
        {
            const MWWorld::Class& cls = mPtr.getClass();
            MWMechanics::CreatureStats& stats = cls.getCreatureStats(mPtr);
--- a/apps/openmw/mwrender/animation.cpp
+++ b/apps/openmw/mwrender/animation.cpp
@ -1695,7 +1695,7 @@ namespace MWRender
                mGlowUpdater->setColor(color);
                mGlowUpdater->setDuration(glowDuration);
            }
-            else if (mObjectRoot)
+            else
                mGlowUpdater = SceneUtil::addEnchantedGlow(mObjectRoot, mResourceSystem, color, glowDuration);
        }
    }
@ -1869,7 +1869,7 @@ namespace MWRender

    void Animation::setAlpha(float alpha)
    {
-        if (alpha == mAlpha || !mObjectRoot)
+        if (alpha == mAlpha)
            return;
        mAlpha = alpha;

--- a/apps/openmw/mwrender/creatureanimation.cpp
+++ b/apps/openmw/mwrender/creatureanimation.cpp
@ -259,8 +259,7 @@ namespace MWRender
    void CreatureWeaponAnimation::addControllers()
    {
        Animation::addControllers();
-        if (mObjectRoot)
-            WeaponAnimation::addControllers(mNodeMap, mActiveControllers, mObjectRoot.get());
+        WeaponAnimation::addControllers(mNodeMap, mActiveControllers, mObjectRoot.get());
    }

    osg::Vec3f CreatureWeaponAnimation::runAnimation(float duration)
--- a/apps/openmw/mwrender/ripples.cpp
+++ b/apps/openmw/mwrender/ripples.cpp
@ -49,6 +49,18 @@ namespace MWRender
            && exts.glslLanguageVersion >= minimumGLVersionRequiredForCompute;
 #endif

+        static bool pipelineLogged = false;
+
+        if (!pipelineLogged)
+        {
+            if (mUseCompute)
+                Log(Debug::Info) << "Initialized compute shader pipeline for water ripples";
+            else
+                Log(Debug::Info) << "Initialized fallback fragment shader pipeline for water ripples";
+
+            pipelineLogged = true;
+        }
+
        for (size_t i = 0; i < mState.size(); ++i)
        {
            osg::ref_ptr<osg::StateSet> stateset = new osg::StateSet;
@ -86,18 +98,6 @@ namespace MWRender
        else
            setupFragmentPipeline();

-        if (mProgramBlobber != nullptr)
-        {
-            static bool pipelineLogged = [&] {
-                if (mUseCompute)
-                    Log(Debug::Info) << "Initialized compute shader pipeline for water ripples";
-                else
-                    Log(Debug::Info) << "Initialized fallback fragment shader pipeline for water ripples";
-                return true;
-            }();
-            (void)pipelineLogged;
-        }
-
        setCullCallback(new osg::NodeCallback);
        setUpdateCallback(new osg::NodeCallback);
    }
@ -109,36 +109,21 @@ namespace MWRender
        Shader::ShaderManager::DefineMap defineMap = { { "rippleMapSize", std::to_string(sRTTSize) + ".0" } };

        osg::ref_ptr<osg::Shader> vertex = shaderManager.getShader("fullscreen_tri.vert", {}, osg::Shader::VERTEX);
-        osg::ref_ptr<osg::Shader> blobber
-            = shaderManager.getShader("ripples_blobber.frag", defineMap, osg::Shader::FRAGMENT);
-        osg::ref_ptr<osg::Shader> simulate
-            = shaderManager.getShader("ripples_simulate.frag", defineMap, osg::Shader::FRAGMENT);
-        if (vertex == nullptr || blobber == nullptr || simulate == nullptr)
-        {
-            Log(Debug::Error) << "Failed to load shaders required for fragment shader ripple pipeline";
-            return;
-        }

-        mProgramBlobber = shaderManager.getProgram(vertex, std::move(blobber));
-        mProgramSimulation = shaderManager.getProgram(std::move(vertex), std::move(simulate));
+        mProgramBlobber = shaderManager.getProgram(
+            vertex, shaderManager.getShader("ripples_blobber.frag", defineMap, osg::Shader::FRAGMENT));
+        mProgramSimulation = shaderManager.getProgram(
+            std::move(vertex), shaderManager.getShader("ripples_simulate.frag", defineMap, osg::Shader::FRAGMENT));
    }

    void RipplesSurface::setupComputePipeline()
    {
        auto& shaderManager = mResourceSystem->getSceneManager()->getShaderManager();

-        osg::ref_ptr<osg::Shader> blobber
-            = shaderManager.getShader("core/ripples_blobber.comp", {}, osg::Shader::COMPUTE);
-        osg::ref_ptr<osg::Shader> simulate
-            = shaderManager.getShader("core/ripples_simulate.comp", {}, osg::Shader::COMPUTE);
-        if (blobber == nullptr || simulate == nullptr)
-        {
-            Log(Debug::Error) << "Failed to load shaders required for compute shader ripple pipeline";
-            return;
-        }
-
-        mProgramBlobber = shaderManager.getProgram(nullptr, std::move(blobber));
-        mProgramSimulation = shaderManager.getProgram(nullptr, std::move(simulate));
+        mProgramBlobber = shaderManager.getProgram(
+            nullptr, shaderManager.getShader("core/ripples_blobber.comp", {}, osg::Shader::COMPUTE));
+        mProgramSimulation = shaderManager.getProgram(
+            nullptr, shaderManager.getShader("core/ripples_simulate.comp", {}, osg::Shader::COMPUTE));
    }

    void RipplesSurface::updateState(const osg::FrameStamp& frameStamp, State& state)
@ -206,9 +191,6 @@ namespace MWRender

    void RipplesSurface::drawImplementation(osg::RenderInfo& renderInfo) const
    {
-        if (mProgramBlobber == nullptr || mProgramSimulation == nullptr)
-            return;
-
        osg::State& state = *renderInfo.getState();
        const std::size_t currentFrame = state.getFrameStamp()->getFrameNumber() % 2;
        const State& frameState = mState[currentFrame];
--- a/apps/openmw/mwworld/containerstore.hpp
+++ b/apps/openmw/mwworld/containerstore.hpp
@ -406,7 +406,8 @@ namespace MWWorld
        template <class T>
        ContainerStoreIteratorBase(const ContainerStoreIteratorBase<T>& other)
        {
-            static_assert(IsConvertible<T, PtrType, void>::value);
+            char CANNOT_CONVERT_CONST_ITERATOR_TO_ITERATOR[IsConvertible<T, PtrType, void>::value ? 1 : -1];
+            ((void)CANNOT_CONVERT_CONST_ITERATOR_TO_ITERATOR);
            copy(other);
        }

--- a/apps/openmw/mwworld/worldimp.cpp
+++ b/apps/openmw/mwworld/worldimp.cpp
@ -290,14 +290,14 @@ namespace MWWorld
        mSwimHeightScale = mStore.get<ESM::GameSetting>().find("fSwimHeightScale")->mValue.getFloat();
    }

-    void World::init(Debug::Level maxRecastLogLevel, osgViewer::Viewer* viewer, osg::ref_ptr<osg::Group> rootNode,
-        SceneUtil::WorkQueue* workQueue, SceneUtil::UnrefQueue& unrefQueue)
+    void World::init(osgViewer::Viewer* viewer, osg::ref_ptr<osg::Group> rootNode, SceneUtil::WorkQueue* workQueue,
+        SceneUtil::UnrefQueue& unrefQueue)
    {
        mPhysics = std::make_unique<MWPhysics::PhysicsSystem>(mResourceSystem, rootNode);

        if (Settings::navigator().mEnable)
        {
-            auto navigatorSettings = DetourNavigator::makeSettingsFromSettingsManager(maxRecastLogLevel);
+            auto navigatorSettings = DetourNavigator::makeSettingsFromSettingsManager();
            navigatorSettings.mRecast.mSwimHeightScale = mSwimHeightScale;
            mNavigator = DetourNavigator::makeNavigator(navigatorSettings, mUserDataPath);
        }
--- a/apps/openmw/mwworld/worldimp.hpp
+++ b/apps/openmw/mwworld/worldimp.hpp
@ -4,7 +4,6 @@
 #include <osg/Timer>
 #include <osg/ref_ptr>

-#include <components/debug/debuglog.hpp>
 #include <components/esm3/readerscache.hpp>
 #include <components/misc/rng.hpp>
 #include <components/settings/settings.hpp>
@ -202,8 +201,8 @@ namespace MWWorld
            Loading::Listener* listener);

        // Must be called after `loadData`.
-        void init(Debug::Level maxRecastLogLevel, osgViewer::Viewer* viewer, osg::ref_ptr<osg::Group> rootNode,
-            SceneUtil::WorkQueue* workQueue, SceneUtil::UnrefQueue& unrefQueue);
+        void init(osgViewer::Viewer* viewer, osg::ref_ptr<osg::Group> rootNode, SceneUtil::WorkQueue* workQueue,
+            SceneUtil::UnrefQueue& unrefQueue);

        virtual ~World();

--- a/apps/openmw_tests/main.cpp
+++ b/apps/openmw_tests/main.cpp
@ -2,7 +2,6 @@
 #include <components/misc/strings/conversion.hpp>
 #include <components/settings/parser.hpp>
 #include <components/settings/values.hpp>
-#include <components/testing/util.hpp>

 #include <gtest/gtest.h>

@ -25,9 +24,5 @@ int main(int argc, char* argv[])
    Settings::StaticValues::init();

    testing::InitGoogleTest(&argc, argv);
-
-    const int result = RUN_ALL_TESTS();
-    if (result == 0)
-        std::filesystem::remove_all(TestingOpenMW::outputDir());
-    return result;
+    return RUN_ALL_TESTS();
 }
--- a/components/CMakeLists.txt
+++ b/components/CMakeLists.txt
@ -59,7 +59,7 @@ list(APPEND COMPONENT_FILES "${OpenMW_BINARY_DIR}/${OSG_PLUGIN_CHECKER_CPP_FILE}

 add_component_dir (lua
    luastate scriptscontainer asyncpackage utilpackage serialization configuration l10n storage utf8
-    shapes/box inputactions yamlloader scripttracker luastateptr
+    shapes/box inputactions yamlloader scripttracker
    )

 add_component_dir (l10n
--- a/components/config/gamesettings.cpp
+++ b/components/config/gamesettings.cpp
@ -1,7 +1,6 @@
 #include "gamesettings.hpp"

 #include <QDir>
-#include <QProgressDialog>
 #include <QRegularExpression>

 #include <components/files/configurationmanager.hpp>
@ -38,13 +37,8 @@ void Config::GameSettings::validatePaths()

    mDataDirs.clear();

-    QProgressDialog progressBar("Validating paths", {}, 0, paths.count() + 1);
-    progressBar.setWindowModality(Qt::WindowModal);
-    progressBar.setValue(0);
-
    for (const auto& dataDir : paths)
    {
-        progressBar.setValue(progressBar.value() + 1);
        if (QDir(dataDir.value).exists())
        {
            SettingValue copy = dataDir;
@ -56,8 +50,6 @@ void Config::GameSettings::validatePaths()
    // Do the same for data-local
    const QString& local = mSettings.value(QString("data-local")).value;

-    progressBar.setValue(progressBar.value() + 1);
-
    if (!local.isEmpty() && QDir(local).exists())
    {
        mDataLocal = QDir(local).canonicalPath();
--- a/components/contentselector/model/contentmodel.cpp
+++ b/components/contentselector/model/contentmodel.cpp
@ -10,7 +10,6 @@
 #include <QDataStream>
 #include <QDebug>
 #include <QDir>
-#include <QDirIterator>
 #include <QFont>
 #include <QIODevice>

@ -79,10 +78,14 @@ ContentSelectorModel::EsmFile* ContentSelectorModel::ContentModel::item(int row)
 }
 const ContentSelectorModel::EsmFile* ContentSelectorModel::ContentModel::item(const QString& name) const
 {
-    bool path = name.contains('/');
+    EsmFile::FileProperty fp = EsmFile::FileProperty_FileName;
+
+    if (name.contains('/'))
+        fp = EsmFile::FileProperty_FilePath;
+
    for (const EsmFile* file : mFiles)
    {
-        if (name.compare(path ? file->filePath() : file->fileName(), Qt::CaseInsensitive) == 0)
+        if (name.compare(file->fileProperty(fp).toString(), Qt::CaseInsensitive) == 0)
            return file;
    }
    return nullptr;
@ -307,6 +310,7 @@ bool ContentSelectorModel::ContentModel::setData(const QModelIndex& index, const
            {
                setCheckState(file->filePath(), success);
                emit dataChanged(index, index);
+                checkForLoadOrderErrors();
            }
            else
                return success;
@ -421,6 +425,7 @@ bool ContentSelectorModel::ContentModel::dropMimeData(

    dataChanged(index(minRow, 0), index(maxRow, 0));
    // at this point we know that drag and drop has finished.
+    checkForLoadOrderErrors();

    return true;
 }
@ -547,13 +552,15 @@ void ContentSelectorModel::ContentModel::addFiles(const QString& path, bool newf

 bool ContentSelectorModel::ContentModel::containsDataFiles(const QString& path)
 {
+    QDir dir(path);
    QStringList filters;
    filters << "*.esp"
            << "*.esm"
            << "*.omwgame"
            << "*.omwaddon";
-    QDirIterator it(path, filters, QDir::Files | QDir::NoDotAndDotDot);
-    return it.hasNext();
+    dir.setNameFilters(filters);
+
+    return dir.entryList().count() != 0;
 }

 void ContentSelectorModel::ContentModel::clearFiles()
@ -700,13 +707,12 @@ void ContentSelectorModel::ContentModel::setNonUserContent(const QStringList& fi

 bool ContentSelectorModel::ContentModel::isLoadOrderError(const EsmFile* file) const
 {
-    int index = indexFromItem(file).row();
-    auto errors = checkForLoadOrderErrors(file, index);
-    return !errors.empty();
+    return mPluginsWithLoadOrderError.contains(file->filePath());
 }

 void ContentSelectorModel::ContentModel::setContentList(const QStringList& fileList)
 {
+    mPluginsWithLoadOrderError.clear();
    int previousPosition = -1;
    for (const QString& filepath : fileList)
    {
@ -719,6 +725,7 @@ void ContentSelectorModel::ContentModel::setContentList(const QStringList& fileL
            if (filePosition < previousPosition)
            {
                mFiles.move(filePosition, previousPosition);
+                emit dataChanged(index(filePosition, 0, QModelIndex()), index(previousPosition, 0, QModelIndex()));
            }
            else
            {
@ -726,7 +733,24 @@ void ContentSelectorModel::ContentModel::setContentList(const QStringList& fileL
            }
        }
    }
-    emit dataChanged(index(0, 0), index(rowCount(), columnCount()));
+    checkForLoadOrderErrors();
+}
+
+void ContentSelectorModel::ContentModel::checkForLoadOrderErrors()
+{
+    for (int row = 0; row < mFiles.count(); ++row)
+    {
+        EsmFile* file = mFiles.at(row);
+        bool isRowInError = checkForLoadOrderErrors(file, row).count() != 0;
+        if (isRowInError)
+        {
+            mPluginsWithLoadOrderError.insert(file->filePath());
+        }
+        else
+        {
+            mPluginsWithLoadOrderError.remove(file->filePath());
+        }
+    }
 }

 QList<ContentSelectorModel::LoadOrderError> ContentSelectorModel::ContentModel::checkForLoadOrderErrors(
@ -767,12 +791,11 @@ QList<ContentSelectorModel::LoadOrderError> ContentSelectorModel::ContentModel::

 QString ContentSelectorModel::ContentModel::toolTip(const EsmFile* file) const
 {
-    int index = indexFromItem(file).row();
-    auto errors = checkForLoadOrderErrors(file, index);
-    if (!errors.empty())
+    if (isLoadOrderError(file))
    {
        QString text("<b>");
-        for (const LoadOrderError& error : errors)
+        int index = indexFromItem(item(file->filePath())).row();
+        for (const LoadOrderError& error : checkForLoadOrderErrors(file, index))
        {
            assert(error.errorCode() != LoadOrderError::ErrorCode::ErrorCode_None);

@ -877,6 +900,7 @@ ContentSelectorModel::ContentFileList ContentSelectorModel::ContentModel::checke

 void ContentSelectorModel::ContentModel::uncheckAll()
 {
+    emit layoutAboutToBeChanged();
    mCheckedFiles.clear();
-    emit dataChanged(index(0, 0), index(rowCount(), columnCount()), { Qt::CheckStateRole, Qt::UserRole + 1 });
+    emit layoutChanged();
 }
--- a/components/contentselector/model/contentmodel.hpp
+++ b/components/contentselector/model/contentmodel.hpp
@ -69,6 +69,9 @@ namespace ContentSelectorModel

        void refreshModel();

+        /// Checks all plug-ins for load order errors and updates mPluginsWithLoadOrderError with plug-ins with issues
+        void checkForLoadOrderErrors();
+
    private:
        void addFile(EsmFile* file);

@ -86,6 +89,7 @@ namespace ContentSelectorModel
        QStringList mNonUserContent;
        std::set<const EsmFile*> mCheckedFiles;
        QHash<QString, bool> mNewFiles;
+        QSet<QString> mPluginsWithLoadOrderError;
        QString mEncoding;
        QIcon mWarningIcon;
        QIcon mErrorIcon;
--- a/components/contentselector/model/esmfile.hpp
+++ b/components/contentselector/model/esmfile.hpp
@ -48,18 +48,18 @@ namespace ContentSelectorModel
        void addGameFile(const QString& name) { mGameFiles.append(name); }
        QVariant fileProperty(const FileProperty prop) const;

-        const QString& fileName() const { return mFileName; }
-        const QString& author() const { return mAuthor; }
+        QString fileName() const { return mFileName; }
+        QString author() const { return mAuthor; }
        QDateTime modified() const { return mModified; }
-        const QString& formatVersion() const { return mVersion; }
-        const QString& filePath() const { return mPath; }
+        QString formatVersion() const { return mVersion; }
+        QString filePath() const { return mPath; }
        bool builtIn() const { return mBuiltIn; }
        bool fromAnotherConfigFile() const { return mFromAnotherConfigFile; }
        bool isMissing() const { return mPath.isEmpty(); }

        /// @note Contains file names, not paths.
        const QStringList& gameFiles() const { return mGameFiles; }
-        const QString& description() const { return mDescription; }
+        QString description() const { return mDescription; }
        QString toolTip() const
        {
            if (isMissing())
--- a/components/contentselector/view/contentselector.cpp
+++ b/components/contentselector/view/contentselector.cpp
@ -211,6 +211,7 @@ void ContentSelectorView::ContentSelector::addFiles(const QString& path, bool ne
        ui->gameFileView->setCurrentIndex(0);

    mContentModel->uncheckAll();
+    mContentModel->checkForLoadOrderErrors();
 }

 void ContentSelectorView::ContentSelector::sortFiles()
@ -253,6 +254,7 @@ void ContentSelectorView::ContentSelector::slotCurrentGameFileIndexChanged(int i
        oldIndex = index;

        setGameFileSelected(index, true);
+        mContentModel->checkForLoadOrderErrors();
    }

    emit signalCurrentGamefileIndexChanged(index);
--- a/components/crashcatcher/windows_crashcatcher.cpp
+++ b/components/crashcatcher/windows_crashcatcher.cpp
@ -237,7 +237,7 @@ namespace Crash
        // must remain until monitor has finished
        waitMonitor();

-        std::string message = "OpenMW has encountered a fatal error.\nCrash dump saved to '"
+        std::string message = "OpenMW has encountered a fatal error.\nCrash log saved to '"
            + Misc::StringUtils::u8StringToString(getCrashDumpPath(*mShm).u8string())
            + "'.\nPlease report this to https://gitlab.com/OpenMW/openmw/issues !";
        SDL_ShowSimpleMessageBox(0, "Fatal Error", message.c_str(), nullptr);
--- a/components/crashcatcher/windows_crashcatcher.hpp
+++ b/components/crashcatcher/windows_crashcatcher.hpp
@ -21,7 +21,6 @@ namespace Crash
    // the main openmw process in task manager.

    static constexpr const int CrashCatcherTimeout = 2500;
-    static constexpr const int CrashCatcherThawTimeout = 250;

    struct CrashSHM;

--- a/components/crashcatcher/windows_crashmonitor.cpp
+++ b/components/crashcatcher/windows_crashmonitor.cpp
@ -87,10 +87,9 @@ namespace Crash
        SetEvent(mSignalAppEvent);
    }

-    bool CrashMonitor::waitApp(bool thawMode) const
+    bool CrashMonitor::waitApp() const
    {
-        return WaitForSingleObject(mSignalMonitorEvent, thawMode ? CrashCatcherThawTimeout : CrashCatcherTimeout)
-            == WAIT_OBJECT_0;
+        return WaitForSingleObject(mSignalMonitorEvent, CrashCatcherTimeout) == WAIT_OBJECT_0;
    }

    bool CrashMonitor::isAppAlive() const
@ -186,7 +185,7 @@ namespace Crash
                    frozen = false;
                }

-                if (!mFreezeAbort && waitApp(frozen))
+                if (!mFreezeAbort && waitApp())
                {
                    shmLock();

@ -216,7 +215,7 @@ namespace Crash
            {
                handleCrash(true);
                TerminateProcess(mAppProcessHandle, 0xDEAD);
-                std::string message = "OpenMW has frozen.\nCrash dump saved to '"
+                std::string message = "OpenMW appears to have frozen.\nCrash log saved to '"
                    + Misc::StringUtils::u8StringToString(getFreezeDumpPath(*mShm).u8string())
                    + "'.\nPlease report this to https://gitlab.com/OpenMW/openmw/issues !";
                SDL_ShowSimpleMessageBox(0, "Fatal Error", message.c_str(), nullptr);
@ -290,10 +289,10 @@ namespace Crash
    {
        std::thread messageBoxThread([&]() {
            SDL_MessageBoxButtonData button = { SDL_MESSAGEBOX_BUTTON_RETURNKEY_DEFAULT, 0, "Abort" };
-            SDL_MessageBoxData messageBoxData = { SDL_MESSAGEBOX_ERROR, nullptr, "OpenMW has frozen",
-                "OpenMW has frozen. This should never happen. Press Abort to terminate it and generate a crash dump to "
-                "help diagnose the problem.\nOpenMW may unfreeze if you wait, and this message box will disappear "
-                "after it becomes responsive.",
+            SDL_MessageBoxData messageBoxData = { SDL_MESSAGEBOX_ERROR, nullptr, "OpenMW appears to have frozen",
+                "OpenMW appears to have frozen. Press Abort to terminate it and generate a crash dump.\nIf OpenMW "
+                "hasn't actually frozen, this message box will disappear a within a few seconds of it becoming "
+                "responsive.",
                1, &button, nullptr };

            int buttonId;
--- a/components/crashcatcher/windows_crashmonitor.hpp
+++ b/components/crashcatcher/windows_crashmonitor.hpp
@ -41,7 +41,7 @@ namespace Crash

        void signalApp() const;

-        bool waitApp(bool thawMode) const;
+        bool waitApp() const;

        bool isAppAlive() const;

--- a/components/debug/debugging.cpp
+++ b/components/debug/debugging.cpp
@ -106,96 +106,94 @@ namespace Debug
        logListener = std::move(listener);
    }

-    namespace
+    class DebugOutputBase : public boost::iostreams::sink
    {
-        class DebugOutputBase : public boost::iostreams::sink
+    public:
+        virtual std::streamsize write(const char* str, std::streamsize size)
        {
-        public:
-            virtual std::streamsize write(const char* str, std::streamsize size)
+            if (size <= 0)
+                return size;
+            std::string_view msg{ str, static_cast<size_t>(size) };
+
+            // Skip debug level marker
+            Level level = All;
+            if (Log::sWriteLevel)
            {
-                if (size <= 0)
-                    return size;
-                std::string_view msg{ str, static_cast<size_t>(size) };
+                level = getLevelMarker(msg[0]);
+                msg = msg.substr(1);
+            }

-                // Skip debug level marker
-                Level level = All;
-                if (Log::sWriteLevel)
-                {
-                    level = getLevelMarker(msg[0]);
-                    msg = msg.substr(1);
-                }
-
-                char prefix[32];
-                std::size_t prefixSize;
-                {
-                    prefix[0] = '[';
-                    const auto now = std::chrono::system_clock::now();
-                    const auto time = std::chrono::system_clock::to_time_t(now);
-                    tm time_info{};
+            char prefix[32];
+            std::size_t prefixSize;
+            {
+                prefix[0] = '[';
+                const auto now = std::chrono::system_clock::now();
+                const auto time = std::chrono::system_clock::to_time_t(now);
+                tm time_info{};
 #ifdef _WIN32
-                    (void)localtime_s(&time_info, &time);
+                (void)localtime_s(&time_info, &time);
 #else
-                    (void)localtime_r(&time, &time_info);
+                (void)localtime_r(&time, &time_info);
 #endif
-                    prefixSize = std::strftime(prefix + 1, sizeof(prefix) - 1, "%T", &time_info) + 1;
-                    char levelLetter = " EWIVD*"[int(level)];
-                    const auto ms
-                        = std::chrono::duration_cast<std::chrono::milliseconds>(now.time_since_epoch()).count();
-                    prefixSize += snprintf(prefix + prefixSize, sizeof(prefix) - prefixSize, ".%03u %c] ",
-                        static_cast<unsigned>(ms % 1000), levelLetter);
-                }
-
-                while (!msg.empty())
-                {
-                    if (msg[0] == 0)
-                        break;
-                    size_t lineSize = 1;
-                    while (lineSize < msg.size() && msg[lineSize - 1] != '\n')
-                        lineSize++;
-                    writeImpl(prefix, prefixSize, level);
-                    writeImpl(msg.data(), lineSize, level);
-                    if (logListener)
-                        logListener(
-                            level, std::string_view(prefix, prefixSize), std::string_view(msg.data(), lineSize));
-                    msg = msg.substr(lineSize);
-                }
-
-                return size;
+                prefixSize = std::strftime(prefix + 1, sizeof(prefix) - 1, "%T", &time_info) + 1;
+                char levelLetter = " EWIVD*"[int(level)];
+                const auto ms = std::chrono::duration_cast<std::chrono::milliseconds>(now.time_since_epoch()).count();
+                prefixSize += snprintf(prefix + prefixSize, sizeof(prefix) - prefixSize, ".%03u %c] ",
+                    static_cast<unsigned>(ms % 1000), levelLetter);
            }

-            virtual ~DebugOutputBase() = default;
-
-        protected:
-            static Level getLevelMarker(char marker)
+            while (!msg.empty())
            {
-                if (0 <= marker && static_cast<unsigned>(marker) < static_cast<unsigned>(All))
-                    return static_cast<Level>(marker);
-                return All;
+                if (msg[0] == 0)
+                    break;
+                size_t lineSize = 1;
+                while (lineSize < msg.size() && msg[lineSize - 1] != '\n')
+                    lineSize++;
+                writeImpl(prefix, prefixSize, level);
+                writeImpl(msg.data(), lineSize, level);
+                if (logListener)
+                    logListener(level, std::string_view(prefix, prefixSize), std::string_view(msg.data(), lineSize));
+                msg = msg.substr(lineSize);
            }

-            virtual std::streamsize writeImpl(const char* str, std::streamsize size, Level debugLevel)
-            {
-                return size;
-            }
-        };
+            return size;
+        }
+
+        virtual ~DebugOutputBase() = default;
+
+    protected:
+        static Level getLevelMarker(char marker)
+        {
+            if (0 <= marker && static_cast<unsigned>(marker) < static_cast<unsigned>(All))
+                return static_cast<Level>(marker);
+            return All;
+        }
+
+        virtual std::streamsize writeImpl(const char* str, std::streamsize size, Level debugLevel)
+        {
+            return size;
+        }
+    };

 #if defined _WIN32 && defined _DEBUG
-        class DebugOutput : public DebugOutputBase
+    class DebugOutput : public DebugOutputBase
+    {
+    public:
+        std::streamsize writeImpl(const char* str, std::streamsize size, Level debugLevel)
        {
-        public:
-            std::streamsize writeImpl(const char* str, std::streamsize size, Level debugLevel)
-            {
-                // Make a copy for null termination
-                std::string tmp(str, static_cast<unsigned int>(size));
-                // Write string to Visual Studio Debug output
-                OutputDebugString(tmp.c_str());
-                return size;
-            }
+            // Make a copy for null termination
+            std::string tmp(str, static_cast<unsigned int>(size));
+            // Write string to Visual Studio Debug output
+            OutputDebugString(tmp.c_str());
+            return size;
+        }

-            virtual ~DebugOutput() = default;
-        };
+        virtual ~DebugOutput() = default;
+    };
 #else

+    namespace
+    {
        struct Record
        {
            std::string mValue;
@ -326,38 +324,22 @@ namespace Debug
            First mFirst;
            Second mSecond;
        };
+    }
 #endif

-        Level toLevel(std::string_view value)
-        {
-            if (value == "ERROR")
-                return Error;
-            if (value == "WARNING")
-                return Warning;
-            if (value == "INFO")
-                return Info;
-            if (value == "VERBOSE")
-                return Verbose;
-            if (value == "DEBUG")
-                return Debug;
-
-            return Verbose;
-        }
-
-        static std::unique_ptr<std::ostream> rawStdout = nullptr;
-        static std::unique_ptr<std::ostream> rawStderr = nullptr;
-        static std::unique_ptr<std::mutex> rawStderrMutex = nullptr;
-        static std::ofstream logfile;
+    static std::unique_ptr<std::ostream> rawStdout = nullptr;
+    static std::unique_ptr<std::ostream> rawStderr = nullptr;
+    static std::unique_ptr<std::mutex> rawStderrMutex = nullptr;
+    static std::ofstream logfile;

 #if defined(_WIN32) && defined(_DEBUG)
-        static boost::iostreams::stream_buffer<DebugOutput> sb;
+    static boost::iostreams::stream_buffer<DebugOutput> sb;
 #else
-        static boost::iostreams::stream_buffer<Tee<Identity, Coloured>> standardOut;
-        static boost::iostreams::stream_buffer<Tee<Identity, Coloured>> standardErr;
-        static boost::iostreams::stream_buffer<Tee<Buffer, Coloured>> bufferedOut;
-        static boost::iostreams::stream_buffer<Tee<Buffer, Coloured>> bufferedErr;
+    static boost::iostreams::stream_buffer<Tee<Identity, Coloured>> standardOut;
+    static boost::iostreams::stream_buffer<Tee<Identity, Coloured>> standardErr;
+    static boost::iostreams::stream_buffer<Tee<Buffer, Coloured>> bufferedOut;
+    static boost::iostreams::stream_buffer<Tee<Buffer, Coloured>> bufferedErr;
 #endif
-    }

    std::ostream& getRawStdout()
    {
@ -377,19 +359,23 @@ namespace Debug
    Level getDebugLevel()
    {
        if (const char* env = getenv("OPENMW_DEBUG_LEVEL"))
-            return toLevel(env);
+        {
+            const std::string_view value(env);
+            if (value == "ERROR")
+                return Error;
+            if (value == "WARNING")
+                return Warning;
+            if (value == "INFO")
+                return Info;
+            if (value == "VERBOSE")
+                return Verbose;
+            if (value == "DEBUG")
+                return Debug;
+        }

        return Verbose;
    }

-    Level getRecastMaxLogLevel()
-    {
-        if (const char* env = getenv("OPENMW_RECAST_MAX_LOG_LEVEL"))
-            return toLevel(env);
-
-        return Error;
-    }
-
    void setupLogging(const std::filesystem::path& logDir, std::string_view appName)
    {
        Log::sMinDebugLevel = getDebugLevel();
--- a/components/debug/debugging.hpp
+++ b/components/debug/debugging.hpp
@ -36,8 +36,6 @@ namespace Debug

    Level getDebugLevel();

-    Level getRecastMaxLogLevel();
-
    // Redirect cout and cerr to the log file
    void setupLogging(const std::filesystem::path& logDir, std::string_view appName);

--- a/components/detournavigator/asyncnavmeshupdater.cpp
+++ b/components/detournavigator/asyncnavmeshupdater.cpp
@ -453,9 +453,9 @@ namespace DetourNavigator
        Misc::setCurrentThreadIdlePriority();
        while (!mShouldStop)
        {
-            if (JobIt job = getNextJob(); job != mJobs.end())
+            try
            {
-                try
+                if (JobIt job = getNextJob(); job != mJobs.end())
                {
                    const JobStatus status = processJob(*job);
                    Log(Debug::Debug) << "Processed job " << job->mId << " with status=" << status
@ -480,20 +480,12 @@ namespace DetourNavigator
                        }
                    }
                }
-                catch (const std::exception& e)
-                {
-                    Log(Debug::Warning) << "Failed to process navmesh job " << job->mId
-                                        << " for worldspace=" << job->mWorldspace << " agent=" << job->mAgentBounds
-                                        << " changedTile=(" << job->mChangedTile << ")"
-                                        << " changeType=" << job->mChangeType
-                                        << " by thread=" << std::this_thread::get_id() << ": " << e.what();
-                    unlockTile(job->mId, job->mAgentBounds, job->mChangedTile);
-                    removeJob(job);
-                }
+                else
+                    cleanupLastUpdates();
            }
-            else
+            catch (const std::exception& e)
            {
-                cleanupLastUpdates();
+                Log(Debug::Error) << "AsyncNavMeshUpdater::process exception: " << e.what();
            }
        }
        Log(Debug::Debug) << "Stop navigator jobs processing by thread=" << std::this_thread::get_id();
@ -501,8 +493,7 @@ namespace DetourNavigator

    JobStatus AsyncNavMeshUpdater::processJob(Job& job)
    {
-        Log(Debug::Debug) << "Processing job " << job.mId << "  for worldspace=" << job.mWorldspace
-                          << " agent=" << job.mAgentBounds << ""
+        Log(Debug::Debug) << "Processing job " << job.mId << " for agent=(" << job.mAgentBounds << ")"
                          << " changedTile=(" << job.mChangedTile << ")"
                          << " changeType=" << job.mChangeType << " by thread=" << std::this_thread::get_id();

@ -552,14 +543,7 @@ namespace DetourNavigator
            return JobStatus::Done;
        }

-        try
-        {
-            writeDebugRecastMesh(mSettings, job.mChangedTile, *recastMesh);
-        }
-        catch (const std::exception& e)
-        {
-            Log(Debug::Warning) << "Failed to write debug recast mesh: " << e.what();
-        }
+        writeDebugRecastMesh(mSettings, job.mChangedTile, *recastMesh);

        NavMeshTilesCache::Value cachedNavMeshData
            = mNavMeshTilesCache.get(job.mAgentBounds, job.mChangedTile, *recastMesh);
@ -682,19 +666,12 @@ namespace DetourNavigator
            mPresentTiles.insert(std::make_tuple(job.mAgentBounds, job.mChangedTile));
        }

-        try
-        {
-            writeDebugNavMesh(mSettings, navMeshCacheItem, navMeshVersion);
-        }
-        catch (const std::exception& e)
-        {
-            Log(Debug::Warning) << "Failed to write debug navmesh: " << e.what();
-        }
+        writeDebugNavMesh(mSettings, navMeshCacheItem, navMeshVersion);

        return isSuccess(status) ? JobStatus::Done : JobStatus::Fail;
    }

-    JobIt AsyncNavMeshUpdater::getNextJob() noexcept
+    JobIt AsyncNavMeshUpdater::getNextJob()
    {
        std::unique_lock<std::mutex> lock(mMutex);

@ -769,7 +746,7 @@ namespace DetourNavigator
        return mJobs.size();
    }

-    void AsyncNavMeshUpdater::cleanupLastUpdates() noexcept
+    void AsyncNavMeshUpdater::cleanupLastUpdates()
    {
        const auto now = std::chrono::steady_clock::now();

--- a/components/detournavigator/asyncnavmeshupdater.hpp
+++ b/components/detournavigator/asyncnavmeshupdater.hpp
@ -244,7 +244,7 @@ namespace DetourNavigator
        inline JobStatus handleUpdateNavMeshStatus(UpdateNavMeshStatus status, const Job& job,
            const GuardedNavMeshCacheItem& navMeshCacheItem, const RecastMesh& recastMesh);

-        inline JobIt getNextJob() noexcept;
+        JobIt getNextJob();

        void postThreadJob(JobIt job, std::deque<JobIt>& queue);

@ -254,7 +254,7 @@ namespace DetourNavigator

        inline std::size_t getTotalJobs() const;

-        inline void cleanupLastUpdates() noexcept;
+        void cleanupLastUpdates();

        inline void waitUntilJobsDoneForNotPresentTiles(Loading::Listener* listener);

--- a/components/detournavigator/makenavmesh.cpp
+++ b/components/detournavigator/makenavmesh.cpp
@ -523,7 +523,7 @@ namespace DetourNavigator
    std::unique_ptr<PreparedNavMeshData> prepareNavMeshTileData(const RecastMesh& recastMesh, ESM::RefId worldspace,
        const TilePosition& tilePosition, const AgentBounds& agentBounds, const RecastSettings& settings)
    {
-        RecastContext context(worldspace, tilePosition, agentBounds, recastMesh.getVersion(), settings.mMaxLogLevel);
+        RecastContext context(worldspace, tilePosition, agentBounds);

        const auto [minZ, maxZ] = getBoundsByZ(recastMesh, agentBounds.mHalfExtents.z(), settings);

--- a/components/detournavigator/recastcontext.cpp
+++ b/components/detournavigator/recastcontext.cpp
@ -1,7 +1,7 @@
 #include "recastcontext.hpp"
 #include "debug.hpp"

-#include <components/debug/debuglog.hpp>
+#include "components/debug/debuglog.hpp"

 #include <sstream>

@ -23,30 +23,25 @@ namespace DetourNavigator
            return Debug::Debug;
        }

-        std::string formatPrefix(ESM::RefId worldspace, const TilePosition& tilePosition,
-            const AgentBounds& agentBounds, const Version& version)
+        std::string formatPrefix(
+            ESM::RefId worldspace, const TilePosition& tilePosition, const AgentBounds& agentBounds)
        {
            std::ostringstream stream;
            stream << "Worldspace: " << worldspace << "; tile position: " << tilePosition.x() << ", "
-                   << tilePosition.y() << "; agent bounds: " << agentBounds << "; version: " << version << "; ";
+                   << tilePosition.y() << "; agent bounds: " << agentBounds << "; ";
            return stream.str();
        }
    }

-    RecastContext::RecastContext(ESM::RefId worldspace, const TilePosition& tilePosition,
-        const AgentBounds& agentBounds, const Version& version, Debug::Level maxLogLevel)
-        : mMaxLogLevel(maxLogLevel)
-        , mPrefix(formatPrefix(worldspace, tilePosition, agentBounds, version))
+    RecastContext::RecastContext(
+        ESM::RefId worldspace, const TilePosition& tilePosition, const AgentBounds& agentBounds)
+        : mPrefix(formatPrefix(worldspace, tilePosition, agentBounds))
    {
    }

    void RecastContext::doLog(const rcLogCategory category, const char* msg, const int len)
    {
-        if (msg == nullptr || len <= 0)
-            return;
-        const Debug::Level level = getLogLevel(category);
-        if (level > mMaxLogLevel)
-            return;
-        Log(level) << mPrefix << std::string_view(msg, static_cast<std::size_t>(len));
+        if (len > 0)
+            Log(getLogLevel(category)) << mPrefix << std::string_view(msg, static_cast<std::size_t>(len));
    }
 }
--- a/components/detournavigator/recastcontext.hpp
+++ b/components/detournavigator/recastcontext.hpp
@ -3,7 +3,6 @@

 #include "tileposition.hpp"

-#include <components/debug/debuglog.hpp>
 #include <components/esm/refid.hpp>

 #include <string>
@ -13,18 +12,15 @@
 namespace DetourNavigator
 {
    struct AgentBounds;
-    struct Version;

    class RecastContext final : public rcContext
    {
    public:
-        explicit RecastContext(ESM::RefId worldspace, const TilePosition& tilePosition, const AgentBounds& agentBounds,
-            const Version& version, Debug::Level maxLogLevel);
+        explicit RecastContext(ESM::RefId worldspace, const TilePosition& tilePosition, const AgentBounds& agentBounds);

        const std::string& getPrefix() const { return mPrefix; }

    private:
-        Debug::Level mMaxLogLevel;
        std::string mPrefix;

        void doLog(rcLogCategory category, const char* msg, int len) override;
--- a/components/detournavigator/settings.cpp
+++ b/components/detournavigator/settings.cpp
@ -44,7 +44,7 @@ namespace DetourNavigator
            };
        }

-        RecastSettings makeRecastSettingsFromSettingsManager(Debug::Level maxLogLevel)
+        RecastSettings makeRecastSettingsFromSettingsManager()
        {
            RecastSettings result;

@ -63,7 +63,6 @@ namespace DetourNavigator
            result.mRegionMergeArea = ::Settings::navigator().mRegionMergeArea;
            result.mRegionMinArea = ::Settings::navigator().mRegionMinArea;
            result.mTileSize = ::Settings::navigator().mTileSize;
-            result.mMaxLogLevel = maxLogLevel;

            return result;
        }
@ -81,11 +80,11 @@ namespace DetourNavigator
        }
    }

-    Settings makeSettingsFromSettingsManager(Debug::Level maxLogLevel)
+    Settings makeSettingsFromSettingsManager()
    {
        Settings result;

-        result.mRecast = makeRecastSettingsFromSettingsManager(maxLogLevel);
+        result.mRecast = makeRecastSettingsFromSettingsManager();
        result.mDetour = makeDetourSettingsFromSettingsManager();

        const NavMeshLimits limits = getNavMeshTileLimits(result.mDetour);
--- a/components/detournavigator/settings.hpp
+++ b/components/detournavigator/settings.hpp
@ -1,8 +1,6 @@
 #ifndef OPENMW_COMPONENTS_DETOURNAVIGATOR_SETTINGS_H
 #define OPENMW_COMPONENTS_DETOURNAVIGATOR_SETTINGS_H

-#include <components/debug/debuglog.hpp>
-
 #include <chrono>
 #include <string>

@ -25,7 +23,6 @@ namespace DetourNavigator
        int mRegionMergeArea = 0;
        int mRegionMinArea = 0;
        int mTileSize = 0;
-        Debug::Level mMaxLogLevel = Debug::Error;
    };

    struct DetourSettings
@ -58,7 +55,7 @@ namespace DetourNavigator

    inline constexpr std::int64_t navMeshFormatVersion = 2;

-    Settings makeSettingsFromSettingsManager(Debug::Level maxLogLevel);
+    Settings makeSettingsFromSettingsManager();
 }

 #endif
--- a/components/esm/luascripts.cpp
+++ b/components/esm/luascripts.cpp
@ -1,9 +1,8 @@
 #include "luascripts.hpp"

-#include <components/esm3/esmreader.hpp>
-#include <components/esm3/esmwriter.hpp>
+#include "components/esm3/esmreader.hpp"
+#include "components/esm3/esmwriter.hpp"

-#include <components/lua/luastateptr.hpp>
 #include <components/lua/serialization.hpp>

 // List of all records, that are related to Lua.
@ -103,16 +102,13 @@ void ESM::LuaScriptsCfg::adjustRefNums(const ESMReader& esm)
            throw std::runtime_error("Incorrect contentFile index");
    };

-    LuaUtil::LuaStatePtr state(luaL_newstate());
-    if (state == nullptr)
-        throw std::runtime_error("Failed to create Lua runtime");
-
+    lua_State* L = luaL_newstate();
    LuaUtil::BasicSerializer serializer(adjustRefNumFn);

    auto adjustLuaData = [&](std::string& data) {
        if (data.empty())
            return;
-        sol::object luaData = LuaUtil::deserialize(state.get(), data, &serializer);
+        sol::object luaData = LuaUtil::deserialize(L, data, &serializer);
        data = LuaUtil::serialize(luaData, &serializer);
    };

@ -127,6 +123,7 @@ void ESM::LuaScriptsCfg::adjustRefNums(const ESMReader& esm)
            refCfg.mRefnumContentFile = adjustRefNumFn(refCfg.mRefnumContentFile);
        }
    }
+    lua_close(L);
 }

 void ESM::LuaScriptsCfg::save(ESMWriter& esm) const
--- a/components/esm3/esmreader.cpp
+++ b/components/esm3/esmreader.cpp
@ -73,10 +73,10 @@ namespace ESM
            int index = getIndex();
            for (int i = 0; i < getIndex(); i++)
            {
-                if (readers.getFileSize(static_cast<std::size_t>(i)) == 0)
+                const ESM::ReadersCache::BusyItem reader = readers.get(static_cast<std::size_t>(i));
+                if (reader->getFileSize() == 0)
                    continue; // Content file in non-ESM format
-                const auto fnamecandidate
-                    = Files::pathToUnicodeString(readers.getName(static_cast<std::size_t>(i)).filename());
+                const auto fnamecandidate = Files::pathToUnicodeString(reader->getName().filename());
                if (Misc::StringUtils::ciEqual(fname, fnamecandidate))
                {
                    index = i;
--- a/components/esm3/readerscache.cpp
+++ b/components/esm3/readerscache.cpp
@ -47,7 +47,6 @@ namespace ESM
                    {
                        it->mReader.open(*it->mName);
                        it->mName.reset();
-                        it->mFileSize.reset();
                    }
                    mBusyItems.splice(mBusyItems.end(), mClosedItems, it);
                    break;
@ -58,46 +57,6 @@ namespace ESM
        return BusyItem(*this, it);
    }

-    const std::filesystem::path& ReadersCache::getName(std::size_t index) const
-    {
-        const auto indexIt = mIndex.find(index);
-        if (indexIt == mIndex.end())
-            throw std::logic_error("ESMReader at index " + std::to_string(index) + " has not been created yet");
-        switch (indexIt->second->mState)
-        {
-            case State::Busy:
-            case State::Free:
-                return indexIt->second->mReader.getName();
-            case State::Closed:
-                if (indexIt->second->mName)
-                    return *indexIt->second->mName;
-                throw std::logic_error("ESMReader at index " + std::to_string(index) + " has forgotten its filename");
-            default:
-                throw std::logic_error("ESMReader at index " + std::to_string(index) + " in unknown state");
-        }
-    }
-
-    std::size_t ReadersCache::getFileSize(std::size_t index)
-    {
-        const auto indexIt = mIndex.find(index);
-        if (indexIt == mIndex.end())
-            return 0;
-        switch (indexIt->second->mState)
-        {
-            case State::Busy:
-            case State::Free:
-                if (!indexIt->second->mReader.getName().empty())
-                    return indexIt->second->mReader.getFileSize();
-                throw std::logic_error("ESMReader at index " + std::to_string(index) + " has not been opened yet");
-            case State::Closed:
-                if (indexIt->second->mFileSize)
-                    return *indexIt->second->mFileSize;
-                throw std::logic_error("ESMReader at index " + std::to_string(index) + " has forgotten its file size");
-            default:
-                throw std::logic_error("ESMReader at index " + std::to_string(index) + " in unknown state");
-        }
-    }
-
    void ReadersCache::closeExtraReaders()
    {
        while (!mFreeItems.empty() && mBusyItems.size() + mFreeItems.size() + 1 > mCapacity)
@ -106,7 +65,6 @@ namespace ESM
            if (it->mReader.isOpen())
            {
                it->mName = it->mReader.getName();
-                it->mFileSize = it->mReader.getFileSize();
                it->mReader.close();
            }
            mClosedItems.splice(mClosedItems.end(), mFreeItems, it);
--- a/components/esm3/readerscache.hpp
+++ b/components/esm3/readerscache.hpp
@ -26,7 +26,6 @@ namespace ESM
            State mState = State::Busy;
            ESMReader mReader;
            std::optional<std::filesystem::path> mName;
-            std::optional<std::size_t> mFileSize;

            Item() = default;
        };
@ -56,10 +55,6 @@ namespace ESM

        BusyItem get(std::size_t index);

-        const std::filesystem::path& getName(std::size_t index) const;
-
-        std::size_t getFileSize(std::size_t index);
-
        void clear();

    private:
--- a/components/esm4/loadqust.cpp
+++ b/components/esm4/loadqust.cpp
@ -114,9 +114,6 @@ void ESM4::Quest::load(ESM4::Reader& reader)
            case ESM::fourCC("NNAM"): // FO3
            case ESM::fourCC("QOBJ"): // FO3
            case ESM::fourCC("NAM0"): // FO3
-            case ESM::fourCC("SLSD"): // FO3
-            case ESM::fourCC("SCVR"): // FO3
-            case ESM::fourCC("SCRV"): // FO3
            case ESM::fourCC("ANAM"): // TES5
            case ESM::fourCC("DNAM"): // TES5
            case ESM::fourCC("ENAM"): // TES5
--- a/components/fallback/validate.hpp
+++ b/components/fallback/validate.hpp
@ -24,6 +24,8 @@ namespace Fallback
    };

    // Parses and validates a fallback map from boost program_options.
+    // Note: for boost to pick up the validate function, you need to pull in the namespace e.g.
+    // by using namespace Fallback;
    void validate(boost::any& v, std::vector<std::string> const& tokens, FallbackMap*, int);
 }

--- a/components/files/windowspath.cpp
+++ b/components/files/windowspath.cpp
@ -36,14 +36,12 @@ namespace Files
    {
        std::filesystem::path userPath = std::filesystem::current_path();

-        PWSTR cString;
-        HRESULT result = SHGetKnownFolderPath(FOLDERID_Documents, 0, nullptr, &cString);
-        if (SUCCEEDED(result))
-            userPath = std::filesystem::path(cString);
-        else
-            Log(Debug::Error) << "Error " << result << " when getting Documents path";
+        WCHAR path[MAX_PATH + 1] = {};

-        CoTaskMemFree(cString);
+        if (SUCCEEDED(SHGetFolderPathW(nullptr, CSIDL_PERSONAL | CSIDL_FLAG_CREATE, nullptr, 0, path)))
+        {
+            userPath = std::filesystem::path(path);
+        }

        return userPath / "My Games" / mName;
    }
@ -56,19 +54,14 @@ namespace Files

    std::filesystem::path WindowsPath::getGlobalConfigPath() const
    {
-        // The concept of a global config path is absurd on Windows.
-        // Always use local config instead.
-        // The virtual base class requires that we provide this, though.
        std::filesystem::path globalPath = std::filesystem::current_path();

-        PWSTR cString;
-        HRESULT result = SHGetKnownFolderPath(FOLDERID_ProgramFiles, 0, nullptr, &cString);
-        if (SUCCEEDED(result))
-            globalPath = std::filesystem::path(cString);
-        else
-            Log(Debug::Error) << "Error " << result << " when getting Program Files path";
+        WCHAR path[MAX_PATH + 1] = {};

-        CoTaskMemFree(cString);
+        if (SUCCEEDED(SHGetFolderPathW(nullptr, CSIDL_PROGRAM_FILES | CSIDL_FLAG_CREATE, nullptr, 0, path)))
+        {
+            globalPath = std::filesystem::path(path);
+        }

        return globalPath / mName;
    }
--- a/components/fontloader/fontloader.cpp
+++ b/components/fontloader/fontloader.cpp
@ -227,10 +227,9 @@ namespace
 namespace Gui
 {

-    FontLoader::FontLoader(ToUTF8::FromType encoding, const VFS::Manager* vfs, float scalingFactor, bool exportFonts)
+    FontLoader::FontLoader(ToUTF8::FromType encoding, const VFS::Manager* vfs, float scalingFactor)
        : mVFS(vfs)
        , mScalingFactor(scalingFactor)
-        , mExportFonts(exportFonts)
    {
        if (encoding == ToUTF8::WINDOWS_1252)
            mEncoding = ToUTF8::CP437;
@ -408,8 +407,7 @@ namespace Gui
        file.reset();

        // Create the font texture
-        const std::string name(name_);
-        const std::string bitmapFilename = "fonts/" + name + ".tex";
+        std::string bitmapFilename = "fonts/" + std::string(name_) + ".tex";

        Files::IStreamPtr bitmapFile = mVFS->get(bitmapFilename);

@ -431,19 +429,6 @@ namespace Gui
                                << bitmapFile->gcount() << "/" << (width * height * 4) << " bytes)";
        bitmapFile.reset();

-        if (mExportFonts)
-        {
-            osg::ref_ptr<osg::Image> image = new osg::Image;
-            image->allocateImage(width, height, 1, GL_RGBA, GL_UNSIGNED_BYTE);
-            assert(image->isDataContiguous());
-            memcpy(image->data(), textureData.data(), textureData.size());
-            // Convert to OpenGL origin for sensible output
-            image->flipVertical();
-
-            Log(Debug::Info) << "Writing " << name + ".png";
-            osgDB::writeImageFile(*image, name + ".png");
-        }
-
        MyGUI::ITexture* tex = MyGUI::RenderManager::getInstance().createTexture(bitmapFilename);
        tex->createManual(width, height, MyGUI::TextureUsage::Write, MyGUI::PixelFormat::R8G8B8A8);
        unsigned char* texData = reinterpret_cast<unsigned char*>(tex->lock(MyGUI::TextureUsage::Write));
@ -641,13 +626,6 @@ namespace Gui
            code->addAttribute("size", "0 0");
        }

-        if (mExportFonts)
-        {
-            Log(Debug::Info) << "Writing " << name + ".xml";
-            xmlDocument.createDeclaration();
-            xmlDocument.save(name + ".xml");
-        }
-
        // Register the font with MyGUI
        MyGUI::ResourceManualFont* font = static_cast<MyGUI::ResourceManualFont*>(
            MyGUI::FactoryManager::getInstance().createObject("Resource", "ResourceManualFont"));
--- a/components/fontloader/fontloader.hpp
+++ b/components/fontloader/fontloader.hpp
@ -25,8 +25,7 @@ namespace Gui
    class FontLoader
    {
    public:
-        /// @param exportFonts export the converted fonts (Images and XML with glyph metrics) to files?
-        FontLoader(ToUTF8::FromType encoding, const VFS::Manager* vfs, float scalingFactor, bool exportFonts);
+        FontLoader(ToUTF8::FromType encoding, const VFS::Manager* vfs, float scalingFactor);

        void overrideLineHeight(MyGUI::xml::ElementPtr _node, std::string_view _file, MyGUI::Version _version);

@ -36,7 +35,6 @@ namespace Gui
        ToUTF8::FromType mEncoding;
        const VFS::Manager* mVFS;
        float mScalingFactor;
-        bool mExportFonts;

        void loadFonts();
        void loadFont(const std::string& fontName, const std::string& fontId);
--- a/components/lua/luastate.cpp
+++ b/components/lua/luastate.cpp
@ -11,7 +11,6 @@
 #include <components/files/conversion.hpp>
 #include <components/vfs/manager.hpp>

-#include "luastateptr.hpp"
 #include "scriptscontainer.hpp"
 #include "utf8.hpp"

@ -152,37 +151,37 @@ namespace LuaUtil
        return newPtr;
    }

-    LuaStatePtr LuaState::createLuaRuntime(LuaState* luaState)
+    lua_State* LuaState::createLuaRuntime(LuaState* luaState)
    {
        if (sProfilerEnabled)
        {
            Log(Debug::Info) << "Initializing LuaUtil::LuaState with profiler";
-            LuaStatePtr state(lua_newstate(&trackingAllocator, luaState));
-            if (state != nullptr)
-                return state;
-            sProfilerEnabled = false;
-            Log(Debug::Error) << "Failed to initialize LuaUtil::LuaState with custom allocator; disabling Lua profiler";
+            lua_State* L = lua_newstate(&trackingAllocator, luaState);
+            if (L)
+                return L;
+            else
+            {
+                sProfilerEnabled = false;
+                Log(Debug::Error)
+                    << "Failed to initialize LuaUtil::LuaState with custom allocator; disabling Lua profiler";
+            }
        }
        Log(Debug::Info) << "Initializing LuaUtil::LuaState without profiler";
-        LuaStatePtr state(luaL_newstate());
-        if (state == nullptr)
-            throw std::runtime_error("Failed to create Lua runtime");
-        return state;
+        lua_State* L = luaL_newstate();
+        if (!L)
+            throw std::runtime_error("Can't create Lua runtime");
+        return L;
    }

    LuaState::LuaState(const VFS::Manager* vfs, const ScriptsConfiguration* conf, const LuaStateSettings& settings)
        : mSettings(settings)
-        , mLuaState([&] {
-            LuaStatePtr state = createLuaRuntime(this);
-            sol::set_default_state(state.get());
-            return state;
-        }())
-        , mSol(mLuaState.get())
+        , mLuaHolder(createLuaRuntime(this))
+        , mSol(mLuaHolder.get())
        , mConf(conf)
        , mVFS(vfs)
    {
        if (sProfilerEnabled)
-            lua_sethook(mLuaState.get(), &countHook, LUA_MASKCOUNT, countHookStep);
+            lua_sethook(mLuaHolder.get(), &countHook, LUA_MASKCOUNT, countHookStep);

        protectedCall([&](LuaView& view) {
            auto& sol = view.sol();
--- a/components/lua/luastate.hpp
+++ b/components/lua/luastate.hpp
@ -10,7 +10,6 @@
 #include <components/vfs/pathutil.hpp>

 #include "configuration.hpp"
-#include "luastateptr.hpp"

 namespace VFS
 {
@ -189,7 +188,7 @@ namespace LuaUtil
        static void countHook(lua_State* L, lua_Debug* ar);
        static void* trackingAllocator(void* ud, void* ptr, size_t osize, size_t nsize);

-        static LuaStatePtr createLuaRuntime(LuaState* luaState);
+        lua_State* createLuaRuntime(LuaState* luaState);

        struct AllocOwner
        {
@ -207,8 +206,25 @@ namespace LuaUtil
        uint64_t mSmallAllocMemoryUsage = 0;
        std::vector<int64_t> mMemoryUsage;

+        class LuaStateHolder
+        {
+        public:
+            LuaStateHolder(lua_State* L)
+                : L(L)
+            {
+                sol::set_default_state(L);
+            }
+            ~LuaStateHolder() { lua_close(L); }
+            LuaStateHolder(const LuaStateHolder&) = delete;
+            LuaStateHolder(LuaStateHolder&&) = delete;
+            lua_State* get() { return L; }
+
+        private:
+            lua_State* L;
+        };
+
        // Must be declared before mSol and all sol-related objects. Then on exit it will be destructed the last.
-        LuaStatePtr mLuaState;
+        LuaStateHolder mLuaHolder;

        sol::state_view mSol;
        const ScriptsConfiguration* mConf;
--- a/components/lua/luastateptr.hpp
+++ b/components/lua/luastateptr.hpp
@ -1,18 +0,0 @@
-#ifndef OPENMW_COMPONENTS_LUA_LUASTATEPTR_H
-#define OPENMW_COMPONENTS_LUA_LUASTATEPTR_H
-
-#include <sol/state.hpp>
-
-#include <memory>
-
-namespace LuaUtil
-{
-    struct CloseLuaState
-    {
-        void operator()(lua_State* state) noexcept { lua_close(state); }
-    };
-
-    using LuaStatePtr = std::unique_ptr<lua_State, CloseLuaState>;
-}
-
-#endif
--- a/components/nifosg/nifloader.cpp
+++ b/components/nifosg/nifloader.cpp
@ -391,7 +391,6 @@ namespace NifOsg
                osg::ref_ptr<SceneUtil::Skeleton> skel = new SceneUtil::Skeleton;
                skel->setStateSet(created->getStateSet());
                skel->setName(created->getName());
-                skel->setUserDataContainer(created->getUserDataContainer());
                for (unsigned int i = 0; i < created->getNumChildren(); ++i)
                    skel->addChild(created->getChild(i));
                created->removeChildren(0, created->getNumChildren());
--- a/components/sceneutil/lightutil.cpp
+++ b/components/sceneutil/lightutil.cpp
@ -119,8 +119,7 @@ namespace SceneUtil
        osg::ref_ptr<osg::Light> light(new osg::Light);
        lightSource->setNodeMask(lightMask);

-        // The minimum scene light radius is 16 in Morrowind
-        const float radius = std::max(esmLight.mRadius, 16.f);
+        float radius = esmLight.mRadius;
        lightSource->setRadius(radius);

        configureLight(light, radius, isExterior);
--- a/components/stereo/frustum.hpp
+++ b/components/stereo/frustum.hpp
@ -2,7 +2,10 @@
 #define STEREO_FRUSTUM_H

 #include <osg/BoundingBox>
+#include <osg/Camera>
 #include <osg/Matrix>
+#include <osg/StateSet>
+#include <osg/Vec3>

 #include <array>
 #include <map>
@ -10,7 +13,6 @@

 namespace osg
 {
-    class Camera;
    class FrameBufferObject;
    class Texture2D;
    class Texture2DMultisample;
@ -22,7 +24,7 @@ namespace osgViewer
    class Viewer;
 }

-namespace osgUtil
+namespace usgUtil
 {
    class CullVisitor;
 }
--- a/components/testing/util.hpp
+++ b/components/testing/util.hpp
@ -1,7 +1,6 @@
 #ifndef OPENMW_COMPONENTS_TESTING_UTIL_H
 #define OPENMW_COMPONENTS_TESTING_UTIL_H

-#include <chrono>
 #include <filesystem>
 #include <initializer_list>
 #include <memory>
@ -15,37 +14,26 @@

 namespace TestingOpenMW
 {
-    inline std::filesystem::path outputDir()
+    inline std::filesystem::path outputFilePath(const std::string name)
    {
-        static const std::string run
-            = std::to_string(std::chrono::system_clock::to_time_t(std::chrono::system_clock::now()));
-        std::filesystem::path dir = std::filesystem::temp_directory_path() / "openmw" / "tests" / run;
-        std::filesystem::create_directories(dir);
-        return dir;
-    }
-
-    inline std::filesystem::path outputFilePath(std::string_view name)
-    {
-        std::filesystem::path dir = outputDir();
+        std::filesystem::path dir("tests_output");
+        std::filesystem::create_directory(dir);
        return dir / Misc::StringUtils::stringToU8String(name);
    }

-    inline std::filesystem::path outputDirPath(const std::filesystem::path& subpath)
-    {
-        std::filesystem::path path = outputDir();
-        path /= subpath;
-        std::filesystem::create_directories(path);
-        return path;
-    }
-
    inline std::filesystem::path outputFilePathWithSubDir(const std::filesystem::path& subpath)
    {
-        std::filesystem::path path = outputDir();
+        std::filesystem::path path("tests_output");
        path /= subpath;
        std::filesystem::create_directories(path.parent_path());
        return path;
    }

+    inline std::filesystem::path temporaryFilePath(const std::string name)
+    {
+        return std::filesystem::temp_directory_path() / name;
+    }
+
    class VFSTestFile : public VFS::File
    {
    public:
--- a/docs/source/manuals/installation/install-openmw.rst
+++ b/docs/source/manuals/installation/install-openmw.rst
@ -28,8 +28,13 @@ A `Launchpad PPA <https://launchpad.net/~openmw/+archive/openmw>`_ is available.
 Add it and install OpenMW::

 	$ sudo add-apt-repository ppa:openmw/openmw
-	$ sudo apt update
-	$ sudo apt install openmw
+	$ sudo apt-get update
+	$ sudo apt-get install openmw openmw-launcher
+
+.. note::
+	OpenMW-CS must be installed separately by typing::
+
+		$ sudo apt-get install openmw-cs

 The Arch Linux Way
 ==================
--- a/docs/source/reference/modding/font.rst
+++ b/docs/source/reference/modding/font.rst
@ -15,8 +15,6 @@ Morrowind .fnt fonts
 Morrowind uses a custom ``.fnt`` file format. It is not compatible with the Windows Font File ``.fnt`` format.
 To our knowledge, the format is undocumented. OpenMW can load this format and convert it on the fly into something usable
 (see font loader `source code <https://gitlab.com/OpenMW/openmw/blob/master/components/fontloader/fontloader.cpp>`_).
-You can use --export-fonts command line option to write the converted font
-(a PNG image and an XML file describing the position of each glyph in the image) to the current directory.

 They can be used instead of TrueType fonts if needed by specifying their ``.fnt`` files names in the ``openmw.cfg``. For example:

--- a/extern/.clang-tidy
+++ b/extern/.clang-tidy
@ -1,3 +0,0 @@
-Checks: >-
-  -clang-analyzer-core.NullDereference,
-  -clang-analyzer-cplusplus.NewDelete
--- a/files/data/fonts/MysticCards.omwfont
+++ b/files/data/fonts/MysticCards.omwfont
@ -2,13 +2,13 @@
    <Resource type="ResourceTrueTypeFont">
        <Property key="Source" value="MysticCards.ttf"/>
        <Property key="Antialias" value="false"/>
-        <Property key="SubstituteCode" value="95"/>
+        <Property key="SubstituteCode" value="63"/>
        <Property key="TabWidth" value="8"/>
        <Property key="OffsetHeight" value="0"/>
        <Property key="Resolution" value="70"/>
        <Codes>
            <Code range="33 126"/>
-            <Code range="160 161"/>
+            <Code range="161"/>
            <Code range="173"/>
            <Code range="175 177"/>
            <Code range="180"/>
--- a/files/data/fonts/MysticCards.ttf
+++ b/files/data/fonts/MysticCards.ttf
--- a/files/data/scripts/omw/crimes.lua
+++ b/files/data/scripts/omw/crimes.lua
@ -24,7 +24,7 @@ return {
    interface = {
        --- Interface version
        -- @field [parent=#Crimes] #number version
-        version = 2,
+        version = 1,

        ---
        -- Commits a crime as if done through an in-game action. Can only be used in global context.
@ -42,7 +42,7 @@ return {
                "faction id passed to commitCrime must be a string or nil")
            assert(type(options.arg) == "number" or options.arg == nil,
                "arg value passed to commitCrime must be a number or nil")
-            assert(type(options.victimAware) == "boolean" or options.victimAware == nil,
+            assert(type(options.victimAware) == "number" or options.victimAware == nil,
                "victimAware value passed to commitCrime must be a boolean or nil")

            assert(options.type ~= nil, "crime type passed to commitCrime cannot be nil")
--- a/files/lua_api/openmw/animation.lua
+++ b/files/lua_api/openmw/animation.lua
@ -221,14 +221,14 @@
 -- Can be used only in local scripts on self. Can also be evoked by sending an AddVfx event to the target actor.
 -- @function [parent=#animation] addVfx
 -- @param openmw.core#GameObject actor
-- @param #string model path (normally taken from a record such as @{openmw.types#StaticRecord.model} or similar)
+-- @param #string model #string model path (normally taken from a record such as @{openmw.types#StaticRecord.model} or similar)
 -- @param #table options optional table of parameters. Can contain:
 --
--   * `loop` - boolean, if true the effect will loop until removed (default: false).
+--   * `loop` - boolean, if true the effect will loop until removed (default: 0).
 --   * `boneName` - name of the bone to attach the vfx to. (default: "")
 --   * `particleTextureOverride` - name of the particle texture to use. (default: "")
 --   * `vfxId` - a string ID that can be used to remove the effect later, using #removeVfx, and to avoid duplicate effects. The default value of "" can have duplicates. To avoid interaction with the engine, use unique identifiers unrelated to magic effect IDs. The engine uses this identifier to add and remove magic effects based on what effects are active on the actor. If this is set equal to the @{openmw.core#MagicEffectId} identifier of the magic effect being added, for example core.magic.EFFECT_TYPE.FireDamage, then the engine will remove it once the fire damage effect on the actor reaches 0. (Default: "").
--   * `useAmbientLighting` - boolean, vfx get a white ambient light attached in Morrowind. If false don't attach this. (default: true)
+--   * `useAmbientLighting` - boolean, vfx get a white ambient light attached in Morrowind. If false don't attach this. (default: 1)
 --
 -- @usage local mgef = core.magic.effects.records[myEffectName]
 -- anim.addVfx(self, 'VFX_Hands', {boneName = 'Bip01 L Hand', particleTextureOverride = mgef.particle, loop = mgef.continuousVfx, vfxId = mgef.id..'_myuniquenamehere'})
--- a/files/lua_api/openmw/nearby.lua
+++ b/files/lua_api/openmw/nearby.lua
@ -59,7 +59,7 @@
 -- @field [parent=#nearby] #COLLISION_TYPE COLLISION_TYPE

 ---
-- Result of raycasting
+-- Result of raycasing
 -- @type RayCastingResult
 -- @field [parent=#RayCastingResult] #boolean hit Is there a collision? (true/false)
 -- @field [parent=#RayCastingResult] openmw.util#Vector3 hitPos Position of the collision point (nil if no collision)
@ -69,7 +69,7 @@
 ---
 -- A table of parameters for @{#nearby.castRay}
 -- @type CastRayOptions
-- @field #any ignore An @{openmw.core#GameObject} or @{openmw.core#ObjectList} to ignore (specify here the source of the ray, or other objects which should not collide)
+-- @field openmw.core#GameObject ignore An object to ignore (specify here the source of the ray)
 -- @field #number collisionType Object types to work with (see @{openmw.nearby#COLLISION_TYPE})
 -- @field #number radius The radius of the ray (zero by default). If not zero then castRay actually casts a sphere with given radius.
 --  NOTE: currently `ignore` is not supported if `radius>0`.
@ -92,7 +92,7 @@
 ---
 -- A table of parameters for @{#nearby.castRenderingRay} and @{#nearby.asyncCastRenderingRay}
 -- @type CastRenderingRayOptions
-- @field #any ignore A @{openmw.core#GameObject} or @{openmw.core#ObjectList} to ignore while doing the ray cast
+-- @field #table ignore A list of @{openmw.core#GameObject} to ignore while doing the ray cast

 ---
 -- Cast ray from one point to another and find the first visual intersection with anything in the scene.
@ -102,7 +102,7 @@
 -- @function [parent=#nearby] castRenderingRay
 -- @param openmw.util#Vector3 from Start point of the ray.
 -- @param openmw.util#Vector3 to End point of the ray.
-- @param #CastRenderingRayOptions options An optional table with additional optional arguments
+-- @param #CastRenderingRayOptions
 -- @return #RayCastingResult

 ---
--- a/files/lua_api/openmw/self.lua
+++ b/files/lua_api/openmw/self.lua
@ -45,7 +45,7 @@
 -- @field [parent=#ActorControls] #boolean run true - run, false - walk
 -- @field [parent=#ActorControls] #boolean sneak If true - sneak
 -- @field [parent=#ActorControls] #boolean jump If true - initiate a jump
-- @field [parent=#ActorControls] #ATTACK_TYPE use Activates the readied weapon/spell according to a provided value. For weapons, keeping this value modified will charge the attack until set to @{#ATTACK_TYPE.NoAttack}. If an @{#ATTACK_TYPE} not appropriate for a currently equipped weapon provided - an appropriate @{#ATTACK_TYPE} will be used instead.
+-- @field [parent=#ActorControls] #ATTACK_TYPE use Activates the readied weapon/spell according to a provided value. For weapons, keeping this value modified will charge the attack until set to @{#ATTACK_TYPE.NoAttack}. If an @#ATTACK_TYPE} not appropriate for a currently equipped weapon provided - an appropriate @{#ATTACK_TYPE} will be used instead.

 ---
 -- Enables or disables standard AI (enabled by default).
--- a/files/lua_api/openmw/types.lua
+++ b/files/lua_api/openmw/types.lua
@ -1123,7 +1123,7 @@
 -- @field #string id The record ID of the NPC
 -- @field #string name
 -- @field #string race
-- @field #string class ID of the NPC's class (e.g. acrobat)
+-- @field #string class Name of the NPC's class (e. g. Acrobat)
 -- @field #string model Path to the model associated with this NPC, used for animations.
 -- @field #string mwscript MWScript on this NPC (can be nil)
 -- @field #string hair Path to the hair body part model
--- a/scripts/data/integration_tests/test_lua_api/menu.lua
+++ b/scripts/data/integration_tests/test_lua_api/menu.lua
@ -1,94 +0,0 @@
-local testing = require('testing_util')
-local matchers = require('matchers')
-local menu = require('openmw.menu')
-
-testing.registerMenuTest('save and load', function()
-    menu.newGame()
-    coroutine.yield()
-    menu.saveGame('save and load')
-    coroutine.yield()
-
-    local directorySaves = {}
-    directorySaves['save_and_load.omwsave'] = {
-        playerName = '',
-        playerLevel = 1,
-        timePlayed = 0,
-        description = 'save and load',
-        contentFiles = {
-            'builtin.omwscripts',
-            'template.omwgame',
-            'landracer.omwaddon',
-            'the_hub.omwaddon',
-            'test_lua_api.omwscripts',
-        },
-        creationTime = matchers.isAny(),
-    }
-    local expectedAllSaves = {}
-    expectedAllSaves[' - 1'] = directorySaves
-
-    testing.expectThat(menu.getAllSaves(), matchers.equalTo(expectedAllSaves))
-
-    menu.loadGame(' - 1', 'save_and_load.omwsave')
-    coroutine.yield()
-
-    menu.deleteGame(' - 1', 'save_and_load.omwsave')
-    testing.expectThat(menu.getAllSaves(), matchers.equalTo({}))
-end)
-
-testing.registerMenuTest('load while teleporting', function()
-    menu.newGame()
-    coroutine.yield()
-
-    testing.runGlobalTest('load while teleporting - init player')
-
-    menu.saveGame('load while teleporting')
-    coroutine.yield()
-
-    testing.runGlobalTest('load while teleporting - teleport')
-
-    menu.loadGame(' - 1', 'load_while_teleporting.omwsave')
-    coroutine.yield()
-
-    menu.deleteGame(' - 1', 'load_while_teleporting.omwsave')
-end)
-
-local function registerGlobalTest(name, description)
-   testing.registerMenuTest(description or name, function()
-       menu.newGame()
-       coroutine.yield()
-       testing.runGlobalTest(name)
-   end)
-end
-
-registerGlobalTest('timers')
-registerGlobalTest('teleport')
-registerGlobalTest('getGMST')
-registerGlobalTest('MWScript')
-registerGlobalTest('record stores')
-registerGlobalTest('record creation')
-registerGlobalTest('UTF-8 characters')
-registerGlobalTest('UTF-8 strings')
-registerGlobalTest('memory limit')
-registerGlobalTest('vfs')
-registerGlobalTest('commit crime')
-registerGlobalTest('record model property')
-
-registerGlobalTest('player yaw rotation', 'rotating player with controls.yawChange should change rotation')
-registerGlobalTest('player pitch rotation', 'rotating player with controls.pitchChange should change rotation')
-registerGlobalTest('player pitch and yaw rotation', 'rotating player with controls.pitchChange and controls.yawChange should change rotation')
-registerGlobalTest('player rotation', 'rotating player should not lead to nan rotation')
-registerGlobalTest('player forward running')
-registerGlobalTest('player diagonal walking')
-registerGlobalTest('findPath')
-registerGlobalTest('findRandomPointAroundCircle')
-registerGlobalTest('castNavigationRay')
-registerGlobalTest('findNearestNavMeshPosition')
-registerGlobalTest('player memory limit')
-registerGlobalTest('player weapon attack', 'player with equipped weapon on attack should damage health of other actors')
-
-return {
-    engineHandlers = {
-        onFrame = testing.makeUpdateMenu(),
-    },
-    eventHandlers = testing.menuEventHandlers,
-}
--- a/scripts/data/integration_tests/test_lua_api/openmw.cfg
+++ b/scripts/data/integration_tests/test_lua_api/openmw.cfg
@ -1,4 +1,4 @@
-content=test_lua_api.omwscripts
+content=test.omwscripts

 # Needed to test `core.getGMST`
 fallback=Water_RippleFrameCount,4
--- a/scripts/data/integration_tests/test_lua_api/player.lua
+++ b/scripts/data/integration_tests/test_lua_api/player.lua
@ -6,7 +6,6 @@ local input = require('openmw.input')
 local types = require('openmw.types')
 local nearby = require('openmw.nearby')
 local camera = require('openmw.camera')
-local matchers = require('matchers')

 types.Player.setControlSwitch(self, types.Player.CONTROL_SWITCH.Controls, false)
 types.Player.setControlSwitch(self, types.Player.CONTROL_SWITCH.Fighting, false)
@ -41,7 +40,7 @@ local function rotateByPitch(object, target)
    rotate(object, target, nil)
 end

-testing.registerLocalTest('player yaw rotation',
+testing.registerLocalTest('playerYawRotation',
    function()
        local initialAlphaXZ, initialGammaXZ = self.rotation:getAnglesXZ()
        local initialAlphaZYX, initialBetaZYX, initialGammaZYX = self.rotation:getAnglesZYX()
@ -61,7 +60,7 @@ testing.registerLocalTest('player yaw rotation',
        testing.expectEqualWithDelta(gamma2, initialGammaZYX, 0.05, 'Gamma rotation in ZYX convention should not change')
    end)

-testing.registerLocalTest('player pitch rotation',
+testing.registerLocalTest('playerPitchRotation',
    function()
        local initialAlphaXZ, initialGammaXZ = self.rotation:getAnglesXZ()
        local initialAlphaZYX, initialBetaZYX, initialGammaZYX = self.rotation:getAnglesZYX()
@ -81,7 +80,7 @@ testing.registerLocalTest('player pitch rotation',
        testing.expectEqualWithDelta(gamma2, targetPitch, 0.05, 'Incorrect gamma rotation in ZYX convention')
    end)

-testing.registerLocalTest('player pitch and yaw rotation',
+testing.registerLocalTest('playerPitchAndYawRotation',
    function()
        local targetPitch = math.rad(-30)
        local targetYaw = math.rad(-60)
@ -100,7 +99,7 @@ testing.registerLocalTest('player pitch and yaw rotation',
        testing.expectEqualWithDelta(gamma2, math.rad(-16), 0.05, 'Incorrect gamma rotation in ZYX convention')
    end)

-testing.registerLocalTest('player rotation',
+testing.registerLocalTest('playerRotation',
    function()
        local rotation = math.sqrt(2)
        local endTime = core.getSimulationTime() + 3
@ -114,17 +113,17 @@ testing.registerLocalTest('player rotation',
            coroutine.yield()

            local alpha1, gamma1 = self.rotation:getAnglesXZ()
-            testing.expectThat(alpha1, matchers.isNotNan(), 'Alpha rotation in XZ convention is nan')
-            testing.expectThat(gamma1, matchers.isNotNan(), 'Gamma rotation in XZ convention is nan')
+            testing.expectThat(alpha1, testing.isNotNan(), 'Alpha rotation in XZ convention is nan')
+            testing.expectThat(gamma1, testing.isNotNan(), 'Gamma rotation in XZ convention is nan')

            local alpha2, beta2, gamma2 = self.rotation:getAnglesZYX()
-            testing.expectThat(alpha2, matchers.isNotNan(), 'Alpha rotation in ZYX convention is nan')
-            testing.expectThat(beta2, matchers.isNotNan(), 'Beta rotation in ZYX convention is nan')
-            testing.expectThat(gamma2, matchers.isNotNan(), 'Gamma rotation in ZYX convention is nan')
+            testing.expectThat(alpha2, testing.isNotNan(), 'Alpha rotation in ZYX convention is nan')
+            testing.expectThat(beta2, testing.isNotNan(), 'Beta rotation in ZYX convention is nan')
+            testing.expectThat(gamma2, testing.isNotNan(), 'Gamma rotation in ZYX convention is nan')
        end
    end)

-testing.registerLocalTest('player forward running',
+testing.registerLocalTest('playerForwardRunning',
    function()
        local startPos = self.position
        local endTime = core.getSimulationTime() + 1
@ -142,7 +141,7 @@ testing.registerLocalTest('player forward running',
        testing.expectEqualWithDelta(direction.y, 1, 0.1, 'Run forward, Y coord')
    end)

-testing.registerLocalTest('player diagonal walking',
+testing.registerLocalTest('playerDiagonalWalking',
    function()
        local startPos = self.position
        local endTime = core.getSimulationTime() + 1
@ -221,7 +220,7 @@ testing.registerLocalTest('findNearestNavMeshPosition',
            'Navigation mesh position ' .. testing.formatActualExpected(result, expected))
    end)

-testing.registerLocalTest('player memory limit',
+testing.registerLocalTest('playerMemoryLimit',
    function()
        local ok, err = pcall(function()
            local str = 'a'
@ -233,7 +232,7 @@ testing.registerLocalTest('player memory limit',
        testing.expectEqual(err, 'not enough memory')
    end)

-testing.registerLocalTest('player weapon attack',
+testing.registerLocalTest('playerWeaponAttack',
    function()
        camera.setMode(camera.MODE.ThirdPerson)

@ -347,5 +346,5 @@ return {
    engineHandlers = {
        onFrame = testing.updateLocal,
    },
-    eventHandlers = testing.localEventHandlers,
+    eventHandlers = testing.eventHandlers
 }
--- a/scripts/data/integration_tests/test_lua_api/global.lua
+++ b/scripts/data/integration_tests/test_lua_api/global.lua
@ -7,7 +7,7 @@ local vfs = require('openmw.vfs')
 local world = require('openmw.world')
 local I = require('openmw.interfaces')

-testing.registerGlobalTest('timers', function()
+local function testTimers()
    testing.expectAlmostEqual(core.getGameTimeScale(), 30, 'incorrect getGameTimeScale() result')
    testing.expectAlmostEqual(core.getSimulationTimeScale(), 1, 'incorrect getSimulationTimeScale result')

@ -39,10 +39,9 @@ testing.registerGlobalTest('timers', function()
    testing.expectGreaterOrEqual(ts1, 0.5, 'async:newSimulationTimer failed')
    testing.expectGreaterOrEqual(th2, 72, 'async:newUnsavableGameTimer failed')
    testing.expectGreaterOrEqual(ts2, 1, 'async:newUnsavableSimulationTimer failed')
-end)
+end

-testing.registerGlobalTest('teleport', function()
-    local player = world.players[1]
+local function testTeleport()
    player:teleport('', util.vector3(100, 50, 500), util.transform.rotateZ(math.rad(90)))
    coroutine.yield()
    testing.expect(player.cell.isExterior, 'teleport to exterior failed')
@ -72,16 +71,16 @@ testing.registerGlobalTest('teleport', function()
    testing.expectEqualWithDelta(player.position.x, 50, 1, 'incorrect position after teleporting')
    testing.expectEqualWithDelta(player.position.y, -100, 1, 'incorrect position after teleporting')
    testing.expectEqualWithDelta(player.rotation:getYaw(), math.rad(-90), 0.05, 'teleporting changes rotation')
-end)
+end

-testing.registerGlobalTest('getGMST', function()
+local function testGetGMST()
    testing.expectEqual(core.getGMST('non-existed gmst'), nil)
    testing.expectEqual(core.getGMST('Water_RippleFrameCount'), 4)
    testing.expectEqual(core.getGMST('Inventory_DirectionalDiffuseR'), 0.5)
    testing.expectEqual(core.getGMST('Level_Up_Level2'), 'something')
-end)
+end

-testing.registerGlobalTest('MWScript', function()
+local function testMWScript()
    local variableStoreCount = 18
    local variableStore = world.mwscript.getGlobalVariables(player)
    testing.expectEqual(variableStoreCount, #variableStore)
@ -101,7 +100,7 @@ testing.registerGlobalTest('MWScript', function()
        indexCheck = indexCheck + 1
    end
    testing.expectEqual(variableStoreCount, indexCheck)
-end)
+end

 local function testRecordStore(store, storeName, skipPairs)
    testing.expect(store.records)
@ -122,7 +121,7 @@ local function testRecordStore(store, storeName, skipPairs)
    testing.expectEqual(status, true, storeName)
 end

-testing.registerGlobalTest('record stores', function()
+local function testRecordStores()
    for key, type in pairs(types) do
        if type.records then
            testRecordStore(type, key)
@ -141,9 +140,9 @@ testing.registerGlobalTest('record stores', function()
    testRecordStore(types.NPC.classes, "classes")
    testRecordStore(types.NPC.races, "races")
    testRecordStore(types.Player.birthSigns, "birthSigns")
-end)
+end

-testing.registerGlobalTest('record creation', function()
+local function testRecordCreation()
    local newLight = {
        isCarriable = true,
        isDynamic = true,
@ -166,9 +165,9 @@ testing.registerGlobalTest('record creation', function()
    for key, value in pairs(newLight) do
        testing.expectEqual(record[key], value)
    end
-end)
+end

-testing.registerGlobalTest('UTF-8 characters', function()
+local function testUTF8Chars()
    testing.expectEqual(utf8.codepoint("😀"), 0x1F600)

    local chars = {}
@ -193,9 +192,9 @@ testing.registerGlobalTest('UTF-8 characters', function()
        testing.expectEqual(utf8.codepoint(char), codepoint)
        testing.expectEqual(utf8.len(char), 1)
    end
-end)
+end

-testing.registerGlobalTest('UTF-8 strings', function()
+local function testUTF8Strings()
    local utf8str = "Hello, 你好, 🌎!"

    local str = ""
@ -206,9 +205,9 @@ testing.registerGlobalTest('UTF-8 strings', function()

    testing.expectEqual(utf8.len(utf8str), 13)
    testing.expectEqual(utf8.offset(utf8str, 9), 11)
-end)
+end

-testing.registerGlobalTest('memory limit', function()
+local function testMemoryLimit()
    local ok, err = pcall(function()
        local t = {}
        local n = 1
@ -219,16 +218,14 @@ testing.registerGlobalTest('memory limit', function()
    end)
    testing.expectEqual(ok, false, 'Script reaching memory limit should fail')
    testing.expectEqual(err, 'not enough memory')
-end)
-
-local function initPlayer()
-    local player = world.players[1]
-    player:teleport('', util.vector3(4096, 4096, 1745), util.transform.identity)
-    coroutine.yield()
-    return player
 end

-testing.registerGlobalTest('vfs', function()
+local function initPlayer()
+    player:teleport('', util.vector3(4096, 4096, 1745), util.transform.identity)
+    coroutine.yield()
+end
+
+local function testVFS()
    local file = 'test_vfs_dir/lines.txt'
    local nosuchfile = 'test_vfs_dir/nosuchfile'
    testing.expectEqual(vfs.fileExists(file), true, 'lines.txt should exist')
@ -272,11 +269,12 @@ testing.registerGlobalTest('vfs', function()
    for _,v in pairs(expectedLines) do
        testing.expectEqual(getLine(), v)
    end
-end)
+end

-testing.registerGlobalTest('commit crime', function()
-    local player = initPlayer()
-    testing.expectEqual(player == nil, false, 'A viable player reference should exist to run `commit crime`')
+local function testCommitCrime()
+    initPlayer()
+    local player = world.players[1]
+    testing.expectEqual(player == nil, false, 'A viable player reference should exist to run `testCommitCrime`')
    testing.expectEqual(I.Crimes == nil, false, 'Crimes interface should be available in global contexts')

    -- Reset crime level to have a clean slate
@ -294,59 +292,82 @@ testing.registerGlobalTest('commit crime', function()
    types.Player.setCrimeLevel(player, 0)
    testing.expectEqual(I.Crimes.commitCrime(player, { victim = victim, type = types.Player.OFFENSE_TYPE.Theft, arg = 50 }).wasCrimeSeen, true, "Running a crime with a valid victim should notify them when the player is not sneaking, even if it's not explicitly passed in")
    testing.expectEqual(types.Player.getCrimeLevel(player), 0, "Crime level should not change if the victim's alarm value is low and there's no other witnesses")
-end)
-
-testing.registerGlobalTest('record model property', function()
-    local player = world.players[1]
-    testing.expectEqual(types.NPC.record(player).model, 'meshes/basicplayer.dae')
-end)
-
-local function registerPlayerTest(name)
-    testing.registerGlobalTest(name, function()
-        local player = initPlayer()
-        testing.runLocalTest(player, name)
-    end)
 end

-registerPlayerTest('player yaw rotation')
-registerPlayerTest('player pitch rotation')
-registerPlayerTest('player pitch and yaw rotation')
-registerPlayerTest('player rotation')
-registerPlayerTest('player forward running')
-registerPlayerTest('player diagonal walking')
-registerPlayerTest('findPath')
-registerPlayerTest('findRandomPointAroundCircle')
-registerPlayerTest('castNavigationRay')
-registerPlayerTest('findNearestNavMeshPosition')
-registerPlayerTest('player memory limit')
-
-testing.registerGlobalTest('player weapon attack', function()
-    local player = initPlayer()
-    world.createObject('basic_dagger1h', 1):moveInto(player)
-    testing.runLocalTest(player, 'player weapon attack')
-end)
-
-testing.registerGlobalTest('load while teleporting - init player', function()
+local function testRecordModelProperty()
+    initPlayer()
    local player = world.players[1]
-    player:teleport('Museum of Wonders', util.vector3(0, -1500, 111), util.transform.rotateZ(math.rad(180)))
-end)
+    testing.expectEqual(types.NPC.record(player).model, 'meshes/basicplayer.dae')
+end

-testing.registerGlobalTest('load while teleporting - teleport', function()
-    local player = world.players[1]
-    local landracer = world.createObject('landracer')
-    landracer:teleport(player.cell, player.position + util.vector3(0, 500, 0))
-    coroutine.yield()
-
-    local door = world.getObjectByFormId(core.getFormId('the_hub.omwaddon', 26))
-    door:activateBy(player)
-    coroutine.yield()
-
-    landracer:teleport(player.cell, player.position)
-end)
+tests = {
+    {'timers', testTimers},
+    {'rotating player with controls.yawChange should change rotation', function()
+        initPlayer()
+        testing.runLocalTest(player, 'playerYawRotation')
+    end},
+    {'rotating player with controls.pitchChange should change rotation', function()
+        initPlayer()
+        testing.runLocalTest(player, 'playerPitchRotation')
+    end},
+    {'rotating player with controls.pitchChange and controls.yawChange should change rotation', function()
+        initPlayer()
+        testing.runLocalTest(player, 'playerPitchAndYawRotation')
+    end},
+    {'rotating player should not lead to nan rotation', function()
+        initPlayer()
+        testing.runLocalTest(player, 'playerRotation')
+    end},
+    {'playerForwardRunning', function()
+        initPlayer()
+        testing.runLocalTest(player, 'playerForwardRunning')
+    end},
+    {'playerDiagonalWalking', function()
+        initPlayer()
+        testing.runLocalTest(player, 'playerDiagonalWalking')
+    end},
+    {'findPath', function()
+        initPlayer()
+        testing.runLocalTest(player, 'findPath')
+    end},
+    {'findRandomPointAroundCircle', function()
+        initPlayer()
+        testing.runLocalTest(player, 'findRandomPointAroundCircle')
+    end},
+    {'castNavigationRay', function()
+        initPlayer()
+        testing.runLocalTest(player, 'castNavigationRay')
+    end},
+    {'findNearestNavMeshPosition', function()
+        initPlayer()
+        testing.runLocalTest(player, 'findNearestNavMeshPosition')
+    end},
+    {'teleport', testTeleport},
+    {'getGMST', testGetGMST},
+    {'recordStores', testRecordStores},
+    {'recordCreation', testRecordCreation},
+    {'utf8Chars', testUTF8Chars},
+    {'utf8Strings', testUTF8Strings},
+    {'mwscript', testMWScript},
+    {'testMemoryLimit', testMemoryLimit},
+    {'playerMemoryLimit', function()
+        initPlayer()
+        testing.runLocalTest(player, 'playerMemoryLimit')
+    end},
+    {'player with equipped weapon on attack should damage health of other actors', function()
+        initPlayer()
+        world.createObject('basic_dagger1h', 1):moveInto(player)
+        testing.runLocalTest(player, 'playerWeaponAttack')
+    end},
+    {'vfs', testVFS},
+    {'testCommitCrime', testCommitCrime},
+    {'recordModelProperty', testRecordModelProperty},
+}

 return {
    engineHandlers = {
-        onUpdate = testing.updateGlobal,
+        onUpdate = testing.testRunner(tests),
+        onPlayerAdded = function(p) player = p end,
    },
-    eventHandlers = testing.globalEventHandlers,
+    eventHandlers = testing.eventHandlers,
 }
--- a/scripts/data/integration_tests/test_lua_api/test.omwscripts
+++ b/scripts/data/integration_tests/test_lua_api/test.omwscripts
@ -0,0 +1,2 @@
+GLOBAL: test.lua
+PLAYER: player.lua
--- a/scripts/data/integration_tests/test_lua_api/test_lua_api.omwscripts
+++ b/scripts/data/integration_tests/test_lua_api/test_lua_api.omwscripts
@ -1,3 +0,0 @@
-MENU: menu.lua
-GLOBAL: global.lua
-PLAYER: player.lua
--- a/scripts/data/integration_tests/testing_util/matchers.lua
+++ b/scripts/data/integration_tests/testing_util/matchers.lua
@ -1,218 +0,0 @@
-local module = {}
-
---
-- Matcher verifying that distance between given value and expected is not greater than maxDistance.
-- @function elementsAreArray
-- @param expected#vector.
-- @usage
-- expectThat(util.vector2(0, 0), closeToVector(util.vector2(0, 1), 1))
-function module.closeToVector(expected, maxDistance)
-    return function(actual)
-        local distance = (expected - actual):length()
-        if distance <= maxDistance then
-            return ''
-        end
-        return string.format('%s is too far from expected %s: %s > %s', actual, expected, distance, maxDistance)
-    end
-end
-
---
-- Matcher verifying that given value is an array each element of which matches elements of expected.
-- @function elementsAreArray
-- @param expected#array of values or matcher functions.
-- @usage
-- local t = {42, 13}
-- local matcher = function(actual)
--     if actual ~= 42 then
--         return string.format('%s is not 42', actual)
--     end
--     return ''
-- end
-- expectThat({42, 13}, elementsAreArray({matcher, 13}))
-function module.elementsAreArray(expected)
-    local expected_matchers = {}
-    for i, v in ipairs(expected) do
-        if type(v) == 'function' then
-            expected_matchers[i] = v
-        else
-            expected_matchers[i] = function (other)
-                if expected[i].__eq(expected[i], other) then
-                    return ''
-                end
-                return string.format('%s element %s does no match expected: %s', i, other, expected[i])
-            end
-        end
-    end
-    return function(actual)
-        if #actual < #expected_matchers then
-            return string.format('number of elements is less than expected: %s < %s', #actual, #expected_matchers)
-        end
-        local message = ''
-        for i, v in ipairs(actual) do
-            if i > #expected_matchers then
-                message = string.format('%s\n%s element is out of expected range: %s', message, i, #expected_matchers)
-                break
-            end
-            local match_message = expected_matchers[i](v)
-            if match_message ~= '' then
-                message = string.format('%s\n%s', message, match_message)
-            end
-        end
-        return message
-    end
-end
-
---
-- Matcher verifying that given number is not a nan.
-- @function isNotNan
-- @usage
-- expectThat(value, isNotNan())
-function module.isNotNan()
-    return function(actual)
-        if actual ~= actual then
-            return 'actual value is nan, expected to be not nan'
-        end
-        return ''
-    end
-end
-
---
-- Matcher accepting any value.
-- @function isAny
-- @usage
-- expectThat(value, isAny())
-function module.isAny()
-    return function(actual)
-        return ''
-    end
-end
-
-local function serializeArray(a)
-    local result = nil
-    for _, v in ipairs(a) do
-        if result == nil then
-            result = string.format('{%s', serialize(v))
-        else
-            result = string.format('%s, %s', result, serialize(v))
-        end
-    end
-    if result == nil then
-        return '{}'
-    end
-    return string.format('%s}', result)
-end
-
-local function serializeTable(t)
-    local result = nil
-    for k, v in pairs(t) do
-        if result == nil then
-            result = string.format('{%q = %s', k, serialize(v))
-        else
-            result = string.format('%s, %q = %s', result, k, serialize(v))
-        end
-    end
-    if result == nil then
-        return '{}'
-    end
-    return string.format('%s}', result)
-end
-
-local function isArray(t)
-    local i = 1
-    for _ in pairs(t) do
-        if t[i] == nil then
-            return false
-        end
-        i = i + 1
-    end
-    return true
-end
-
-function serialize(v)
-    local t = type(v)
-    if t == 'string' then
-        return string.format('%q', v)
-    elseif t == 'table' then
-        if isArray(v) then
-            return serializeArray(v)
-        end
-        return serializeTable(v)
-    end
-    return string.format('%s', v)
-end
-
-local function compareScalars(v1, v2)
-    if v1 == v2 then
-        return ''
-    end
-    if type(v1) == 'string' then
-        return string.format('%q ~= %q', v1, v2)
-    end
-    return string.format('%s ~= %s', v1, v2)
-end
-
-local function collectKeys(t)
-    local result = {}
-    for key in pairs(t) do
-        table.insert(result, key)
-    end
-    table.sort(result)
-    return result
-end
-
-local function compareTables(t1, t2)
-    local keys1 = collectKeys(t1)
-    local keys2 = collectKeys(t2)
-    if #keys1 ~= #keys2 then
-        return string.format('table size mismatch: %d ~= %d', #keys1, #keys2)
-    end
-    for i = 1, #keys1 do
-        local key1 = keys1[i]
-        local key2 = keys2[i]
-        if key1 ~= key2 then
-            return string.format('table keys mismatch: %q ~= %q', key1, key2)
-        end
-        local d = compare(t1[key1], t2[key2])
-        if d ~= '' then
-            return string.format('table values mismatch at key %s: %s', serialize(key1), d)
-        end
-    end
-    return ''
-end
-
-function compare(v1, v2)
-    local type1 = type(v1)
-    local type2 = type(v2)
-    if type2 == 'function' then
-        return v2(v1)
-    end
-    if type1 ~= type2 then
-        return string.format('types mismatch: %s ~= %s', type1, type2)
-    end
-    if type1 == 'nil' then
-        return ''
-    elseif type1 == 'table' then
-        return compareTables(v1, v2)
-    elseif type1 == 'nil' or type1 == 'boolean' or type1 == 'number' or type1 == 'string' then
-        return compareScalars(v1, v2)
-    end
-    error('unsupported type: %s', type1)
-end
-
---
-- Matcher verifying that given value is equal to expected. Accepts nil, boolean, number, string and table or matcher
-- function.
-- @function equalTo
-- @usage
-- expectThat({a = {42, 'foo', {b = true}}}, equalTo({a = {42, 'foo', {b = true}}}))
-function module.equalTo(expected)
-    return function(actual)
-        local diff = compare(actual, expected)
-        if diff == '' then
-            return ''
-        end
-        return string.format('%s; actual: %s; expected: %s', diff, serialize(actual, ''), serialize(expected, ''))
-    end
-end
-
-return module
--- a/scripts/data/integration_tests/testing_util/testing_util.lua
+++ b/scripts/data/integration_tests/testing_util/testing_util.lua
@ -2,22 +2,23 @@ local core = require('openmw.core')
 local util = require('openmw.util')

 local M = {}
-
-local menuTestsOrder = {}
-local menuTests = {}
-
-local globalTestsOrder = {}
-local globalTests = {}
-local globalTestRunner = nil
-local currentGlobalTest = nil
-local currentGlobalTestError = nil
-
-local localTests = {}
-local localTestRunner = nil
 local currentLocalTest = nil
 local currentLocalTestError = nil

-local function makeTestCoroutine(fn)
+function M.testRunner(tests)
+    local fn = function()
+        for i, test in ipairs(tests) do
+            local name, fn = unpack(test)
+            print('TEST_START', i, name)
+            local status, err = pcall(fn)
+            if status then
+                print('TEST_OK', i, name)
+            else
+                print('TEST_FAILED', i, name, err)
+            end
+        end
+        core.quit()
+    end
    local co = coroutine.create(fn)
    return function()
        if coroutine.status(co) ~= 'dead' then
@ -26,64 +27,6 @@ local function makeTestCoroutine(fn)
    end
 end

-local function runTests(tests)
-    for i, test in ipairs(tests) do
-        local name, fn = unpack(test)
-        print('TEST_START', i, name)
-        local status, err = pcall(fn)
-        if status then
-            print('TEST_OK', i, name)
-        else
-            print('TEST_FAILED', i, name, err)
-        end
-    end
-    core.quit()
-end
-
-function M.makeUpdateMenu()
-    return makeTestCoroutine(function()
-        print('Running menu tests...')
-        runTests(menuTestsOrder)
-    end)
-end
-
-function M.makeUpdateGlobal()
-    return makeTestCoroutine(function()
-        print('Running global tests...')
-        runTests(globalTestsOrder)
-    end)
-end
-
-function M.registerMenuTest(name, fn)
-    menuTests[name] = fn
-    table.insert(menuTestsOrder, {name, fn})
-end
-
-function M.runGlobalTest(name)
-    currentGlobalTest = name
-    currentGlobalTestError = nil
-    core.sendGlobalEvent('runGlobalTest', name)
-    while currentGlobalTest do
-        coroutine.yield()
-    end
-    if currentGlobalTestError then
-        error(currentGlobalTestError, 2)
-    end
-end
-
-function M.registerGlobalTest(name, fn)
-    globalTests[name] = fn
-    table.insert(globalTestsOrder, {name, fn})
-end
-
-function M.updateGlobal()
-    if globalTestRunner and coroutine.status(globalTestRunner) ~= 'dead' then
-        coroutine.resume(globalTestRunner)
-    else
-        globalTestRunner = nil
-    end
-end
-
 function M.runLocalTest(obj, name)
    currentLocalTest = name
    currentLocalTestError = nil
@ -96,21 +39,7 @@ function M.runLocalTest(obj, name)
    end
 end

-function M.registerLocalTest(name, fn)
-    localTests[name] = fn
-end
-
-function M.updateLocal()
-    if localTestRunner and coroutine.status(localTestRunner) ~= 'dead' then
-        if not core.isWorldPaused() then
-            coroutine.resume(localTestRunner)
-        end
-    else
-        localTestRunner = nil
-    end
-end
-
-function M.expect(cond, msg)
+function M.expect(cond, delta, msg)
    if not cond then
        error(msg or '"true" expected', 2)
    end
@ -158,6 +87,76 @@ function M.expectNotEqual(v1, v2, msg)
    end
 end

+function M.closeToVector(expected, maxDistance)
+    return function(actual)
+        local distance = (expected - actual):length()
+        if distance <= maxDistance then
+            return ''
+        end
+        return string.format('%s is too far from expected %s: %s > %s', actual, expected, distance, maxDistance)
+    end
+end
+
+---
+-- Matcher verifying that given value is an array each element of which matches elements of expected.
+-- @function elementsAreArray
+-- @param expected#array of values or matcher functions.
+-- @usage
+-- local t = {42, 13}
+-- local matcher = function(actual)
+--     if actual ~= 42 then
+--         return string.format('%s is not 42', actual)
+--     end
+--     return ''
+-- end
+-- expectThat({42, 13}, elementsAreArray({matcher, 13}))
+function M.elementsAreArray(expected)
+    local expected_matchers = {}
+    for i, v in ipairs(expected) do
+        if type(v) == 'function' then
+            expected_matchers[i] = v
+        else
+            expected_matchers[i] = function (other)
+                if expected[i].__eq(expected[i], other) then
+                    return ''
+                end
+                return string.format('%s element %s does no match expected: %s', i, other, expected[i])
+            end
+        end
+    end
+    return function(actual)
+        if #actual < #expected_matchers then
+            return string.format('number of elements is less than expected: %s < %s', #actual, #expected_matchers)
+        end
+        local message = ''
+        for i, v in ipairs(actual) do
+            if i > #expected_matchers then
+                message = string.format('%s\n%s element is out of expected range: %s', message, i, #expected_matchers)
+                break
+            end
+            local match_message = expected_matchers[i](v)
+            if match_message ~= '' then
+                message = string.format('%s\n%s', message, match_message)
+            end
+        end
+        return message
+    end
+end
+
+---
+-- Matcher verifying that given number is not a nan.
+-- @function isNotNan
+-- @usage
+-- expectThat(value, isNotNan())
+function M.isNotNan(expected)
+    return function(actual)
+        if actual ~= actual then
+            return 'actual value is nan, expected to be not nan'
+        end
+        return ''
+    end
+end
+
 ---
 -- Verifies that given value matches provided matcher.
 -- @function expectThat
@ -183,50 +182,28 @@ function M.formatActualExpected(actual, expected)
    return string.format('actual: %s, expected: %s', actual, expected)
 end

-- used only in menu scripts
-M.menuEventHandlers = {
-    globalTestFinished = function(data)
-        if data.name ~= currentGlobalTest then
-            error(string.format('globalTestFinished with incorrect name %s, expected %s', data.name, currentGlobalTest), 2)
-        end
-        currentGlobalTest = nil
-        currentGlobalTestError = data.errMsg
-    end,
-}
+local localTests = {}
+local localTestRunner = nil

-- used only in global scripts
-M.globalEventHandlers = {
-    runGlobalTest = function(name)
-        fn = globalTests[name]
-        local types = require('openmw.types')
-        local world = require('openmw.world')
-        if not fn then
-            types.Player.sendMenuEvent(world.players[1], 'globalTestFinished', {name=name, errMsg='Global test is not found'})
-            return
-        end
-        globalTestRunner = coroutine.create(function()
-            local status, err = pcall(fn)
-            if status then
-                err = nil
-            end
-            types.Player.sendMenuEvent(world.players[1], 'globalTestFinished', {name=name, errMsg=err})
-        end)
-    end,
-    localTestFinished = function(data)
-        if data.name ~= currentLocalTest then
-            error(string.format('localTestFinished with incorrect name %s, expected %s', data.name, currentLocalTest), 2)
-        end
-        currentLocalTest = nil
-        currentLocalTestError = data.errMsg
-    end,
-}
+function M.registerLocalTest(name, fn)
+    localTests[name] = fn
+end

-- used only in local scripts
-M.localEventHandlers = {
-    runLocalTest = function(name)
+function M.updateLocal()
+    if localTestRunner and coroutine.status(localTestRunner) ~= 'dead' then
+        if not core.isWorldPaused() then
+            coroutine.resume(localTestRunner)
+        end
+    else
+        localTestRunner = nil
+    end
+end
+
+M.eventHandlers = {
+    runLocalTest = function(name)  -- used only in local scripts
        fn = localTests[name]
        if not fn then
-            core.sendGlobalEvent('localTestFinished', {name=name, errMsg='Local test is not found'})
+            core.sendGlobalEvent('localTestFinished', {name=name, errMsg='Test not found'})
            return
        end
        localTestRunner = coroutine.create(function()
@ -237,6 +214,13 @@ M.localEventHandlers = {
            core.sendGlobalEvent('localTestFinished', {name=name, errMsg=err})
        end)
    end,
+    localTestFinished = function(data)  -- used only in global scripts
+        if data.name ~= currentLocalTest then
+            error(string.format('localTestFinished with incorrect name %s, expected %s', data.name, currentLocalTest))
+        end
+        currentLocalTest = nil
+        currentLocalTestError = data.errMsg
+    end,
 }

 return M
--- a/Show more
+++ b/Show more