hexchat/share/doc/hacking.md

58 lines
1.5 KiB
Markdown
Raw Normal View History

2012-10-27 19:31:51 +02:00
# HexChat Hacking Guidelines
Just some tips if you're going to help with HexChat code (patches etc):
2013-05-13 13:38:01 +02:00
* Use tabs, not spaces, to indent and align code.
2012-10-27 19:31:51 +02:00
2013-05-13 13:38:01 +02:00
* Use a tab size of 4 (most editors will let you choose this).
Type :set ts=4 in vim/gvim.
2012-10-27 19:31:51 +02:00
2012-10-28 09:55:45 +01:00
* Try to stick to the same consistant coding style (vertically aligned braces, a space after if, while, functions etc.):
2012-10-27 19:31:51 +02:00
2012-10-28 09:58:09 +01:00
<pre>void
routine (void)
2012-10-27 19:31:51 +02:00
{
2012-10-28 09:58:09 +01:00
if (function (a, b, c))
{
x = a + 1;
}
}</pre>
2012-10-27 19:31:51 +02:00
* Don't use "//" C++ style comments, some compilers don't like them.
* When opening a file with Unix level functions (open, read/write, close)
as opposed to the C level functions (fopen, fwrite/fread, fclose), use
the OFLAGS macro. This makes sure it'll work on Win32 aswell as Unix e.g.:
<pre>fh = open ("file", OFLAGS | O_RDONLY);</pre>
* Use closesocket() for sockets, and close() for normal files.
* Don't read() from sockets, use recv() instead.
* Please provide unified format diffs (run diff -u).
* Call your patch something more meaningfull than hexchat.diff.
* To make a really nice and clean patch, do something like this:
* Have two directories, unpacked from the original archive:
<pre>hexchat-2.9.0/
hexchat-2.9.0p1/</pre>
* Then edit/compile the hexchat-2.9.0p1 directory. To create a patch:
* Windows:
<pre>rmdir /q /s hexchat-2.9.0p1/win32/build
rmdir /q /s hexchat-2.9.0p1/win32/build-xp
diff -ruN --strip-trailing-cr hexchat-2.9.0 hexchat-2.9.0p1 > hexchat-something.diff
</pre>
* Unix:
<pre>diff -ruN hexchat-2.9.0 hexchat-2.9.0p1 > hexchat-something.diff</pre>