lkml.org 
[lkml]   [2008]   [Sep]   [10]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH] drivers/ide/ide-probe.c: uninitialized variable 'rc'

* Steven Noonan <steven@uplinklabs.net> wrote:

> Fixed the warning by initializing 'rc' to zero.
>
> Signed-off-by: Steven Noonan <steven@uplinklabs.net>
> ---
> drivers/ide/ide-probe.c | 2 +-
> 1 files changed, 1 insertions(+), 1 deletions(-)

i've Cc:-ed Bartlomiej, who maintains drivers/ide/ide-probe.c. Your
fix/cleanup looks fine to me.

About the "whom to Cc:" question. Sadly, the MAINTAINERS file is
non-obvious to parse to newbies: there's no clear mapping from file to
maintainer. (it's rather useless even to oldbies.)

The method i use to determine whom to Cc: if i change something in a
file is this ~/bin/git-authors script:

#!/bin/bash
git log $@ | grep Author: | cut -d: -f2 | sort | uniq -c | sort -n | tail -5

(also attached)

for drivers/ide/ide-probe.c, it gives:

earth4:~/tip> git-authors-email drivers/ide/ide-probe.c
2 Christoph Lameter <christoph@lameter.com>
2 Greg Kroah-Hartman <gregkh@suse.de>
3 Jens Axboe <jens.axboe@oracle.com>
4 Jens Axboe <axboe@suse.de>
135 Bartlomiej Zolnierkiewicz <bzolnier@gmail.com>

if there output of git-authors-email is too flat and you are unsure
about whom to Cc:, take a look at git-log drivers/ide/ide-probe.c output
and chose the people who do material changes to a file (not drive-by
changes).

Ingo

#[ $# -lt "1" ] && { echo 'usage: git authors <files...>'; exit -1; }

git log $@ | grep Author: | cut -d: -f2 | sort | uniq -c | sort -n | tail -5


\
 
 \ /
  Last update: 2008-09-10 08:59    [W:0.063 / U:0.168 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site