mirror of
https://github.com/isledecomp/isle-portable.git
synced 2024-11-22 15:37:55 -05:00
Create CONTRIBUTING.md
This commit is contained in:
parent
526c6af1f6
commit
9de317af55
1 changed files with 31 additions and 0 deletions
31
CONTRIBUTING.md
Normal file
31
CONTRIBUTING.md
Normal file
|
@ -0,0 +1,31 @@
|
||||||
|
# Contributing
|
||||||
|
|
||||||
|
## Learning Decompilation
|
||||||
|
|
||||||
|
Generally, decompilation is a fairly advanced skill. If you aren't already familiar with it, it will likely take you months, or even years, to learn the skills necessary to do it (depending on your current proficiency with C/C++ and x86 assembly). If you're still interested, [part 1 of the decompilation vlog](https://www.youtube.com/watch?v=MToTEqoVv3I) covers the overall process and should give you a starting point that you can dive in from.
|
||||||
|
|
||||||
|
## Ghidra Server
|
||||||
|
|
||||||
|
For documenting the original binaries and generating pseudocode that we decompile with, we primarily use [Ghidra](https://ghidra-sre.org/) (it's free and open source). To help with collaboration, we have a shared Ghidra repository with all of our current work. It is available to the public but read-only; to contribute to it yourself, you'll need approval from a current maintainer.
|
||||||
|
|
||||||
|
To access the Ghidra repository, use the following details:
|
||||||
|
|
||||||
|
- Address: `server.mattkc.com`
|
||||||
|
- Port: `13100`
|
||||||
|
|
||||||
|
## Code Style
|
||||||
|
|
||||||
|
In general, we're not exhaustively strict about coding style, but there are some preferable guidelines to follow:
|
||||||
|
|
||||||
|
- Indent: 2 spaces
|
||||||
|
- `PascalCase` for classes and function names.
|
||||||
|
- `m_camelCase` for member variables.
|
||||||
|
- `g_camelCase` for global variables.
|
||||||
|
|
||||||
|
## Kinds of Contributions
|
||||||
|
|
||||||
|
This repository has only one goal: accuracy to the original executables. As such, we are not likely to accept pull requests that attempt to modernize the code, or improve compatibility in a newer compiler that ends up reducing compatibility in MSVC 4.20. Essentially, accuracy is king, everything else is secondary. For modernizations and enhancements, it's recommended to create a fork downstream from this one instead.
|
||||||
|
|
||||||
|
## Questions?
|
||||||
|
|
||||||
|
For any further questions, feel free to ask in either the [Matrix chatroom](https://matrix.to/#/#isledecomp:matrix.org) or on the [forum](https://forum.mattkc.com/viewforum.php?f=1).
|
Loading…
Reference in a new issue