On Wed, Jul 1, 2015 at 2:22 PM, Sohny Thomas <sohnytho...@zoho.com> wrote: > > > On Wednesday 01 July 2015 05:37 PM, Frans Klaver wrote: >> On Wed, Jul 1, 2015 at 1:56 PM, Sohny Thomas <sohnytho...@zoho.com> wrote: >>> - removed non-existant issue from TODO file >> >> s,existant,existent, > Thanks missed that >> >>> kuid_t or uid_t not present in staging/android >>> - fixed 80 char limit exceeding line >>> - a newline after decelartions as per checkpatch.pl >>> - fixed an unnecessary typedef as reported by checkpatch.pl >> >> Fix one issue per patch, please. > Since these were all simple Fixes of about 1/2 lines , I thought to make a > single patch.
They are simple fixes, but a reviewer still has to figure out what comment belongs to which code change. Since there's no reason for these changes to be atomic, you might as well split them up to make reviewing easier. Frans -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/