Hi Johan, 2010/11/22 Johan Hedberg <johan.hedberg@xxxxxxxxx>: > Hi Jose, > > On Mon, Nov 22, 2010, Jose Antonio Santos Cadenas wrote: >> Also a destroy function is added in order to avoid memory leaks >> --- >> health/mcap.c | 28 +++++++++++++++++++--------- >> 1 files changed, 19 insertions(+), 9 deletions(-) > > The patch content itself is fine but you need to fix the commit message. > Now you have half of the info in the summary and half in the message > body (both halves being equaly relevant). The summary line should be a > summary of the whole patch (in this case it could be e.g. "Fix MDL > reference counting for IO watchers") and the body should contain the > details. So please reformulate, resend, and pay more attention to this > kind of things in the future. Thanks :) Sorry for for the inconvenience, I'm fixing this and sending the new patch. I'll try be more carefully with the commit messages in the future. Regards > > Johan > -- To unsubscribe from this list: send the line "unsubscribe linux-bluetooth" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
