Rockbox

  • Status Closed
  • Percent Complete
    100%
  • 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
Attached to Project: Rockbox
Opened by AsaelReiter - 2009-08-03
Last edited by tomers - 2010-02-22

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:  

Partly accepted. The author will open separate Flyspray items for the changes that were not committed yet.

Some fixings.

   bm.diff (44.9 KiB)

Some change of the ball-brick check.

   bm.diff (44.2 KiB)

Some simplification off the powerups.

   bm.diff (44.8 KiB)

Fix the left edge of the paddle.

   bm.diff (44.8 KiB)

Fix the glue

   bm.diff (47.1 KiB)

Simplify the fires and the powerups. Fix the saving.

   bm.diff (48.8 KiB)

A little fixing.

Simplify the powerups, fire new balls (”B” powerup) from the hit point.

OOPS! here is the file.

   bm.diff (51.1 KiB)

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.

   bm.diff (7.2 KiB)

I simplified the fire-brick check.

   bm.diff (7.3 KiB)

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 :)

   bm.diff (22.9 KiB)
By the way, why is ‘brick’ a one-dimensional array?
I also wondered about that. If you think it is better to have it two dimensional, please post a patch and we’ll see (notice there is the interdiff utility to compare to patches, which allows you to easily develop plugins which are dependent on each other).
Can you please post each of your patches separately, and close this task? It is hard to follow what other modifications this patch contains, and it is not a good practice for one FlySpray item to hold several different patches.

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

this patch make brick a two dimensional array

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.

Loading...

Available keyboard shortcuts

Tasklist

Task Details

Task Editing