summary refs log tree commit diff stats
path: root/share/doc/HACKING
diff options
context:
space:
mode:
authorBerke Viktor <bviktor@hexchat.org>2012-10-26 01:03:52 +0200
committerBerke Viktor <bviktor@hexchat.org>2012-10-26 01:03:52 +0200
commitfb8d1a383da0bb39465c2010d8d5a158e7490a28 (patch)
treefe92499d866459cc3cf3cde622cf09ed628b5389 /share/doc/HACKING
parenta3c3608b1ddb0b37331b12ac40ad9aee39b7c9cb (diff)
Converting, renaming, relocating
Diffstat (limited to 'share/doc/HACKING')
-rw-r--r--share/doc/HACKING51
1 files changed, 51 insertions, 0 deletions
diff --git a/share/doc/HACKING b/share/doc/HACKING
new file mode 100644
index 00000000..5d68d0df
--- /dev/null
+++ b/share/doc/HACKING
@@ -0,0 +1,51 @@
+Just some tips if you're going to help with HexChat code (patches etc):
+
+* Use tabs, not spaces, to indent code.
+
+* Use a tab size of 3 (most editors will let you choose this).
+  Type :set ts=3 in vim/gvim.
+
+* Try to stick to the same consistant coding style:
+
+void
+routine (void)
+{
+	if (function (a, b, c))
+	{
+		x = a + 1;
+	}
+}
+
+	(vertically aligned braces, a space after if, while, functions etc).
+
+* 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:
+
+	fh = open ("file", OFLAGS | O_RDONLY);
+
+* Use closesocket() for sockets, and close() for normal files.
+
+* Don't read() from sockets, instead use recv().
+
+* Please provide unified format diffs (run diff -u).
+
+* Call your patch something more meaningfull than hexchat.diff (I get a
+  million of these files!).
+
+* To make a really nice and clean patch, do something like this:
+
+Have two directories, unpacked from the original archive:
+hexchat-2.0.0/
+hexchat-2.0.0p1/
+Then edit/compile the hexchat-2.0.0p1 directory. When you're done, make
+a patch with:
+
+cd hexchat-2.0.0p1
+make distclean
+cd ..
+diff -urN hexchat-2.0.0 hexchat-2.0.0p1 > hexchat-something.diff
+
+If using nmake (Windows) replace "make distclean" with "nmake -f makefile.msc clean"