facebook / facebook/zstd

Disabled assert() could use its argument to avoid unused variable warnings

オープン
#2,868 コメント 1 件 リアクション 0 件 担当者 0 名 GitHub で見る
主要言語
C
スター
27.9k
フォーク
2.6k
平均マージ
1日 3時間
マージ済み PR(30日)
8

説明

Test this construction for disabling assertions:

```c
#define assert(condition) \
do { \
if (0) { \
(void)(condition); \
} \
} while (0)
```

Additionally, do the same for `RAWLOG()` and `DEBUGLOG()` when `DEBUGLEVEL < 2`.

This would avoid unused variable warnings that show up in production.

コントリビューションガイド

コントリビューションガイドを開く

評価

この issue はまだ評価されていません。

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。