Re: [Bluez-devel] [DBUS-PATCH] cleanup

[Marcel Holtmann <marcel@holtmann.org>]

Hi Claudio,

> There are some programming styles in the code that we need to change,
> because this code is getting ugly otherwise.

here are some more coding style things I like to see. This is the bad
example:

	for (i = 0; i < 10; i++) {
		if (test_something()) {
			do_something();
		}
	}

This will end up in some very nested loops. So the better way to write
it is this:

	for (i = 0; i < 10; i++) {
		if (!test_something())
			continue;

		do_something();
	}

Please keep these things in mind when coding. It will make it a lot more
readable and understandable for outsiders.

Regards

Marcel




-------------------------------------------------------
This SF.Net email is sponsored by:
Power Architecture Resource Center: Free content, downloads, discussions,
and more. http://solutions.newsforge.com/ibmarch.tmpl
_______________________________________________
Bluez-devel mailing list
Bluez-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/bluez-devel