On Tue, 26 Mar 2013 13:57:09 +1030 Rusty Russell <ru...@rustcorp.com.au> wrote:
> Alexandru Gheorghiu <gheorghiuan...@gmail.com> writes: > > > Used PTR_RET function instead of IS_ERR and PTR_ERR. > > Patch found using coccinelle. > > WTF is PTR_RET? PTR_RET doesn't return anything. Why is it called > that? It doesn't even make sense. > > ZERO_OR_PTR_ERR() maybe. > > But what problem are we solving? Insufficient churn in the tree? Code > being too readable? This isn't some hard-to-get right corner case, or a > missed optimization. > > Andrew, what am I missing here? It seemed like a good idea at the time. Merged it two years ago and have since been keenly awaiting an opportunity to use it. It seems that people _have_ been using it, but mainly netfilter people and we know they're all crazy ;) -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/