diff options
| author | Steve Huston <shuston@riverace.com> | 2012-05-24 15:14:20 +0000 |
|---|---|---|
| committer | Steve Huston <shuston@riverace.com> | 2012-05-24 15:14:20 +0000 |
| commit | c4bd4a2781acc18a6ea7c5671241ace49d0603d5 (patch) | |
| tree | 4a53ba016cdbe0969904cf66b563f11f92bbab96 | |
| parent | 9c17e3b28fa599926b9da01c34961c52e3e4d554 (diff) | |
| download | ATCD-c4bd4a2781acc18a6ea7c5671241ace49d0603d5.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 ed90b505cd3..d5da2cdf64e 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 17 17:46:56 EDT 2012 Steve Huston <shuston@riverace.com> * ACE version 6.0b released. diff --git a/ace/Cache_Map_Manager_T.cpp b/ace/Cache_Map_Manager_T.cpp index 7d27cc9a311..68708524581 100644 --- a/ace/Cache_Map_Manager_T.cpp +++ b/ace/Cache_Map_Manager_T.cpp @@ -323,7 +323,7 @@ ACE_Cache_Map_Manager<ACE_T2>::find (const KEY &key) { 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 a23dea8b42d..bc96e726e62 100644 --- a/tests/Cache_Map_Manager_Test.cpp +++ b/tests/Cache_Map_Manager_Test.cpp @@ -241,13 +241,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_ASSERT (result != -1); ACE_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); } |