>Author: elemings >Date: Thu May 22 13:54:39 2008 >New Revision: 659253 > >URL: http://svn.apache.org/viewvc?rev=659253&view=rev >Log: >2008-05-22 Eric Lemings <[EMAIL PROTECTED]> > > STDCXX-550 [...] > * tests/algorithms/25.generate.cpp (test_generate_n): Explicitly > convert `size_t' value to `int' type to silence 64-bit > conversion warnings. > * tests/algorithms/25.random.shuffle.cpp (test_random_shuffle): > Replaced `size_t' type for line parameters with `int' type. [...]
[...] > * tests/algorithms/25.fill.cpp (test_fill_n), > tests/algorithms/25.generate.cpp (test_generate_n): First > parameter in `Size' constructor should be an `int'. > * tests/algorithms/25.random.shuffle.cpp (test_random_shuffle): > Change line parameter from `size_t' type to `int' type. [...] Looks like there is some duplication of summaries here. Both of these are about the same changes. The log should probably be modified. Also, I noticed that the formatting you're using isn't consistent with what we normally use for changelog entries. As an example, I see that you are indenting some lines. In the above snip you've indented the line that begins with tests/algorithms/25.generate.cpp in the second block. There should be an asterisk there. You are doing the same thing with function names in other places. Also, if the description of the resolution is the same as the entry above, you should probably just write 'Ditto.' and leave it at that. >Modified: stdcxx/branches/4.2.x/tests/algorithms/25.fill.cpp >URL: >http://svn.apache.org/viewvc/stdcxx/branches/4.2.x/tests/algori >thms/25.fill.cpp?rev=659253&r1=659252&r2=659253&view=diff >=============================================================== >=============== >--- stdcxx/branches/4.2.x/tests/algorithms/25.fill.cpp (original) >+++ stdcxx/branches/4.2.x/tests/algorithms/25.fill.cpp Thu May >22 13:54:39 2008 >@@ -146,7 +146,7 @@ > const OutputIterator begin = > make_iter (buf, buf, buf_end, dummy_iter); > >- const Size n (i, 0 /* dummy */); >+ const Size n (int (i), 0 /* dummy */); > const T value; > > // the number of invocations of the assignment operator > Wouldn't the correct way to fix this be to add a member typedef to the Size class template and then use it? That way the code would be correct if Size is actually Size<short> or whatever. _TYPENAME Size::IntegralType(i) is pretty verbose, but it is correct. >Modified: stdcxx/branches/4.2.x/tests/algorithms/25.generate.cpp >URL: >http://svn.apache.org/viewvc/stdcxx/branches/4.2.x/tests/algori >thms/25.generate.cpp?rev=659253&r1=659252&r2=659253&view=diff >=============================================================== >=============== >--- stdcxx/branches/4.2.x/tests/algorithms/25.generate.cpp (original) >+++ stdcxx/branches/4.2.x/tests/algorithms/25.generate.cpp Thu >May 22 13:54:39 2008 >@@ -186,7 +186,7 @@ > const ForwardIterator begin = > make_iter (buf, buf, buf_end, gen_iter); > >- const Size n (i, 0); >+ const Size n (int (i), 0); > const Generator<T> gen (0, 0); > > // store the value of the next element > Same here.
