Re: [PATCH 2/2] sata_mv: More error handling for phy_power_off() in mv_platform_remove()

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, Feb 03, 2015 at 07:48:14AM -0500, Tejun Heo wrote:
> That's too anal to be helpful and too spurious to become even remotely

Let me clarify the "too anal" part a bit.  Let's say people push this
like crazy and we successfully mark all places to either check the
return value or have (void) in front.  It's not difficult to realize
that that would be the exact same situation as right now just with
more visual clutter.  That's the problem when these things become
obsessive - they become indisciminate and thus meaningless.  Annotate
the exceptionals, not the other way around.

Thanks.

-- 
tejun
--
To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Kernel Development]     [Kernel Announce]     [Kernel Newbies]     [Linux Networking Development]     [Share Photos]     [IDE]     [Security]     [Git]     [Netfilter]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Device Mapper]

  Powered by Linux