[collada2eus_urdfmodel.cpp] print links in the order of link names #252
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
#250 (comment) に対応したPull Requestです。
これまで、collada2eusのソースコードに変更を加える度に、生成されるeusモデルのリンクの宣言順が変わってしまって、生成されるeusモデルに大きなdiffが発生してしまい、collada2eusに加えた変更が妥当かどうかの検証がしづらい、という問題が有りました。
考えられる原因は、次のとおりです。
collada2eusは、
jsk_model_tools/euscollada/src/collada2eus_urdfmodel.cpp
Lines 777 to 781 in 6749740
jsk_model_tools/euscollada/src/collada2eus_urdfmodel.cpp
Lines 1862 to 1866 in 6749740
このPull Reqeustは、mapのイテレータを回したときに、リンク名のアルファベット順で回るように変更することで、この問題を解決するものです。