[tor-commits] [collector/release] Avoid calling bytesFor() unnecessarily.
karsten at torproject.org
karsten at torproject.org
Mon Mar 19 15:25:44 UTC 2018
commit 8c1bdd9a8403a0a19149b6388bbc940ee632eaae
Author: Karsten Loesing <karsten.loesing at gmx.net>
Date: Sat Mar 17 10:45:55 2018 +0100
Avoid calling bytesFor() unnecessarily.
Found while looking into #25522.
---
src/main/java/org/torproject/collector/webstats/SanitizeWeblogs.java | 5 ++++-
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/src/main/java/org/torproject/collector/webstats/SanitizeWeblogs.java b/src/main/java/org/torproject/collector/webstats/SanitizeWeblogs.java
index df178b4..fd301b0 100644
--- a/src/main/java/org/torproject/collector/webstats/SanitizeWeblogs.java
+++ b/src/main/java/org/torproject/collector/webstats/SanitizeWeblogs.java
@@ -169,9 +169,12 @@ public class SanitizeWeblogs extends CollecTorMain {
OutputStream os = FileType.XZ.outputStream(baos)) {
for (Map.Entry<String, Long> entry : lines.entrySet()) {
long count = entry.getValue();
- byte[] batch = bytesFor(entry.getKey(), BATCH);
+ byte[] batch = null;
while (count > 0) {
if (count > BATCH) {
+ if (null == batch) {
+ batch = bytesFor(entry.getKey(), BATCH);
+ }
os.write(batch);
count -= BATCH;
} else {
More information about the tor-commits
mailing list