SKELETON::View: Fix member initialization (#260)

メンバ変数の初期化を変更してcppcheckの警告 (warning) Member variable 'View::XXX' is not initialized in the constructor. を修正する。 [src/skeleton/view.cpp:19]: (warning) Member variable 'View::m_width_client' is not initialized in the constructor. [src/skeleton/view.cpp:19]: (warning) Member variable 'View::m_height_client' is not initialized in the constructor.
JDimproved · May 1, 2020 · b92e38a · b92e38a
1 parent d3406cf
commit b92e38a
Show file tree

Hide file tree

Showing 2 changed files with 18 additions and 29 deletions.
diff --git a/src/skeleton/view.cpp b/src/skeleton/view.cpp
@@ -17,22 +17,11 @@
 using namespace SKELETON;
 
 View::View( const std::string& url, const std::string& arg1 ,const std::string& arg2 )
-    : m_url( url ),
-      m_parent_win( nullptr ),
-      m_status( std::string() ),
-      m_enable_mg( false ),
-      m_enable_autoreload( false ),
-      m_autoreload_mode( AUTORELOAD_NOT ),
-      m_autoreload_sec( 0 ),
-      m_autoreload_counter( 0 ),
-      m_keyjump_counter( 0 ),
-      m_keyjump_num( 0 ),
-      m_lockable( true ),
-      m_locked( false ),
-      m_writeable( true ),
-      m_id_toolbar( 0 ),
-      m_popup_upside( false ),
-      m_reget( false )
+    : m_url( url )
+    , m_autoreload_mode( AUTORELOAD_NOT )
+    , m_lockable( true )
+    , m_writeable( true )
+    , m_id_toolbar( 0 ) // 派生クラスでコンテキストが異なる
 {}
 
 

diff --git a/src/skeleton/view.h b/src/skeleton/view.h
@@ -27,11 +27,11 @@ namespace SKELETON
         SIG_RESIZE_POPUP m_sig_resize_popup;
 
         std::string m_url;
-        Gtk::Window* m_parent_win;
+        Gtk::Window* m_parent_win{};
 
         // クライアント領域の幅、高さ
-        int m_width_client;
-        int m_height_client;
+        int m_width_client{};
+        int m_height_client{};
 
         // 入力コントローラ
         CONTROL::Control m_control;
@@ -51,38 +51,38 @@ namespace SKELETON
         std::string m_status;
 
         // true ならマウスジェスチャ使用
-        bool m_enable_mg; 
+        bool m_enable_mg{};
 
         // オートリロード
-        bool m_enable_autoreload; // true ならオートリロード可能(デフォルト:off)
+        bool m_enable_autoreload{}; // true ならオートリロード可能(デフォルト:off)
         int m_autoreload_mode; // モード
-        int m_autoreload_sec; // 何秒おきにリロードするか
-        int m_autoreload_counter; // オートリロード用のカウンタ
+        int m_autoreload_sec{}; // 何秒おきにリロードするか
+        int m_autoreload_counter{}; // オートリロード用のカウンタ
 
         // キーボード数字入力ジャンプ用
-        int m_keyjump_counter;
-        int m_keyjump_num;
+        int m_keyjump_counter{};
+        int m_keyjump_num{};
 
         // ロック可能か
         bool m_lockable;
 
         // ロック状態
-        bool m_locked;
+        bool m_locked{};
 
         // 書き込み可能か
         bool m_writeable;
 
-        // ツールバーのID
+        // ツールバーのID (派生クラスでコンテキストが異なる)
         int m_id_toolbar;
 
         // 検索文字列
         std::string m_search_query;
 
         // ポップアップ時に全ての領域を表示できないならカーソルの上に表示
-        bool m_popup_upside;
+        bool m_popup_upside{};
 
         // ロード時にキャッシュを削除してからviewを再読み込みする
-        bool m_reget;
+        bool m_reget{};
 
       protected: