zeroc-ice / ice

All-in-one solution for creating networked applications with RPC, pub/sub, server deployment, and more.
https://zeroc.com
GNU General Public License v2.0
2k stars 592 forks source link

Random Cleanup #2290

Closed InsertCreativityHere closed 2 weeks ago

InsertCreativityHere commented 2 weeks ago

This PR removes all the remaining references to the old cpp11 and cpp98 mappings. It also:

After this PR, we only mention cpp98 is completely gone, and we only mention cpp11 in 3 places. I'm not sure what to do about them, so feel free to advise!

1) in cpp/test/IceUtil/stacktrace/test.py We set this option: TestSuite(__name__, options={"cpp11": [False]}) Can I just delete this option? This whole file?

2) in cpp/test/Ice/scope/AllTests.cpp We have 4 of these comments in the test: "// C++ 11 Future-Based Async Function" I guess it's referring to the language version, and not the mapping, so I left it alone.

3) in Make.project.rules we have a comment which gives this outdated example:

# Check for platform and configuration specific variable. For example, if variable
# is `objdir', for the configuration `cpp11-static' and the `osx' platform, the
# resulting value will be: $(osx_objdir) $(cpp11_objdir) $(static_objdir).

The make file scares me, and I'm not sure how to update the example.