ACE_TAO icon indicating copy to clipboard operation
ACE_TAO copied to clipboard

Bug 4207

Open milan-mpathix opened this issue 8 years ago • 6 comments

Bug fix for Bug 4207:

  • added test case exposing the bug
  • fixed memory leak in TAO/tao/Cache_Entries
  • fixed semantics of ACE_Multi_Hash_Map_Manager, to preserve pointers of existing elements when new elements are added or removed
  • Transport_Cache_Manager uses multi-hash map instead of map of indexed elements
  • removed index_ from Cache_ExtId

milan-mpathix avatar Nov 07 '15 18:11 milan-mpathix

Have you compiled and ran all TAO unit tests? That is something we can't do on travis/snap, maybe you can setup a branch build to show that all tests do work without problems at runtime? This change is in the core of TAO and it could be that there are unforeseen issues.

jwillemsen avatar Nov 16 '15 19:11 jwillemsen

I will re-run all the tests, and verify it compiles/runs with versioned namespaces.

milan-mpathix avatar Nov 16 '15 19:11 milan-mpathix

@jwillemsen (setup a branch build) .... not sure how to do this...

What is the simplest way to post build/test results. I will run $ACE_ROOT/bin/auto_run_tests.pl and capture all the output, but the resulting file tends to be quite large...

milan-mpathix avatar Nov 16 '15 20:11 milan-mpathix

Check the scoreboard for existing builds, the autobuild repo has all tools and scripts we use

jwillemsen avatar Nov 16 '15 20:11 jwillemsen

Please see the scoreboard: http://support.mpathix.com/tao_scoreboard/index.html

milan-mpathix avatar Nov 19 '15 15:11 milan-mpathix

All patches for this pull request are complete. Please review.

milan-mpathix avatar Nov 30 '15 16:11 milan-mpathix