Rockbox

Tasklist

FS#10485 - Some changes for brickmania

Attached to Project: Rockbox
Opened by Asael Reiter (AsaelReiter) - Monday, 03 August 2009, 13:34 GMT
Last edited by Tomer Shalev (tomers) - Monday, 22 February 2010, 04:37 GMT
Task Type Patches
Category Games
Status Closed
Assigned To No-one
Operating System All players
Severity Low
Priority Normal
Reported Version Version 3.3
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 0
Private No

Details

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 220x176 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
This task depends upon

Closed by  Tomer Shalev (tomers)
Monday, 22 February 2010, 04:37 GMT
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.
Comment by Asael Reiter (AsaelReiter) - Monday, 03 August 2009, 17:08 GMT
Some fixings.
   bm.diff (44.9 KiB)
Comment by Asael Reiter (AsaelReiter) - Monday, 03 August 2009, 23:55 GMT
Some change of the ball-brick check.
   bm.diff (44.2 KiB)
Comment by Asael Reiter (AsaelReiter) - Thursday, 06 August 2009, 00:31 GMT
Some simplification off the powerups.
   bm.diff (44.8 KiB)
Comment by Asael Reiter (AsaelReiter) - Thursday, 06 August 2009, 01:01 GMT
Fix the left edge of the paddle.
   bm.diff (44.8 KiB)
Comment by Asael Reiter (AsaelReiter) - Sunday, 09 August 2009, 16:43 GMT
Fix the glue
   bm.diff (47.1 KiB)
Comment by Asael Reiter (AsaelReiter) - Wednesday, 12 August 2009, 17:15 GMT
Simplify the fires and the powerups. Fix the saving.
   bm.diff (48.8 KiB)
Comment by Asael Reiter (AsaelReiter) - Wednesday, 12 August 2009, 20:45 GMT
A little fixing.
Comment by Asael Reiter (AsaelReiter) - Sunday, 16 August 2009, 18:11 GMT
Simplify the powerups, fire new balls ("B" powerup) from the hit point.
Comment by Asael Reiter (AsaelReiter) - Sunday, 16 August 2009, 18:13 GMT
OOPS! here is the file.
   bm.diff (51.1 KiB)
Comment by Tomer Shalev (tomers) - Wednesday, 17 February 2010, 06:52 GMT
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
Comment by Asael Reiter (AsaelReiter) - Thursday, 18 February 2010, 17:23 GMT
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)
Comment by Asael Reiter (AsaelReiter) - Friday, 19 February 2010, 00:57 GMT
I simplified the fire-brick check.
   bm.diff (7.3 KiB)
Comment by Asael Reiter (AsaelReiter) - Friday, 19 February 2010, 14:03 GMT
sync to r24771
Comment by Tomer Shalev (tomers) - Friday, 19 February 2010, 16:36 GMT
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

קודד ביום חול ;-)
Comment by Asael Reiter (AsaelReiter) - Sunday, 21 February 2010, 02:30 GMT
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.
Comment by Asael Reiter (AsaelReiter) - Sunday, 21 February 2010, 02:31 GMT
OOPS! I forgot the file :)
   bm.diff (22.9 KiB)
Comment by Tomer Shalev (tomers) - Sunday, 21 February 2010, 04:08 GMT
> 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.


Comment by Asael Reiter (AsaelReiter) - Sunday, 21 February 2010, 12:37 GMT
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
Comment by Tomer Shalev (tomers) - Sunday, 21 February 2010, 12:55 GMT
> 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
Comment by Asael Reiter (AsaelReiter) - Monday, 22 February 2010, 00:42 GMT
OK. you can close it. I listed my old changes.

Loading...