[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[nongnu] elpa/pacmacs ed40addf95 116/472: Refactor pacmacs--put-object (
From: |
ELPA Syncer |
Subject: |
[nongnu] elpa/pacmacs ed40addf95 116/472: Refactor pacmacs--put-object (#74) |
Date: |
Thu, 6 Jan 2022 21:59:17 -0500 (EST) |
branch: elpa/pacmacs
commit ed40addf956a940d81873f29c67dc04c7a832716
Author: rexim <reximkut@gmail.com>
Commit: rexim <reximkut@gmail.com>
Refactor pacmacs--put-object (#74)
---
pacmacs.el | 4 +---
test/pacmacs-test.el | 14 ++++++++------
2 files changed, 9 insertions(+), 9 deletions(-)
diff --git a/pacmacs.el b/pacmacs.el
index 31ed6f7b22..e2453a8ce3 100644
--- a/pacmacs.el
+++ b/pacmacs.el
@@ -286,9 +286,7 @@
(plist-bind ((row :row)
(column :column))
anim-object
- (when (and (<= 0 row) (<= row (1- pacmacs-board-height))
- (<= 0 column) (<= column (1- pacmacs-board-width)))
- (aset (aref pacmacs-board row) column anim-object))))
+ (pacmacs--cell-set pacmacs-board row column anim-object)))
(defun pacmacs-render-track-board ()
(dotimes (row pacmacs-board-height)
diff --git a/test/pacmacs-test.el b/test/pacmacs-test.el
index a86d1a316d..e27a3af2e1 100644
--- a/test/pacmacs-test.el
+++ b/test/pacmacs-test.el
@@ -33,12 +33,14 @@
(pacmacs--track-object game-object))))
(ert-deftest pacmacs--put-object-test ()
- (let ((pacmacs-board-width 2)
- (pacmacs-board-height 2)
- (pacmacs-board [[nil nil]
- [nil nil]])
+ (let ((pacmacs-board (list :width 2
+ :height 2
+ :data [[nil nil]
+ [nil nil]]))
(anim-object (list :row 0 :column 1)))
(pacmacs--put-object anim-object)
- (should (equal [[nil (:row 0 :column 1)]
- [nil nil]]
+ (should (equal (list :width 2
+ :height 2
+ :data [[nil (:row 0 :column 1)]
+ [nil nil]])
pacmacs-board))))
- [nongnu] elpa/pacmacs 98ce700143 023/472: Hide cursor in the game's buffer. Close #12, (continued)
- [nongnu] elpa/pacmacs 98ce700143 023/472: Hide cursor in the game's buffer. Close #12, ELPA Syncer, 2022/01/06
- [nongnu] elpa/pacmacs 3070cf0e6a 063/472: Merge branch 'refactor-image-resource-51'. Close #51, ELPA Syncer, 2022/01/06
- [nongnu] elpa/pacmacs 1a678dfee7 110/472: Refactor out pacmacs--object-at-p (#74), ELPA Syncer, 2022/01/06
- [nongnu] elpa/pacmacs 5492ed0a87 118/472: Refactor pacmacs-render-state (#74), ELPA Syncer, 2022/01/06
- [nongnu] elpa/pacmacs b5078d2632 141/472: Move most of the rendering stuff to pacman-render, ELPA Syncer, 2022/01/06
- [nongnu] elpa/pacmacs a460cd8b06 148/472: Merge pull request #107 from rexim/feature/new-maps, ELPA Syncer, 2022/01/06
- [nongnu] elpa/pacmacs c426f6ac91 078/472: Fix some accidental paredit typo, ELPA Syncer, 2022/01/06
- [nongnu] elpa/pacmacs 4341b23d87 094/472: Implement wraparound. Close #67, ELPA Syncer, 2022/01/06
- [nongnu] elpa/pacmacs 12942ddeff 113/472: Refactor out pacmacs--fill-board (#74), ELPA Syncer, 2022/01/06
- [nongnu] elpa/pacmacs 91408c074d 115/472: Refactor pacmacs--track-object (#74), ELPA Syncer, 2022/01/06
- [nongnu] elpa/pacmacs ed40addf95 116/472: Refactor pacmacs--put-object (#74),
ELPA Syncer <=
- [nongnu] elpa/pacmacs eac9d47c62 117/472: Refactor pacmacs-render-track-board (#74), ELPA Syncer, 2022/01/06
- [nongnu] elpa/pacmacs d763bd0f82 121/472: Fix the UTs, ELPA Syncer, 2022/01/06
- [nongnu] elpa/pacmacs 9527a965e1 134/472: Merge branch 'switch-game-state-in-logic-functions-97'. Close #97, ELPA Syncer, 2022/01/06
- [nongnu] elpa/pacmacs 86f76c4bef 129/472: Implement switching level mechanics. Close #79, ELPA Syncer, 2022/01/06
- [nongnu] elpa/pacmacs fa41daf2ca 149/472: Merge branch 'master' into score-table-92, ELPA Syncer, 2022/01/06
- [nongnu] elpa/pacmacs 85069784fb 147/472: Add a couple of new maps., ELPA Syncer, 2022/01/06
- [nongnu] elpa/pacmacs 39fc09baf3 161/472: Load cl-lib.el for using its macros and functions, ELPA Syncer, 2022/01/06
- [nongnu] elpa/pacmacs 9ddaedae20 174/472: UT for pacmacs--file-content-test, ELPA Syncer, 2022/01/06
- [nongnu] elpa/pacmacs b6141dc8e3 175/472: UT for pacmacs--anim-object-list-next-frame, ELPA Syncer, 2022/01/06
- [nongnu] elpa/pacmacs 1186dcdaa1 197/472: Double dash pacmacs-render-state function (#126), ELPA Syncer, 2022/01/06