OpenRTM / OpenRTM-aist

OpenRTM-aist: RT-Middleware and OMG RTC implementation in C++ implemented by AIST
https://openrtm.org/
Other
19 stars 12 forks source link

#213 の変更が元に戻っている #505

Closed tonboAkinori closed 5 years ago

tonboAkinori commented 5 years ago

Describe the bug

213 の変更が元にもどっている。

原因不明。

To Reproduce

Reproducibility

Expected behavior

213 の変更が再度取り込まれること。

Screenshots or Logs

Environment

Additional context Add any other context about the problem here.

tonboAkinori commented 5 years ago

本件、マージ時のコンフリクト発生で変更が元に戻った可能性が高いです。

213 (3c84140) の変更

 -      return (size_t)kind < sizeof(kind)/sizeof(char*) ? kinds[kind] : "";
 +      return (size_t)kind < sizeof(kinds)/sizeof(kinds[0]) ? kinds[kind] : "";

203 (22699da) の変更の元となっったコミット(4aaebc84)の変更

-      return (size_t)kind < sizeof(kind)/sizeof(char*) ? kinds[kind] : "";
+      return static_cast<size_t>(kind) < sizeof(kind)/sizeof(char*) ? kinds[kind] : "";

ログを見ると #213 が先に取り込まれ、そのあと 4aaebc84 の変更が取り込まれた。 上記とも変更元のソースは同じであるためコンフリクトが発生しマージ時に下記の #203 の内容となった。

最終的なマージ時の#203 の コミット内容

-      return (size_t)kind < sizeof(kinds)/sizeof(kinds[0]) ? kinds[kind] : "";
+      return static_cast<size_t>(kind) < sizeof(kind)/sizeof(char*) ? kinds[kind] : "";