On 09/28/13 00:08, Shawn Walker wrote:
On 09/27/13 11:29, Erik Trauschke wrote:Done!
On 09/26/13 10:11 PM, Thejaswini wrote:
With all the below comments incorporated the new webrev is at
You changed %(len)s to %(len)i so you should also change %(maxt)s
This really should manually specify more than the maximum number of
terms instead of relying on shell globbing.
Other than that, +1.
I have made the changes and executed the testcase and they run fine.
The webrev is at https://ips.java.net/webrev/tk241774/recur_seach/rev10/
Incase there are no more comments I will push the changes.
[pkg-discuss] Re: Code review request: pkg search bug 15776364