[REDMINE1D-397] [RM-8633] [code] Remove compilation warnings at swig stage Created: 03/Feb/24  Updated: 03/Feb/24

Status: Open
Project: 1D Redmine
Component/s: None
Affects Version/s: None
Fix Version/s: None

Type: Task Priority: Normal
Reporter: Redmine-Jira Migtation Assignee: Redmine-Jira Migtation
Resolution: Unresolved Votes: 0
Labels: None
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified


 Description   

Created on 2024-02-02 10:30:29 by Ali Allaoui. % Done: 0

@boost/smart_ptr/detail/shared_count.hpp:395:33: warning: ‘template<class> class std::auto_ptr’ is deprecated [-Wdeprecated-declarations]
395 | explicit shared_count( std::auto_ptr<Y> & r ): pi_( new sp_counted_impl_p<Y>( r.get() )@ -> Upgrade boost ?
@net/CESAM/amazed/venvs/shared/include/boost/mpl/assert.hpp: At global scope:
/net/CESAM/amazed/venvs/shared/include/boost/mpl/assert.hpp:188:21: warning: unnecessary parentheses in declaration of ‘assert_arg’ [-Wparentheses]
188 | failed ************ (Pred::************@ idem

@ RedshiftLibrary/RedshiftLibrary/linemodel/abstractfitter.h:120:19: warning: comparison of integer expressions of different signedness: ‘std::__shared_ptr_access<int, __gnu_cxx::_S_atomic, false, false>::element_type’

{aka ‘int’}

and ‘std::vector<std::shared_ptr<const NSEpic::CSpectrum> >::size_type’

{aka ‘long unsigned int’}

[-Wsign-compare]
120 | if (*m_curObs >= m_inputSpcs->size())
@
fix other lines in abstractfitter.h and in continuummanager.h

 Comments   
Comment by Redmine-Jira Migtation [ 03/Feb/24 ]

Comment by Didier Vibert on 2024-02-02 13:21:20:
oui tout à fait,

  • pour le premier point, c'est bizarre, on ne devrait pas utiliser les smart ptr boost...
  • 2ème point,on n'a pas la main ? effectivement upgrade boost ?
  • j'avais hésité à le faire, pour le 3ème point. Il faut remplacer les @Int32@ par des @size_t@ dans les indices de boucles sur des index de vecteurs.
    rmq: ces boucles ne devraient peut-être pas se retrouver dans des header, qui génère le warning à la compil du cpp généré par swig. Mais ça fait pas de mal de corriger ça de toute façon, même si dans les fichiers cpp on a des index de vecteur en Int32 en peu partout...
Generated at Sat Feb 10 15:32:19 JST 2024 using Jira 8.3.4#803005-sha1:1f96e09b3c60279a408a2ae47be3c745f571388b.