yquake2remaster/CONTRIBUTE

44 lines
2.3 KiB
Text
Raw Normal View History

2012-05-03 10:34:14 +00:00
* ****************************** *
* Yamagi Quake II *
* http://www.yamagi.org/quake2 *
* http://github.com/yquake2 *
* ****************************** *
2014-05-08 15:20:42 +00:00
TODO List
2012-05-03 10:34:14 +00:00
===============================================================================
2018-08-14 13:35:51 +00:00
At this time there're no open tasks regaring Quake II. Nevertheless some hints
2016-06-25 08:19:45 +00:00
for working with the code:
2012-05-03 10:34:14 +00:00
2016-12-03 08:23:13 +00:00
- Sign up for a Github account and fork our yquake2 repository. This allows the
easy integration of upstream changes into your branch and sending of pull
requests. You'll get a wiki and a bugtracker for free.
- To contribute your changes back into the main project send pull requests via
Github. It's much easier to review and merge pull requests than patches.
2012-05-03 10:34:14 +00:00
Please send only pull reqeuests from a distinct branch at not from your
"master" branch!
- Quake II has a very fragile and broken codebase. Even after years of cleanup
it's still a disaster. Therefore:
- Do only one change at a time!
- Test after each change (play at least through base1.bsp)
2018-08-14 13:35:51 +00:00
- Commit early and commit often to create a fine grained history. This helps
2016-12-03 08:23:13 +00:00
"git bisect" to find bugs and errors.
2012-05-03 10:34:14 +00:00
- Do not try to clean up things or even rewrite code that you do not
2016-12-03 08:23:13 +00:00
understand to 110%! Even small behavioral changes can introduce gameplay
changes and trigger new bugs! Especially everything that depends on map data
(e.g. path finding or collision detection) is very likely to break in
interesting ways!
2012-05-03 10:34:14 +00:00
- Do not add new dependencies. If you must add a new one contact the Yamagi
2016-12-03 08:23:13 +00:00
Quake II developers prior to it! Everything that adds dependencies should be
hided behint preprocessor macros.
2012-05-03 10:34:14 +00:00
- If your changes change the gameplay experience, make them optional by
introducing a new cvar.
- Linux is not the only operating system out there. All changes should be
2016-12-03 08:23:13 +00:00
portable to other platform (writing pure ANSI-C or C99 is recommended but not
always applicable).
- x86 ist not the only CPU architecture. All changes should be done in pure C
(e.g. no inline assembler) and in an endianess independed way.
2012-05-03 10:34:14 +00:00
- gcc is not the only compiler. Test your changes with clang.