lkml.org 
[lkml]   [2010]   [Feb]   [11]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
SubjectRE: [PATCH 2/2] Staging: hv: General maintenance of TODO file
Date
>> 
>> Removed two line;
>> Audit block driver
>> Audit scsi driver
>> They were added by us as part of the original code
>> Submission to make sure these drivers kept on working
>> Correctly with all the patches that came in from the
>> Community.
>
>Um, no, those things still need to be done.

What additional things are expected? We solved the original
reason we added the lines, so I want to make sure we do
additional work as is expected. We are going through general
code cleanup, and the TODO file now.

>> Added a new line to fix mostly insane camelcase naming
>> We are working on that right now, but it will take a bit
>> Due to the pervasive usage of it.
>
>That's fine, but it fits under the general "fix coding style issues".
>
>Hm, that isn't spelled out, but it is implicit, I doubt this really
>needs to be documented in the file, it's just common knowledge :)

Want me to add a proper line item for that? Or drop this patch
completely for the time being?

Thanks,

Hank.



\
 
 \ /
  Last update: 2010-02-11 23:25    [W:0.070 / U:0.432 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site