Zach Brown <zab@xxxxxxxxx> wrote: > /* haha, continuing the fine tradition of terrible names in this api.. */ > static inline void *PTR_PTR(void *err_ptr) { > BUG_ON(!IS_ERR(err_ptr) || !err_ptr); > return err_ptr; > } How about ERR_CAST() instead? Or maybe CAST_ERR()? struct dentry *blah(...) { struct inode *inode; inode = thing(...); if (IS_ERR(inode)) return ERR_CAST(inode); } Where ERR_CAST is defined as: static inline void *ERR_CAST(const void *error) { return (void *) x; } David - To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html