xfs: = vs == typo in ASSERT()

[poige]

$ git show aa9e104
commit aa9e10409eae9dd61a336c6307d0a0b538063970
Author: Dan Carpenter
Date:   Thu Sep 12 00:17:31 2013 +0300

xfs: = vs == typo in ASSERT()

There is a '=' vs '==' typo so the ASSERT()s are always true.

Signed-off-by: Dan Carpenter
    Reviewed-by: Dave Chinner
    Reviewed-by: Eric Sandeen     Signed-off-

Comments

[mynegation]

Classic! Компиляторы же вроде бы давно об этом предупреждают.

[poige]

Ну… Большой беды, конечно, в этом нет, ибо если ожидается такое значение, то установка в такое же, разумеется, не должна повредить, но, опять же, разумеется, проверки в действительности нет.

[levgem]

поэтому есть такой непривычный, неудобный, но разумный совет константу ставить слева

> есть такой непривычный

[poige]

Есть такая книжка - «ален голуб: веревка достаточной длины чтобы выстрелить себе в ногу». Я прочёл её где-то в 1990-х. Кроме этого трюка там было ещё много чего неплохого.