search

JDimproved / JDim

2ch browser for linux
https://jdimproved.github.io/JDim/
GNU General Public License v2.0
46 stars 11 forks source link

font: Convert signedness for integer type to compare ASCII character #1401

Closed ma8ma closed 3 months ago

ma8ma commented 3 months ago

条件文の比較はASCIIの範囲[0, 127]を想定しているが、char型はCPUアーキテクチャによって符号の有無が変わるので明示的にunsigned char型へ変換して比較します。

また、C言語スタイルのキャストを使用しているとclangに指摘されたためC++スタイルのキャストstatic_cast<...>(value)に変更してコンパイラー警告を修正します。

clang-17のレポート (file pathを一部省略)

src/article/font.cpp:108:38: warning: use of old-style cast [-Wold-style-cast]
  108 |             const int pre_char_num = ( int ) pre_char;
      |                                      ^       ~~~~~~~~