Shourya Shukla <shouryashukla.oo@xxxxxxxxx> writes: > The definitions of 'verify_submodule_committish()' and > 'print_submodule_summary()' had wrong styling in terms of the asterisk > placement. Amend them. I pointed out only these two, but that does not necessarily mean they are the only ones. Have you checked all the new code added by the series? > Also, the warning printed in case of an unexpected file mode printed the > mode in decimal. Print it in octal for enhanced readability. I actually did check this side ;-) and am reasonably sure that there aren't any other irrational choice of format specifiers. Thanks.