Review Request 124237: Remove ConvenientIterator::ref usage from oldcpp codebase.

Milian Wolff mail at milianw.de
Thu Jul 2 22:19:29 UTC 2015


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/124237/
-----------------------------------------------------------

Review request for KDevelop.


Repository: kdevelop


Description
-------

This requires some changes, as we cannot take the ptr address of
the macro anymore. Instead, we use them as value classes everywhere.
This may potentially degrade performance, but I don't have the time
to measure it right now with a release build and compare the impact.


Diffs
-----

  languages/cpp/cppduchain/cppduchain.cpp 9ec2eae62cfda472f5ebe3e2d92102df9bdad201 
  languages/cpp/cppduchain/cpppreprocessenvironment.h 0d74f58ac44ec43134d5a4041863bef709e74ce3 
  languages/cpp/cppduchain/cpppreprocessenvironment.cpp c99118cec7080a38814a86d92f744f6d441b2e5a 
  languages/cpp/cppduchain/environmentmanager.h fe9743e8144fc9c822d7ec22290aa0a2fb469333 
  languages/cpp/cppduchain/environmentmanager.cpp 21e2547253c9906c9c98da17648277ecd84df604 
  languages/cpp/cppduchain/tests/test_environment.cpp f9002881523da8a65a43ab11f4b35c701149dfc2 
  languages/cpp/cpplanguagesupport.h c4decfd237f6cd901dddb960520b480c64668216 
  languages/cpp/cpplanguagesupport.cpp 282e1ff247d88247a9a7f29ab70d3ea45cf5be21 
  languages/cpp/cppparsejob.cpp 73da7553bd714165ce676019a3bf5eb470d532a1 
  languages/cpp/parser/rpp/pp-engine.cpp 0eb423119fc2abf1f9d82fb2fa963c32987426ce 
  languages/cpp/parser/rpp/pp-environment.h 07e6dddeb4c7d5301870dbc206b9aa1673bb2c90 
  languages/cpp/parser/rpp/pp-environment.cpp beabb71f400dea7db400c8233e7d7bc6aec45dc6 
  languages/cpp/parser/rpp/pp-macro-expander.h b16b0c26f50b7984e6bcb283da1196d05a1d55fd 
  languages/cpp/parser/rpp/pp-macro-expander.cpp 11867d451ee960c52f0ab1a5a42a49791a6f462d 
  languages/cpp/parser/rpp/pp-macro.h 0e39b3f945324359173b11cb3a72ae24ffcb20dc 
  languages/cpp/parser/rpp/pp-macro.cpp 1742dc4a1bab81831e24d1ee2842dc57145923fe 

Diff: https://git.reviewboard.kde.org/r/124237/diff/


Testing
-------


Thanks,

Milian Wolff

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20150702/dbd813ea/attachment.html>


More information about the KDevelop-devel mailing list