Merge ea01d1963c into 6d89a6bc15

laf

@@ -1 +1 @@
-Subproject commit 8ec4b553f1618f7a4b47cdcf4cfc2663266111ac
+Subproject commit a2bb9ec7fb98354279a2c49870a4a47a67a8e86e

src/app/resource_finder.cpp

@@ -213,7 +213,7 @@ void ResourceFinder::includeDesktopDir(const char* filename)
 #ifdef _WIN32
 
   std::vector<wchar_t> buf(MAX_PATH);
-  HRESULT hr = SHGetFolderPath(NULL, CSIDL_DESKTOP, NULL, SHGFP_TYPE_CURRENT, &buf[0]);
+  HRESULT hr = SHGetFolderPath(NULL, CSIDL_DESKTOPDIRECTORY, NULL, SHGFP_TYPE_DEFAULT, &buf[0]);
   if (hr == S_OK) {
     addPath(base::join_path(base::to_utf8(&buf[0]), filename));
   }

src/app/script/site_class.cpp

@@ -13,8 +13,6 @@
 #include "app/console.h"
 #include "app/context.h"
 #include "app/context_observer.h"
-#include "app/doc.h"
-#include "app/doc_undo.h"
 #include "app/script/docobj.h"
 #include "app/script/engine.h"
 #include "app/script/luacpp.h"

src/app/script/sprite_class.cpp

@@ -64,7 +64,6 @@
 #include "doc/tag.h"
 #include "doc/tileset.h"
 #include "doc/tilesets.h"
-#include "undo/undo_state.h"
 
 #include <algorithm>
 
@@ -1030,42 +1029,6 @@ int Sprite_set_useLayerUuids(lua_State* L)
   return 0;
 }
 
-int Sprite_get_undoHistory(lua_State* L)
-{
-  const auto* sprite = get_docobj<Sprite>(L, 1);
-  const auto* doc = static_cast<Doc*>(sprite->document());
-  const auto* history = doc->undoHistory();
-
-  if (!history) {
-    lua_pushnil(L);
-    return 1;
-  }
-
-  const undo::UndoState* currentState = history->currentState();
-  const undo::UndoState* s = history->firstState();
-  const bool canRedo = history->canRedo();
-  bool pastCurrent = !currentState && canRedo;
-
-  int undoSteps = 0;
-  int redoSteps = 0;
-  while (s) {
-    if (pastCurrent && canRedo)
-      redoSteps++;
-    else if (currentState || !canRedo)
-      undoSteps++;
-
-    if (s == currentState || !currentState)
-      pastCurrent = true;
-
-    s = s->next();
-  }
-
-  lua_newtable(L);
-  setfield_integer(L, "undoSteps", undoSteps);
-  setfield_integer(L, "redoSteps", redoSteps);
-  return 1;
-}
-
 const luaL_Reg Sprite_methods[] = {
   { "__eq",              Sprite_eq                },
   { "resize",            Sprite_resize            },
@@ -1131,7 +1094,6 @@ const Property Sprite_properties[] = {
   { "events",               Sprite_get_events,               nullptr                         },
   { "tileManagementPlugin", Sprite_get_tileManagementPlugin, Sprite_set_tileManagementPlugin },
   { "useLayerUuids",        Sprite_get_useLayerUuids,        Sprite_set_useLayerUuids        },
-  { "undoHistory",          Sprite_get_undoHistory,          nullptr                         },
   { nullptr,                nullptr,                         nullptr                         }
 };
 

src/app/ui/editor/writing_text_state.cpp

@@ -443,7 +443,12 @@ bool WritingTextState::onSetCursor(Editor* editor, const gfx::Point& mouseScreen
   return true;
 }
 
-bool WritingTextState::onKeyDown(Editor*, KeyMessage* msg)
+bool WritingTextState::onKeyDown(Editor*, KeyMessage*)
+{
+  return false;
+}
+
+bool WritingTextState::onKeyUp(Editor*, KeyMessage* msg)
 {
   // Cancel loop pressing Esc key
   if (msg->scancode() == ui::kKeyEsc) {
@@ -452,17 +457,7 @@ bool WritingTextState::onKeyDown(Editor*, KeyMessage* msg)
   // Drop text pressing Enter key
   else if (msg->scancode() == ui::kKeyEnter) {
     drop();
-    return true;
   }
-  return false;
-}
-
-bool WritingTextState::onKeyUp(Editor*, KeyMessage* msg)
-{
-  // Note: We cannot process kKeyEnter key here to drop the text as it
-  // could be received after the Enter key is pressed in the IME
-  // dialog to accept the composition (not to accept the text). So we
-  // process kKeyEnter in onKeyDown().
   return true;
 }
 

tests/scripts/sprite.lua

@@ -228,69 +228,3 @@ do
   c = app.open(fn)
   assert(c.tileManagementPlugin == nil)
 end
-
--- Undo History
-
-function test_undo_history()
-  local sprite = Sprite(1, 1)
-
-  assert(sprite.undoHistory.undoSteps == 0)
-  assert(sprite.undoHistory.redoSteps == 0)
-
-  sprite:resize(10, 10)
-
-  assert(sprite.undoHistory.undoSteps == 1)
-  assert(sprite.undoHistory.redoSteps == 0)
-
-  sprite:resize(10, 15)
-
-  assert(sprite.undoHistory.undoSteps == 2)
-  assert(sprite.undoHistory.redoSteps == 0)
-
-  sprite:resize(10, 30)
-
-  assert(sprite.undoHistory.undoSteps == 3)
-  assert(sprite.undoHistory.redoSteps == 0)
-
-  app.undo()
-  assert(sprite.undoHistory.undoSteps == 2)
-  assert(sprite.undoHistory.redoSteps == 1)
-
-  app.undo()
-  assert(sprite.undoHistory.undoSteps == 1)
-  assert(sprite.undoHistory.redoSteps == 2)
-
-  app.redo()
-  assert(sprite.undoHistory.undoSteps == 2)
-  assert(sprite.undoHistory.redoSteps == 1)
-
-  app.undo()
-  app.undo()
-
-  assert(sprite.undoHistory.undoSteps == 0)
-  assert(sprite.undoHistory.redoSteps == 3)
-
-  sprite:resize(10, 30)
-
-  if (app.preferences.undo.allow_nonlinear_history) then
-    assert(sprite.undoHistory.undoSteps == 4)
-    assert(sprite.undoHistory.redoSteps == 0)
-  else
-    assert(sprite.undoHistory.undoSteps == 1)
-    assert(sprite.undoHistory.redoSteps == 0)
-  end
-end
-
-do
-  local prevSetting = app.preferences.undo.allow_nonlinear_history
-  app.preferences.undo.allow_nonlinear_history = true
-  test_undo_history()
-  app.preferences.undo.allow_nonlinear_history = prevSetting
-end
-
-do
-  local prevSetting = app.preferences.undo.allow_nonlinear_history
-  app.preferences.undo.allow_nonlinear_history = false
-  test_undo_history()
-  app.preferences.undo.allow_nonlinear_history = prevSetting
-end