On 11/05/2011 05:09 PM, Wolfgang Denk wrote: [NOTE: I removed the quoting from the hunks as it would not make any sense]
-int _do_env_set (int flag, int argc, char * const argv[]) +int env_check_apply(const char *name, const char *oldval, + const char *newval, int flag) > Please use only TAB for indentation. Please fix globally. From fs/ubibfs/ubifs.h: int ubifs_decompress(const void *buf, int len, void *out, int *out_len, int compr_type); Here there are 2 tabs + 5 spaces to align arguments from the first line with those on the second line. I did the same (except that in a patch hunk the plus sign before the first tab shifts it to the right). Could you please provide some examples as to what would be the correct coding style for function declarations and/or function calls that spawn on multiple lines? I could not find anything on the topic. As for: - if (ep) { /* variable exists */ + if ((oldval != NULL) /* variable exists */ + && ((flag & H_FORCE) == 0)) { /* and we are not forced */ Ditto. Here I was trying to align parentheses. Could you please provide an example? In my opinion, using only tabs would definitely make the two lines not aligned and make the code much harder to read. As for: + if (himport_ex(&env_htab, (char *)default_environment, + sizeof(default_environment), '\0', 0, + 0, NULL, apply_function) == 0) { What should be the right indentation? Perhaps for function calls you would want something like: + if (himport_ex(&env_htab, (char *)default_environment, + sizeof(default_environment), '\0', 0, + 0, NULL, apply_function) == 0) { As soon as I have these questions answered, I will apply all the other changes you requested, and submit a new patchset. I would very much appreciate some feedback about the idea though. Thank you, Gerlando Falauto _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot