diff options
author | Steve Huston <shuston@riverace.com> | 2012-05-24 15:16:19 +0000 |
---|---|---|
committer | Steve Huston <shuston@riverace.com> | 2012-05-24 15:16:19 +0000 |
commit | edb9365bd77569044e95aa3c4618f772d0938482 (patch) | |
tree | 24485878f974f0427a3207c7fd42a41e848afbac | |
parent | 647a2e5a4444a8486e3e6214208bf29ecfa81d7c (diff) | |
download | ATCD-edb9365bd77569044e95aa3c4618f772d0938482.tar.gz |
ChangeLogTag:Thu May 24 14:35:04 UTC 2012 Steve Huston <shuston@riverace.com>
-rw-r--r-- | ChangeLog | 9 | ||||
-rw-r--r-- | ace/Cache_Map_Manager_T.cpp | 2 | ||||
-rw-r--r-- | tests/Cache_Map_Manager_Test.cpp | 6 |
3 files changed, 15 insertions, 2 deletions
diff --git a/ChangeLog b/ChangeLog index b33fd579b99..61f993fcda1 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,12 @@ +Thu May 24 14:35:04 UTC 2012 Steve Huston <shuston@riverace.com> + + * ace/Cache_Map_Manager_T.cpp (find): Remove extraneous () from + 'second' - leftover from ACE_Pair days. + + * tests/Cache_Map_Manager_Test.cpp: Add call to the above method. + + * THANKS: Thanks to Milind Pangarkar for the above test, and fix. + Tue Apr 3 22:49:11 UTC 2012 Steve Huston <shuston@riverace.com> * ace/SOCK_Dgram_Mcast.cpp (subscribe_ifs): Set the error code when diff --git a/ace/Cache_Map_Manager_T.cpp b/ace/Cache_Map_Manager_T.cpp index fa9caf53041..cd041afc75d 100644 --- a/ace/Cache_Map_Manager_T.cpp +++ b/ace/Cache_Map_Manager_T.cpp @@ -320,7 +320,7 @@ ACE_Cache_Map_Manager<KEY, VALUE, CMAP_TYPE, ITERATOR_IMPL, REVERSE_ITERATOR_IMP { int result = this->caching_strategy_.notify_find (find_result, - cache_value.second ()); + cache_value.second); // Unless the find and notification operations go thru, this // method is not successful. diff --git a/tests/Cache_Map_Manager_Test.cpp b/tests/Cache_Map_Manager_Test.cpp index 7953a125251..e66b590ddd5 100644 --- a/tests/Cache_Map_Manager_Test.cpp +++ b/tests/Cache_Map_Manager_Test.cpp @@ -238,13 +238,17 @@ find_test_cache (MAP_CACHE &cache) { VALUE j = 0; int result = cache.find (lookup_array[i], j); - + result = cache.find (lookup_array[i]); ACE_TEST_ASSERT (result != -1); ACE_TEST_ASSERT (j == lookup_array[i]); if (debug) ACE_DEBUG ((LM_DEBUG, ACE_TEXT ("%d "), j)); + // Once more, with only the key. Test Bugzilla #4029. + result = cache.find (lookup_array[i]); + ACE_TEST_ASSERT (result != -1); + ACE_UNUSED_ARG (result); } |