Topic: Question about Github code contributions

[fprietog]

Hello,

I have a question about Github code contributions. Do you allow pull requests from any user?

In my case, as I've now a way to compile and test the FBNeo builds, I want to add some games that are missing from the software lists that are worth the effort. For instance, I love SNES games so I started with "Double Dragon V - The Shadow Falls" (USA & Europe versions) that It's currently only available for Megadrive.

So, I've forked the FBNeo repo, modified the file d_snes.cpp to add both versions of the game following the naming convention of the other games and the instructions of burnint.h to fill the structs. After testing I think that the game is working as expected.

At this point I want to know if it's possible to contribute it to your repo, and if so, the steps to do it correctly.

BTW, I'm a programmer but I learnt C and C++ almost 30 years ago and never used them (I'm mainly an ILE RPG programmer for IBM Power, formerly known as AS400).

Thanks and best regards.

[dink]

Hi fprietog,
Of course, anyone can contribute.  It's actually a very welcome and nice thing

As for making a pull request, I've never actually done this.  I'm sure any video on youtube for making a github PR would help, though!

Feel free to add whatever you'd like.  If you need help with anything, or would like me to check/give feedback on anything before submitting, you could post or pm it.

best regards,
- dink

[fprietog]

@dink, thank you very much for your offer of help!.

My fork of your repo is at https://github.com/fprietog/FBNeo

I'm going to do a Pull Request. First time too.

Edit: done. It's in your hands now

[Gab75]

Hi fprietog,
Thanks! Your contribution is very welcome!

[dink]

success!

[fprietog]

@Gab75 Thank you very much! I'll follow your tradition and will publish the changes done in a post with screenshots 

[fprietog]

success!

Fantastic!

I realize that some SNES games doesn't match the MAME rom names so history.xml doesn't work for these... I'll correct the ones I found when adding games.

[fprietog]

I hate to quote myself but I have a question...

I realize that some SNES games doesn't match the MAME rom names so history.xml doesn't work for these... I'll correct the ones I found when adding games.

Well, I want to add support for the game "Super Ghouls'n Ghosts (Euro)" as clone of the USA version. These are the related games:

Game Name	FBNeo	MAME
Super Ghouls'n Ghosts (USA)	supergng	sgngu
Chou Makaimura (Japan)	choumakaimura	chomakai

For the USA version, if I change the FBNeo ROM name "supergng" by the MAME ROM name "sgngu" the game info will show the history.xml info.
But the side effect is that it's pictures of the Titles, Previews, etc, will disappear as these will become missnamed.
Same for the Japanese version.

What do you think: should I leave the current FBNeo ROM name and forget about the history.xml info or should I change it to MAME ROM name?

I personally prefer the MAME ROM name and having the history available. I can also replace the previews and titles renamed in the FBNeo-extras Github repo minimizing the problem.

In any case, it's something quite important to decide by myself... So please, tell me what to do. 

Thanks and best regards.

[Gab75]

The MAME "zipped files name structure" is based on a 8 characters limit (at least for the parent title) while the FBN "zipped files name structure" often uses a more free and variable name system (in this way the zipped game is easier to identify).
IMHO it's better to leave things be, also because there would be a lot of files (archives and pictures) to rename!
Obviously I also wait the opinion of Others!

PS: for "Super Ghouls'n Ghosts (Euro)" the zipped file (FBN style) should be named: supergnge

[barbudreadmon]

There are 2 lookup tables for games named differently from MAME :
- https://github.com/finalburnneo/FBNeo/blob/master/src/burn/hiscore.cpp#L336-L375 for hiscore.dat
- https://github.com/finalburnneo/FBNeo/blob/master/src/burner/win32/sel.cpp#L1235-L1276 for history.dat (i think ?)

It might be a good idea to merge those 2 tables into one (in some separate header file for easy include ?), and expand it with additional entries ?

One reason those romsets are named differently from MAME is that their content is usually different (single file for FBNeo, multiple files for MAME).

[dink]

For the consoles I worked on: I made it a point to not follow MAME at all. Especially since they're not based on any MAME code.  (except for the cpu and/or sound core)

best regards,
- dink

[fprietog]

There are 2 lookup tables for games named differently from MAME :
- https://github.com/finalburnneo/FBNeo/blob/master/src/burn/hiscore.cpp#L336-L375 for hiscore.dat
- https://github.com/finalburnneo/FBNeo/blob/master/src/burner/win32/sel.cpp#L1235-L1276 for history.dat (i think ?)

It might be a good idea to merge those 2 tables into one (in some separate header file for easy include ?), and expand it with additional entries ?

Sounds interesting, lot's of work, but interesting. At first sight I'm not sure if these tables are only for arcade or also for home system games as there isn't a single entry with prefix like nes_ or snes_...

[barbudreadmon]

Home systems support shouldn't be a concern.

Populating that list still seems like less work than changing driver names.

The only concern i might have is speed since the list has to be iterated until a match is found. I'm wondering if there is faster way to do this without using functions that require recent cpp revisions.