diff options
| author | Ivan Frade <ifrade@google.com> | 2018-11-15 21:27:20 -0800 |
|---|---|---|
| committer | Ivan Frade <ifrade@google.com> | 2018-11-15 22:36:43 -0800 |
| commit | d9ac7ddf1026123fee6c4477d172d614522dfc08 (patch) | |
| tree | 031a110e54fc1158523fe8971443f8a4d9ba6df2 /org.eclipse.jgit/src/org/eclipse/jgit/lib/ProgressMonitor.java | |
| parent | c56afbf18cb14402705a56ca6a2dff6441208cd3 (diff) | |
| download | jgit-d9ac7ddf1026123fee6c4477d172d614522dfc08.tar.gz jgit-d9ac7ddf1026123fee6c4477d172d614522dfc08.zip | |
Remove unnecessary modifiers from interfaces
From Oracle's "Defining an interface":
"All abstract, default, and static methods in an interface are
implicitly public, so you can omit the public modifier."
(Without any modifier, the interface methods are also abstract, so we
omit also the "abstract")
"In addition, an interface can contain constant declarations. All
constant values defined in an interface are implicitly public, static,
and final. Once again, you can omit these modifiers."
This makes the code more consistent. Now all interfaces under
org.eclipse.jgit follow the guidelines.
Change-Id: I4fe6deb111899ec1b4318ab5a6050f3851fa1fd3
Signed-off-by: Ivan Frade <ifrade@google.com>
Diffstat (limited to 'org.eclipse.jgit/src/org/eclipse/jgit/lib/ProgressMonitor.java')
| -rw-r--r-- | org.eclipse.jgit/src/org/eclipse/jgit/lib/ProgressMonitor.java | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/org.eclipse.jgit/src/org/eclipse/jgit/lib/ProgressMonitor.java b/org.eclipse.jgit/src/org/eclipse/jgit/lib/ProgressMonitor.java index d81ee45c9e..9d8d71a0be 100644 --- a/org.eclipse.jgit/src/org/eclipse/jgit/lib/ProgressMonitor.java +++ b/org.eclipse.jgit/src/org/eclipse/jgit/lib/ProgressMonitor.java @@ -49,7 +49,7 @@ package org.eclipse.jgit.lib; */ public interface ProgressMonitor { /** Constant indicating the total work units cannot be predicted. */ - public static final int UNKNOWN = 0; + int UNKNOWN = 0; /** * Advise the monitor of the total number of subtasks. |