1. 05 Dec, 2020 1 commit
  2. 20 Nov, 2020 3 commits
    • Bernd Schmidt's avatar
      f1164fbf
    • Bernd Schmidt's avatar
      Sync the config once after loading · fc90d0f5
      Bernd Schmidt authored
      Previously this could potentially call fdatasync once per puzzle, which
      was pretty bad for loading times. That was exacerbated by the previous
      date comparison bug that caused the config to be regenerated at every
      start.
      fc90d0f5
    • Bernd Schmidt's avatar
      First pass of removing threading · 9b706bf8
      Bernd Schmidt authored
      My guess would be that this was an optimization for very slow load times,
      which I've also seen.  But the reason for these are repeated fdatasync
      calls which are completely unnecessary. By removing the threading, we can
      actually more easily optimize this in a followup patch.
      
      This is probably not a complete removal of all the threading-related cruft.
      I suspected a number of mutexes are no longer necessary now, but that can
      be left for later.
      9b706bf8
  3. 23 Apr, 2019 1 commit
  4. 17 Oct, 2018 1 commit
    • Albert Astals Cid's avatar
      Fix crash on startup · ff1e5bc8
      Albert Astals Cid authored
      Summary:
      There was various threads using the same kconfig[group] at the same time, so bad things happened.
      Add a mutex
      
      BUGS: 398416
      
      Test Plan:
      Run
        rm -f ~/.config/palapeli* && palapeli
      a few times, see it doesn't crash anymore
      
      Reviewers: #kde_games
      
      Subscribers: chehrlic, kde-games-devel
      
      Differential Revision: https://phabricator.kde.org/D16118
      ff1e5bc8
  5. 15 Sep, 2018 1 commit
  6. 27 Aug, 2017 1 commit
    • Christian Ehrlicher's avatar
      Fix KStandardDirs::locateLocal() porting · c3beee6c
      Christian Ehrlicher authored
      Summary:
      KStandardDirs::locateLocal() created the directory if the requested
      file and/or directory did not exist and returned the path. This
      was not considered during porting to QStandardPaths
      
      Test Plan:
      - remove local palapeli config files (~/.local/share/palapeli/)
      - start palapeli, shipped puzzles should appear and
        ~/.local/share/palapeli/collection/ should contain a '.puzzle'
        file for every puzzle.
      -  Try to create a new puzzle.
      
      Reviewers: #kde_games, ltoscano
      
      Reviewed By: ltoscano
      
      Subscribers: ltoscano, #kde_games
      
      Differential Revision: https://phabricator.kde.org/D7566
      c3beee6c
  7. 22 Aug, 2017 1 commit
  8. 04 Jan, 2015 1 commit
  9. 27 Dec, 2014 1 commit
  10. 07 Dec, 2014 1 commit
  11. 05 Dec, 2014 1 commit
  12. 02 Apr, 2011 1 commit
    • Stefan Majewsky's avatar
      Replace the tabwindow interface by a standard XMLGUI window. · 0a029560
      Stefan Majewsky authored
      DIGEST
      
      While the tabwindow interface had its time and justification, the new
      solution is simpler and more consistent. It will make it easier to do
      some more widget reordering, and add new interface components to the
      puzzle table.
      
      Intensive testing indicates that this change has not caused any
      regressions. In fact, I've applied some polish here and there,
      although there are still some pieces (esp. the "Restart" and "Back to
      collection" actions) which do not fit in at the moment, but I promise
      that these will fall into place shortly.
      
      svn path=/trunk/KDE/kdegames/palapeli/; revision=1226739
      0a029560
  13. 25 Mar, 2011 6 commits
    • Stefan Majewsky's avatar
      Kill the awful first-run assistant. The collection will now automatically... · a6fe38ee
      Stefan Majewsky authored
      Kill the awful first-run assistant. The collection will now automatically build desktop+jpg into puzzles.
      
      DIGEST: one of the few user-visible changes induced by the new
              multithreaded Puzzle class
      
      svn path=/trunk/KDE/kdegames/palapeli/; revision=1225964
      a6fe38ee
    • Stefan Majewsky's avatar
      Remove usage of KIO. · c7851216
      Stefan Majewsky authored
      KIO was used very inconsistently at only few places. It should be
      possible to implement KIO on top of the new multithreaded Puzzle
      class, but this requires one to wrap one's mind around it, and I do
      not have the time to do that. So IMO it's better to remove the few
      usages of KIO to send a clear message to the user about the feature
      set that he can expect.
      
      svn path=/trunk/KDE/kdegames/palapeli/; revision=1225957
      c7851216
    • Stefan Majewsky's avatar
      Refactor Collection classes, remove OldPuzzle. · f3c43066
      Stefan Majewsky authored
      All collection classes are merged into one which corresponds to
      LocalCollection. Instead of throwing modelindexes around, Puzzle
      instances are used whenever this is enough.
      
      svn path=/trunk/KDE/kdegames/palapeli/; revision=1225956
      f3c43066
    • Stefan Majewsky's avatar
      Remove all storage from OldPuzzle. · 85282827
      Stefan Majewsky authored
      Because injectMetadata() won't work anymore, this step includes the
      introduction of a new CollectionStorageComponent which controls the
      metadata cache. Another new component is the CopyComponent which
      powers the copy-constructor of Palapeli::OldPuzzle.
      
      svn path=/trunk/KDE/kdegames/palapeli/; revision=1225954
      85282827
    • Stefan Majewsky's avatar
      Move identifier management from Collection to Puzzle. · 6f95f4d9
      Stefan Majewsky authored
      The Scene class is now independent of OldPuzzle.
      
      svn path=/trunk/KDE/kdegames/palapeli/; revision=1225953
      6f95f4d9
    • Stefan Majewsky's avatar
      Rename Puzzle class to OldPuzzle, in anticipation of upcoming refactoring. · 2f7f26a5
      Stefan Majewsky authored
      Also, ThumbnailBaseSize moves to Palapeli::PuzzleMetadata.
      
      svn path=/trunk/KDE/kdegames/palapeli/; revision=1225949
      2f7f26a5
  14. 25 May, 2010 1 commit
  15. 27 Oct, 2009 1 commit
  16. 13 Oct, 2009 1 commit
    • Stefan Majewsky's avatar
      Improve the user experience with puzzle loading: · 332bf86b
      Stefan Majewsky authored
      * The TAR archive is extracted in a separate thread whenever possible.
      * The scene returns control to the event loop in short cycles to keep the interface responsive.
      * Introduce a LoadingWidget which obscurs the puzzle table when loading puzzles, to visualise the process.
      
      svn path=/trunk/playground/games/palapeli/; revision=1034778
      332bf86b
  17. 11 Oct, 2009 2 commits