Re: [PATCH 3/3] Staging: lustre: tracefile: Remove function prototype

2015-11-06 Thread Shivani Bhardwaj
On Sat, Nov 7, 2015 at 1:02 PM, Dilger, Andreas wrote: > On 2015/11/06, 10:19, "Shivani Bhardwaj" wrote: > >>Remove the prototype of function cfs_trace_free_string_buffer() as it is >>no longer needed. > > These patches would be a lot more useful if the summary contained the name > of the

Re: [PATCH 3/3] Staging: lustre: tracefile: Remove function prototype

2015-11-06 Thread Dilger, Andreas
On 2015/11/06, 10:19, "Shivani Bhardwaj" wrote: >Remove the prototype of function cfs_trace_free_string_buffer() as it is >no longer needed. These patches would be a lot more useful if the summary contained the name of the function being removed, rather than "remove function prototype" and

Re: [PATCH 3/3] Staging: lustre: tracefile: Remove function prototype

2015-11-06 Thread Shivani Bhardwaj
On Sat, Nov 7, 2015 at 1:02 PM, Dilger, Andreas wrote: > On 2015/11/06, 10:19, "Shivani Bhardwaj" wrote: > >>Remove the prototype of function cfs_trace_free_string_buffer() as it is >>no longer needed. > > These patches would be a lot more useful

Re: [PATCH 3/3] Staging: lustre: tracefile: Remove function prototype

2015-11-06 Thread Dilger, Andreas
On 2015/11/06, 10:19, "Shivani Bhardwaj" wrote: >Remove the prototype of function cfs_trace_free_string_buffer() as it is >no longer needed. These patches would be a lot more useful if the summary contained the name of the function being removed, rather than "remove