Hi Johan, On Mon, Dec 27, 2010 at 5:48 PM, Johan Hedberg <johan.hedberg@xxxxxxxxx> wrote: > Beware of words like "also" and "and" in commit messages. Often they are > a good indication that the patch shold be split into multiple ones. > Though in this case I can't really make up my mind if it's fine since > the changes are so strongly related. I'll leave it up to you to think > about it and decide if you can cleanly split this into two patches or > not. I can split into two patches without problem. > It looks to me like shortened names are completely ignored after this > patch, even to the extent that if there's a shortened name the memory > will be leaked if adapter_update_found_devices returns TRUE. > > If we don't have any name in storage and all we have is a shortened name > then I think it should be included in the DeviceFound signal (which is > what the existing code does). You are right. Somehow I forgot about the shortened name case while trying to refactor this part of the code. I'll resend the patch with just the memory leak fix (taking care to not introduce another one!) and if I still have any idea to simplify this code (which I don't believe anymore) a second patch will come. Regards, -- Anderson Lizardo OpenBossa Labs - INdT Manaus - Brazil -- 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
