raccy / obs-rtmp-nicolive

ニコニコ生放送用 OBS Studio プラグイン
MIT License
57 stars 8 forks source link

Windows 版 Visual Studio 2015 で、v2.2.0-pre1 でC4716のビルドエラー #27

Open shousa opened 7 years ago

shousa commented 7 years ago

nicolive.cpp(188) error C4716: 'NicoLive::enabledStopBeforeEndTime': 値を返さなければいけません 193行目も同様のエラーと表示される(実際は194行目だと思うが) return true;で値は返していると思うのだが、true固定で返しているのが駄目なのかも? Visual Studio 2015 で開いてビルドしてエラーの出る関数の部分に対して、

pragma warning(push)

pragma warning(disable:4716)

警告出してた部分

pragma warning(pop)

で警告を抑制してあげれば一応ビルドは通るようにはなりますが、 // TODO: 実装! とあるので、 return true; でtrue固定で返すのではなく、 本来あるべき実装をして頂いた方が良いかと思います。

raccy commented 7 years ago

たぶん日本語のコメかもです。最終ではなくす…って、なぜなくなってない>< TODOの部分はpreがとれるころには無くなっていると思います。preってα版ですから。

shousa commented 7 years ago

// TODO: 実装! の日本語コメントを、 // TODO: Implement! の英語コメントに変更で、 pre-1のビルド時に出ていたC4716のエラーはなくなったようです。 今、pre-2タグでチェックアウトしたものだと、 VS2015でも問題なくビルド出来ているのを確認しました。