diff options
author | Pierre Neidhardt <ambrevar@gmail.com> | 2015-10-20 15:30:32 +0200 |
---|---|---|
committer | Allan McRae <allan@archlinux.org> | 2015-10-21 06:50:56 +0200 |
commit | 4b3df10d5d53af94475e006c71ed262efcab6f0f (patch) | |
tree | 217e30971faac0586d337f7277aa07c37541d2e0 | |
parent | 2c72c8b822d21c45a8020e49f03691830f3bcff2 (diff) | |
download | pacman-4b3df10d5d53af94475e006c71ed262efcab6f0f.tar.gz pacman-4b3df10d5d53af94475e006c71ed262efcab6f0f.tar.xz |
HACKING: Allow the use of 'sizeof' on values
Signed-off-by: Pierre Neidhardt <ambrevar@gmail.com>
Signed-off-by: Allan McRae <allan@archlinux.org>
-rw-r--r-- | HACKING | 14 |
1 files changed, 3 insertions, 11 deletions
@@ -75,15 +75,7 @@ alpm_list_t *alpm_list_add(alpm_list_t *list, void *data) NOT return(0); -6. The sizeof() operator should accept a type, not a value. (TODO: in certain - cases, it may be better- should this be a set guideline? Read "The Practice - of Programming") - - sizeof(alpm_list_t); - NOT - sizeof(*mylist); - -7. When using strcmp() (or any function that returns 0 on success) in a +6. When using strcmp() (or any function that returns 0 on success) in a conditional statement, use != 0 or == 0 and not the negation (!) operator. It reads much cleaner for humans (using a negative to check for success is confusing) and the compiler will treat it correctly anyway. @@ -92,7 +84,7 @@ alpm_list_t *alpm_list_add(alpm_list_t *list, void *data) NOT if(!strcmp(a, b)) -8. Use spaces around almost all arithmetic, comparison and assignment +7. Use spaces around almost all arithmetic, comparison and assignment operators and after all ',;:' separators. foobar[2 * size + 1] = function(a, 6); @@ -103,7 +95,7 @@ alpm_list_t *alpm_list_add(alpm_list_t *list, void *data) NOT for(a=0;a<n&&n>0;a++,n--) {} -9. Declare all variables at the start of the block. +8. Declare all variables at the start of the block. [source,C] ------------------------------------------- int SYMEXPORT alpm_db_remove_server(alpm_db_t *db, const char *url) |