BugReport/75

http://sakura.qp.land.to/?BugReport%2F75


Top > BugReport > 75

ふぁんくらぶ14 243

  • ページ: BugReport
  • 投稿者: ryoji
  • 優先順位: 低
  • 状態: 完了
  • カテゴリー:
  • 投稿日: 2011-04-04 (月) 21:15:33
  • バージョン: ssrc-2-0-1-0.tar.gz

メッセージ

243 :名無しさん@お腹いっぱい。:2011/04/04(月) 22:15:27.76 ID:3xroZ0rf0

   ・macro\CKeyMacroMgr.cpp L328 nEndが不定の可能性
   ・macro\CMacro.h L61 戻り値を返さない関数で戻り値を返すコード
   ・macro\CSMacroMgr.cpp L547 ppMacroがNULLの可能性
   ・prop\CPropComMacro.cpp L555,585 コピー元のほうが大きい
   ・prop\CPropComMainMenu.cpp L915 条件式誤り
   ・typeprop\CDlgTypeList.cpp L160,187,221 コピー元のほうが大きい
   ・types\CType_Java.cpp L294 コピー元のほうが大きい
   ・types\CType_Sql.cpp L133 コピー元のほうが大きい
   ・types\CType_Sql.cpp L135 重複した条件

対処リスト

お手数ですが対処したら下の表の状態を更新してください.

状態:提案/着手/SVN待ち/完了/保留/却下
項番内容状態対策Rev.対策Rev.(ANSI版)
1macro\CKeyMacroMgr.cpp L328 nEndが不定の可能性完了rev1906rev1933
2macro\CMacro.h L61 戻り値を返さない関数で戻り値を返すコード却下対策不要
3macro\CSMacroMgr.cpp L547 ppMacroがNULLの可能性却下対策不要
4prop\CPropComMacro.cpp L555,585 コピー元のほうが大きい完了rev2010rev2014
5prop\CPropComMainMenu.cpp L915 条件式誤り却下対策不要非該当
6typeprop\CDlgTypeList.cpp L160,187,221 コピー元のほうが大きい却下対策不要非該当
7types\CType_Java.cpp L294 コピー元のほうが大きい完了rev2010rev2014
8types\CType_Sql.cpp L133 コピー元のほうが大きい完了rev2010rev2014
9types\CType_Sql.cpp L135 重複した条件完了rev2010rev2014

  • 「戻り値を返さないvoid型関数でreturn (void型になる式);」とするのはそのほうが適切な場合もあるし記述者の好みやコーディングスタイルの問題であって禁止すべきではない -- 2011-04-06 (水) 09:45:00
  • 「戻り値を返さないvoid型関数でreturn (void型になる式);とするのが適切な場合」ってのが思いつかないんですが、具体的にどんな場合ですか? -- 2011-04-07 (木) 19:28:27
  • ある関数の薄いラッパー関数を書くときは戻り値の型に関係なく、return (元関数呼び出し);という記法に統一するのが妥当でしょう。元関数にvoid以外を返すような仕様変更があった場合も最少の修正で済みます。指摘箇所はうっかり間違えたのではなく気を利かせて意図的に書いたもののように見受けられます。void型関数を、戻り値なしの関数としてではなくvoid値を返す関数と捉えれば違和感もないと思います。 -- 2011-04-07 (木) 20:22:48

URL B I U SIZE Black Maroon Green Olive Navy Purple Teal Gray Silver Red Lime Yellow Blue Fuchsia Aqua White
[添付ファイル一覧] [全ページの添付ファイル一覧]
アップロード可能最大ファイルサイズは 2,048KB です。

リロード   新規 編集 凍結 差分 添付 複製 名前変更   ホーム 一覧 単語検索 最終更新 バックアップ リンク元   ヘルプ   最終更新のRSS
Last-modified: 2012-01-16 (月) 16:07:59 (2493d)