mirror of
https://github.com/4jcraft/4jcraft.git
synced 2026-04-29 18:23:36 +00:00
docs: clarify comment policy
This commit is contained in:
parent
d6b8d665df
commit
10009abcc5
|
|
@ -42,7 +42,7 @@ At this time, we are trying to stay as close to the vanilla game as close as pos
|
|||
|
||||
### Bugfixes should be clearly commented.
|
||||
|
||||
If you are submitting a bugfix or a minor addition to *game-relevant code*, clearly comment the purpose of this fix in a comment starting with `// 4jcraft:`. This indicates a patch over the original game code and lets us keep track of what has been modified in case it needs to be reverted later.
|
||||
If you are submitting a minor addition to *game-relevant code*, clearly comment the purpose of this in a comment starting with `// 4jcraft:`. This indicates a patch over the original game code and lets us keep track of what has been modified in case it needs to be reverted later.
|
||||
|
||||
> **Example:**
|
||||
>
|
||||
|
|
@ -54,6 +54,8 @@ If you are submitting a bugfix or a minor addition to *game-relevant code*, clea
|
|||
> dos->writeShort((id & 0x07FF) | 0x800);
|
||||
> ```
|
||||
|
||||
This rule may not apply to contributions fixing *trivially incorrect* 4J code that affects all platforms, such as instances of undefined behavior.
|
||||
|
||||
### Try to fix the cause, not the effect.
|
||||
|
||||
If you are fixing a bug, avoid submitting "hacks" that attempt to patch the effects of the bug rather than the root cause.
|
||||
|
|
|
|||
Loading…
Reference in a new issue