On 05.12.2012 15:30, Konstantin Kolinko wrote: > 2012/12/5 <rj...@apache.org>: >> Author: rjung >> Date: Wed Dec 5 14:00:15 2012 >> New Revision: 1417424 >> >> URL: http://svn.apache.org/viewvc?rev=1417424&view=rev >> Log: >> - Style >> - Fix Javadoc >> - Move default setting to the right place. > > By a quick look: You say "move", but I see only the new code and no > removed code. > Is it as intended?
Yes, although I must confirm it is not totally clear. The move part of TC 7 is about setting the default already when declaring the field and not only during startInternal(). For trunk I also removed the overwriting of an empty configured value by the code defaults (which would have been the "remove" of "move=remove plus add"). This one I kept for TC 7 because of compatibility with existing configs. I could have been more clear about the word "partially" in the commit message .. Will change log revprop. Regards, Rainer --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org For additional commands, e-mail: dev-h...@tomcat.apache.org
