Sample

int-db09d0974300-I11

Split eligible integer literals into pairs of short-named intermediate variables

Matched original source

Lines151

Scores

MethodScoreScale
Posnett00–1; higher is more readable
Scalabrino0.5780–1; higher is more readable
Dorn0.870–1; higher is more readable
Mi0.2440–1; higher is more readable
LLOC43Logical lines; fewer is more readable
DeepSeek V4 Pro17.670–20; mean of three runs
GPT-6.1 Sol190–20; mean of three runs
BTTF0.4830–1; higher is more readable

Views

Source code

package com.google.common.util.concurrent; import static com.google.common.base.Preconditions.checkNotNull; import static com.google.common.base.Throwables.throwIfUnchecked; import static com.google.common.util.concurrent.Platform.restoreInterruptIfIsInterruptedException; import static java.util.concurrent.Executors.callable; import com.google.common.annotations.GwtIncompatible; import com.google.common.annotations.J2ktIncompatible; import com.google.common.collect.ImmutableList; import com.google.errorprone.annotations.CanIgnoreReturnValue; import java.util.Collection; import java.util.List; import java.util.concurrent.Callable; import java.util.concurrent.ExecutionException; import java.util.concurrent.ExecutorService; import java.util.concurrent.Future; import java.util.concurrent.TimeUnit; import java.util.concurrent.TimeoutException; import org.jspecify.annotations.Nullable; /** * An abstract {@code ExecutorService} that allows subclasses to {@linkplain #wrapTask(Callable) * wrap} tasks before they are submitted to the underlying executor. * * <p>Note that task wrapping may occur even if the task is never executed. * * <p>For delegation without task-wrapping, see {@link ForwardingExecutorService}. * * @author Chris Nokleberg */ @J2ktIncompatible @GwtIncompatible abstract class WrappingExecutorService implements ExecutorService { private final ExecutorService delegate; protected WrappingExecutorService(ExecutorService delegate) { this.delegate = checkNotNull(delegate); } /** * Wraps a {@code Callable} for submission to the underlying executor. This method is also applied * to any {@code Runnable} passed to the default implementation of {@link #wrapTask(Runnable)}. */ protected abstract <T extends @Nullable Object> Callable<T> wrapTask(Callable<T> callable); /** * Wraps a {@code Runnable} for submission to the underlying executor. The default implementation * delegates to {@link #wrapTask(Callable)}. */ protected Runnable wrapTask(Runnable command) { Callable<Object> wrapped = wrapTask(callable(command, null)); return () -> { try { wrapped.call(); } catch (Exception e) { restoreInterruptIfIsInterruptedException(e); throwIfUnchecked(e); throw new RuntimeException(e); } }; } /** * Wraps a collection of tasks. * * @throws NullPointerException if any element of {@code tasks} is null */ private <T extends @Nullable Object> ImmutableList<Callable<T>> wrapTasks( Collection<? extends Callable<T>> tasks) { ImmutableList.Builder<Callable<T>> builder = ImmutableList.builder(); for (Callable<T> task : tasks) { builder.add(wrapTask(task)); } return builder.build(); } // These methods wrap before delegating. @Override public final void execute(Runnable command) { delegate.execute(wrapTask(command)); } @Override public final <T extends @Nullable Object> Future<T> submit(Callable<T> task) { return delegate.submit(wrapTask(checkNotNull(task))); } @Override public final Future<?> submit(Runnable task) { return delegate.submit(wrapTask(task)); } @Override public final <T extends @Nullable Object> Future<T> submit( Runnable task, @ParametricNullness T result) { return delegate.submit(wrapTask(task), result); } @Override public final <T extends @Nullable Object> List<Future<T>> invokeAll( Collection<? extends Callable<T>> tasks) throws InterruptedException { return delegate.invokeAll(wrapTasks(tasks)); } @Override public final <T extends @Nullable Object> List<Future<T>> invokeAll( Collection<? extends Callable<T>> tasks, long timeout, TimeUnit unit) throws InterruptedException { return delegate.invokeAll(wrapTasks(tasks), timeout, unit); } @Override public final <T extends @Nullable Object> T invokeAny(Collection<? extends Callable<T>> tasks) throws InterruptedException, ExecutionException { return delegate.invokeAny(wrapTasks(tasks)); } @Override public final <T extends @Nullable Object> T invokeAny( Collection<? extends Callable<T>> tasks, long timeout, TimeUnit unit) throws InterruptedException, ExecutionException, TimeoutException { return delegate.invokeAny(wrapTasks(tasks), timeout, unit); } // The remaining methods just delegate. @Override public final void shutdown() { delegate.shutdown(); } @Override @CanIgnoreReturnValue public final List<Runnable> shutdownNow() { return delegate.shutdownNow(); } @Override public final boolean isShutdown() { return delegate.isShutdown(); } @Override public final boolean isTerminated() { return delegate.isTerminated(); } @Override public final boolean awaitTermination(long timeout, TimeUnit unit) throws InterruptedException { return delegate.awaitTermination(timeout, unit); } }

Loading feature diagnosis…

Source regions

Feature contributions

Feature contributions

FeatureValueContribution
(log-odds)
Halstead volumeHow much symbolic information the code contains. It combines the total number of operators and values with the number of distinct ones.5186.14−171.143
Source linesThe number of non-empty source lines.151+60.4
Byte entropyHow varied the bytes in the source text are. Repeated bytes give a lower value; a more even mixture gives a higher value.4.87−7.305

Intercept 8.87 + feature contributions = -109.178 (log-odds).
Score = sigmoid(-109.178) = 0.

Score0.578

Feature contributions

FeatureValueContribution
(log-odds)
Comment layout frequencyHow frequently the layout switches between lines containing only comments or whitespace and lines containing code.152−4.346
Indentation frequencyHow rapidly indentation changes from line to line.130+3.759
Identifier layout (vertical)How rapidly the visual pattern of identifier characters changes from top to bottom.150.901+1.699
Comment layout (horizontal)How rapidly the visual pattern of comment characters changes from left to right.96.072+0.746
Identifier layout (horizontal)How rapidly the visual pattern of identifier characters changes from left to right. It measures spatial frequency, not the meaning of the names.99.914−0.567
Number layout frequencyHow rapidly the number of numeric values per line changes through the code.1+0.398
Operator visual areaThe share of the code’s visual area occupied by operators such as +, =, and *.0.083+0.138

Intercept 0.07 + feature contributions = 1.898 (log-odds).
Score = sigmoid(1.898) = 0.87.

Score0.244
Score43
Run 1

This code is highly readable. It follows a clear, logical structure: documentation comments explain the class purpose and wrapping behavior, fields are declared first, then the constructor, protected abstract/overridable methods, private helpers, and finally override methods grouped by behavior. The nesting is minimal, with only a shallow loop and a lambda. Intent is clear from method names like wrapTask, wrapTasks, and delegate. The code is concise—each method performs a single delegation or wrapping action, so action granularity is excellent. Reading flow is natural: the core wrapping logic is presented before the boilerplate delegation. The only minor drawback is the presence of type-use annotations (e.g., <T extends @Nullable Object>) and a slightly dense lambda body, which may add slight noise but are idiomatic for Guava.

Run 2

This code is highly readable due to its clear logical structure, minimal nesting, and consistent patterns. The class is organized into distinct sections: constructor, abstract wrapTask methods, the wrapTasks helper, methods that wrap before delegating, and methods that simply delegate. Nesting is shallow, appearing only in the lambda and loop, both of which are straightforward. Intent is explicit through descriptive method names and Javadoc explaining the wrapping behavior. The code is concise without sacrificing clarity—each method delegates cleanly with a single purpose. Action granularity is excellent: each line or statement performs one clear operation. Reading flow is natural, moving from setup to core wrapping logic to delegation. The only minor drawback is the repetitive delegation methods, but this is typical for wrapper classes and does not hinder understanding. Overall, it is a model of clean, maintainable code with strong separation of concerns and minimal complexity.

Run 3

The code is highly readable. It follows a clear, logical structure with distinct sections for task wrapping and delegation. Nesting is minimal, primarily limited to a single lambda and a loop. Methods clearly express intent and are concise, adhering to single-action-per-line. The reading flow is natural: high-level wrapping methods first, then simple delegation methods. Javadoc is helpful but not excessive. Minor deductions are for slightly verbose generic syntax and the multi-line generic method signatures, which slightly impact scanability.

Run 1

Code Structure: Clearly separates wrapping helpers, task-submission methods, and lifecycle delegation. Nesting: Shallow overall; only the Runnable wrapper introduces a lambda with try/catch. Clarity of intent: Descriptive names, Javadoc, and section comments make the task-wrapping purpose explicit. Code length: Appropriate for implementing ExecutorService; repeated delegation methods are necessary interface boilerplate rather than unnecessary verbosity. Action granularity: Statements generally perform one clear action, though nested wrapping and delegation calls combine steps compactly. Reading flow: Consistent formatting and predictable method grouping support natural top-to-bottom reading. Generic bounds, nullness annotations, and exception-handling utilities add minor visual and conceptual complexity.

Run 2

Code Structure: Clearly separates wrapping helpers, task-submission methods, and lifecycle delegation. Nesting: Shallow overall; only the Runnable adapter introduces a lambda with try/catch. Clarity of intent: Descriptive names, Javadoc, and section comments make the wrapping-and-delegation purpose explicit. Code length: Methods are concise; repetition largely reflects the ExecutorService interface rather than unnecessary verbosity. Action granularity: Statements generally perform one action each, though nested wrapping and delegation calls combine related operations compactly. Reading flow: Progresses naturally from construction and customization hooks to helpers, submission methods, and lifecycle methods. Minor readability costs come from dense generic/nullness annotations, long signatures, and tightly packed imports.

Run 3

Code Structure: Clearly separates wrapping helpers, task-submission methods, and lifecycle delegation. Nesting: Shallow overall; only the Runnable adapter introduces a lambda with try/catch. Clarity of intent: Descriptive names, focused Javadoc, and section comments make the wrapping-and-delegation purpose explicit. Code length: Methods are concise; repetition is largely required by the ExecutorService interface, though annotations and generic signatures add visual bulk. Action granularity: Statements generally perform one clear action, with nested wrapping calls remaining easy to understand. Reading flow: Progresses naturally from construction and customization hooks to submission and lifecycle methods. Minor readability costs come from dense imports and verbose nullness-aware type declarations.