Previous fix for NOT RUN tab (6702426) fixed some behavior, but the custom filter was still not always updating things.
Integrated review feedback in rev 1438. Closing this review.
Looks good to me. A couple minor comments:
1) new method [i]void update(InterviewParameters ip) [/i] introduced but not used.
2) new constants added for all, but 4
private static int KEY_FILTER = 0;
private static int URL_FILTER = 1;
private static int JTX_FILTER = 2;
private static int STATUS_FILTER = 3;
activeFilters[KEY_FILTER] = keyFilter;
activeFilters[URL_FILTER] = urlFilter;
activeFilters[JTX_FILTER] = jtxFilter;
activeFilters[STATUS_FILTER] = statusFilter;
activeFilters = tsfFilter;
Would it make sense to define one more constant?
Unmodified code integrated, rev 1420.
Yes Dima, I didn't do that cuz I think was thinking something else at the time, but now I agree. I'll make a followup change for that.
Your use of this web site or any of its content or software indicates your agreement to be bound by these Terms of Participation.
Copyright © 2015, Oracle and/or its affiliates. All rights reserved. Oracle and Java are registered trademarks of Oracle and/or its affiliates. Other names may be trademarks of their respective owners.