joerghoh commented on code in PR #401:
URL:
https://github.com/apache/jackrabbit-filevault/pull/401#discussion_r2627795864
##########
vault-core/src/main/java/org/apache/jackrabbit/vault/fs/impl/AggregateImpl.java:
##########
@@ -714,12 +675,17 @@ private void prepare(Node node, boolean descend)
// get a node iterator suitable for the current node and the
applicable filters
NodeIterator nIter = getNodeIteratorFor(node, filter);
+ int visited = 0;
+ long startTime = System.nanoTime();
+
// include "our" nodes to the include set and delegate the others to
the
// respective aggregator building sub aggregates
while (nIter.hasNext()) {
Node n = nIter.nextNode();
+ visited += 1;
Review Comment:
nope :-) In my opinion it's just more readable, as with "+=" I always need
to parse the number behind it, while "++" is hardwired to "increase a counter"
in my brain.
But I am nitpicking here :-)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
