On 12 May 2016 at 07:29, David Sterba <dsterba@xxxxxxx> wrote:
> On Wed, May 11, 2016 at 07:50:35PM -0400, Nicholas D Steeves wrote:
>> There were a couple of instances where I wasn't sure what to do; I've
>> annotated them, and they are long lines now. To find them, search or
>> grep the diff for 'Steeves'.
>
> Found and updated, most of them real typos, 'strtoull' is name of a C
> library functioon.
Hi David,
Thank you for reviewing this patch. Sorry for missing the context of
the strtoull comment; I should have been able to infer that and am
embarrassed that I failed to. Also, embarrassed because I think I've
used it in some C++ code!
I learned how to use git rebase and git reset today, and can submit a
v2 patch diffed against master at your earliest convenience. My only
remaining question is this:
mkfs.c: printf:("Incompatible features: %s", features_buf)
* Should this be left as "Imcompat features"?
Regards,
Nicholas
--
To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
