c - 对小操作使用宏，这是好习惯吗？

Question

我有一小段代码需要读取以 32 位整数位打包的 4 位值。由于我需要多次调用此操作，即使它很简单，我也需要最大速度。 我在思考宏和内联函数，因此我制作了这个宏:

#define UI32TO4(x, p) (x >> ((p - 1) *4) & 15)

我有一个内联函数可以做同样的事情。

static inline Uint8 foo_getval(Uint32 bits, int pos){
    return (bits >> ((pos-1)*4)) & 15;
}

考虑到操作的简单性，并且值已经为这个调用准备好了(所以不可能调用错误的类型，或者传递太大的值等等)，什么是最好的利用？或者，至少，对于以后可能阅读/修改代码的其他人来说是最容易理解的？

编辑!忘了说了，我用的是 C99。

Answer 1

功能更安全。您认为值始终“正确”的假设仅在您开发该代码时成立。您无法判断线下的某个人(或您疲倦时的自己)是否会传递意想不到的值。

编译器会在认为有效时进行内联。尽可能使用类型安全的函数，只有在别无选择时才使用宏。