hiyohiyo / CrystalDiskInfo

CrystalDiskInfo
https://github.com/hiyohiyo/CrystalDiskInfo
MIT License
1.59k stars 177 forks source link

CAtaSmart: CorrectAttribute, AMD_RC2 fix #222

Closed GakutoMatsumura closed 1 year ago

GakutoMatsumura commented 1 year ago

English

Bug fixes: [AMD_RC2]

Suggested changes: [CAtaSmart::CorrectDiskAttributeStatus]

  1. The content is similar to CAtaSmart::CheckDiskStatus, so it will be easier to maintain in CAtaSmart.
  2. Since CAtaSmart can manage information for each Attribute, the UI part can be devoted to the UI.

my best regards,

Japanese

不具合修正: [AMD_RC2]

提案としての変更: [CAtaSmart::CorrectDiskAttributeStatus]

  1. CAtaSmart::CheckDiskStatusと類似した内容なので、CAtaSmart内にまとめたほうが今後のメンテナンスが楽になる。
  2. CAtaSmartで完結してAttribute単位の情報を管理できるため、UI部分はUIに専念できる。

よろしくお願いいたします。

GakutoMatsumura commented 1 year ago

CAtaSmart::CorrectDiskAttributeStatusを採用する場合の11672行目の警告の修正 > INT_PTR if( (INT_PTR)index >= vars.GetCount() ) return DISK_STATUS_UNKNOWN;