search

OpenRTM / OpenRTP-aist

OpenRTP-aist: RT-Middleware and OMG RTC based component and system development tools implemented by AIST
Other
1 stars 6 forks source link

[RTCB]#ifdef TAO_IDLを無視するように修正 #441

Closed ga-sakamoto closed 2 years ago

ga-sakamoto commented 2 years ago

Identify the Bug

Link to #440

Description of the Change

使用するIDLファイル内に，#ifdef TAO_IDLの定義がある場合には，無視するように修正させて頂きました．

ただ，たいへん申し訳ございませんが，現状では，単純に#ifdefを使用して定義している場合のみ対応しています．

ifdef TAO_IDL内で，更に#ifdefが存在する場合などには対応できておりません．

(考え得るパターンが色々とあるため，正規表現で全てにマッチさせる方法がわからなかったためです)

Verification

n-ando commented 2 years ago

@ga-sakamoto さん、TAO_IDL の ifdefに特化した処理ではなく、ifdefで囲まれたincludeに関しては、対象ファイルがなくても警告を出しつつ処理継続を試みるように変更をお願いします。

n-ando commented 2 years ago

現状でも、includeエラー時に、継続するかどうかの確認ダイアログがでているはずなので、確認。 ワーニングのメッセージを少し変更。

Warning:Include IDL[tao/orb.idl] not found. ↓ Warning:Include IDL[tao/orb.idl] not found. The generated file might be incomplete. It can be solved by adding the include directives. Continue (Y/N)

Warning: IDLファイル [tao/orb.idl] がありません。 生成されたファイルは不完全かもしれません。 生成ファイルにinclude文を追加する必要があるかもしれません。 続けますか？ (Y/N)

ga-sakamoto commented 2 years ago

IDLファイル内のincludeファイルが存在しない場合，表示される確認ダイアログで｢Yes｣を選択すると，コード生成される事を確認しました．

また，その際に表示されるメッセージを上記の内容に修正させて頂きました．