- Status Closed
- Percent Complete
- Task Type Patches
- Category Games
- Assigned To No-one
- Operating System All players
- Severity Low
- Priority Very Low
- Reported Version Version 3.3
- Due in Version Undecided
-
Due Date
Undecided
- Votes
- Private
FS#10485 - Some changes for brickmania
After adding the enlarge&shrink powerups ( FS#10463 ), I wanted to make it really grow and shrink (not only 3 sets of pads).
During the working on that, I notice some code that I thought I can improve, so I rewrite some parts of the code.
To use this code, you need the BMPs of the parts of the pad.
(brickmania_left_pads,brickmania_middle_pads,brickmania_right_pads)
If your resolution is 220×176 or monochrome, you can use attached files (even they are a little ugly).
However, don’t forget to add them to apps/plugins/bitmaps/native/SOURCES
Closed by tomers
2010-02-22 04:37
Reason for closing: Accepted
Additional comments about closing: Warning: Undefined array key "typography" in /home/rockbox/flyspray/plugins/dokuwiki/inc/parserutils.php on line 371 Warning: Undefined array key "camelcase" in /home/rockbox/flyspray/plugins/dokuwiki/inc/parserutils.php on line 407
2010-02-22 04:37
Reason for closing: Accepted
Additional comments about closing: Warning: Undefined array key "typography" in /home/rockbox/flyspray/plugins/dokuwiki/inc/parserutils.php on line 371 Warning: Undefined array key "camelcase" in /home/rockbox/flyspray/plugins/dokuwiki/inc/parserutils.php on line 407
Partly accepted. The author will open
separate Flyspray items for the changes
that were not committed yet.
Loading...
Available keyboard shortcuts
- Alt + ⇧ Shift + l Login Dialog / Logout
- Alt + ⇧ Shift + a Add new task
- Alt + ⇧ Shift + m My searches
- Alt + ⇧ Shift + t focus taskid search
Tasklist
- o open selected task
- j move cursor down
- k move cursor up
Task Details
- n Next task
- p Previous task
- Alt + ⇧ Shift + e ↵ Enter Edit this task
- Alt + ⇧ Shift + w watch task
- Alt + ⇧ Shift + y Close Task
Task Editing
- Alt + ⇧ Shift + s save task
Some fixings.
Some change of the ball-brick check.
Some simplification off the powerups.
Fix the left edge of the paddle.
Fix the glue
Simplify the fires and the powerups. Fix the saving.
A little fixing.
Simplify the powerups, fire new balls (”B” powerup) from the hit point.
OOPS! here is the file.
Hi Asael,
I would like to commit this patch. Can you please sync it?
This patch is rather large. It would be best if you can split it into several different patches (post them all here). e.g. have a separate patch to replace hard-coded constants with #defines.
Regards,
Tomer
This part of the patch should simplify the way that the fires work.
Someone (kkurbjun?) changed the fire-brick check, to be more smart (I guess it’s help when the speed of the fires is more than the height of the bricks).
I did not touch it yet. I will check if I can improve that later.
I simplified the fire-brick check.
sync to r24771
bm2.diff with several changes made by me is now committed in r24755.
Asael, Can you please create an updated patch with what is left from the original patch? Thanks
קודד ביום חול
This patch simplify the powerups (and make them very close to the fires)
By the way, why is ‘brick’ a one-dimensional array?
If there is no reason, I will change it.
OOPS! I forgot the file :)
Patch file naming: Please rename bm.diff to something nicer, like brickmania_powerups_rework.diff, this way it will be easier to follow.
FlySpray item naming: ‘Some changes for brickmania’ is not a very descriptive name.
It will be easier for the committer (me) to just put the FlySpray item’s name in the commit log message. In r24755 I’ve used this item’s name as log message and got responses on IRC that I should have used some more descriptive text.
this patch make brick a two dimensional array.
I did not understand how to work with interdiff, so I diffed the versions of brickmania.c
Asael, I would like to close this FlySpray item. Could you please create a new FS item explicitly for this patch?
Also, please list whatever changes the original patch contains, otherwise it might be forgotten after this FS item is closed.
Thanks, Tomer
OK. you can close it. I listed my old changes.