Skip to content

Enable strict compiler settings #413

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 42 commits into from
Dec 13, 2023
Merged
Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
42 commits
Select commit Hold shift + click to select a range
15e39ec
enable compiler warnings and linting
bpkroth Dec 7, 2023
d66eaba
address some trailing whitespace warnings
bpkroth Dec 7, 2023
9fde340
Merge branch 'main' into strict-compiler
bpkroth Dec 8, 2023
3b3906c
compiler fixups
bpkroth Dec 8, 2023
092e06d
compiler fixups
bpkroth Dec 8, 2023
8038636
add some warning suppressions for unused variables
bpkroth Dec 8, 2023
139c953
omit those entirely
bpkroth Dec 8, 2023
259cca6
fixups
bpkroth Dec 8, 2023
77c74ae
whitespace
bpkroth Dec 8, 2023
7abe2e8
use autoclose
bpkroth Dec 8, 2023
a3f7fec
remove more suppressed cast issues
bpkroth Dec 8, 2023
554406d
unnecessary cast
bpkroth Dec 8, 2023
79e7187
avoid unused errors
bpkroth Dec 8, 2023
d93caf7
add serialization requirements
bpkroth Dec 8, 2023
7f89237
whitespace
bpkroth Dec 8, 2023
ff5507d
more minor avoid raw type issues
bpkroth Dec 8, 2023
fc97c1e
in the couple of utility classes, just suppress more warnings
bpkroth Dec 8, 2023
5220c3f
convert throw type to avoid InterruptedException handling warnings
bpkroth Dec 8, 2023
df432f3
docs
bpkroth Dec 8, 2023
d93c61d
enable deprecations
bpkroth Dec 8, 2023
b0398b1
more compiler warning fixups
bpkroth Dec 8, 2023
db891e9
Merge branch 'main' into strict-compiler
bpkroth Dec 8, 2023
c53ea80
revert order
bpkroth Dec 8, 2023
e5e9c00
need the col++
bpkroth Dec 8, 2023
94385e7
address some unused and other lint issues
bpkroth Dec 8, 2023
448ca24
fixup
bpkroth Dec 8, 2023
1e411d4
fixup
bpkroth Dec 8, 2023
2937b46
add a readme with additional developer notes per PR suggestion
bpkroth Dec 11, 2023
4b22f8d
tweak comment
bpkroth Dec 11, 2023
fa34754
add more developer notes
bpkroth Dec 11, 2023
be0f414
include some extension recommendations
bpkroth Dec 11, 2023
43b5cee
update extensions
bpkroth Dec 11, 2023
4556f10
address more unused warnings
bpkroth Dec 11, 2023
5939e2d
ignore another error
bpkroth Dec 11, 2023
4f0fe4c
Merge branch 'main' into strict-compiler
bpkroth Dec 13, 2023
ebb52de
addressing some new serialization warnings
bpkroth Dec 13, 2023
adb1a96
comments
bpkroth Dec 13, 2023
acd47aa
adjust constructor to avoid this-escape errors
bpkroth Dec 13, 2023
d350c6e
convert a pile of classes to 'final' to avoid this-escape errors
bpkroth Dec 13, 2023
3437b2d
auto-whitespace
bpkroth Dec 13, 2023
b2e7489
comments about this-escape warnings
bpkroth Dec 13, 2023
b526294
exception
bpkroth Dec 13, 2023
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
more compiler warning fixups
  • Loading branch information
bpkroth committed Dec 8, 2023
commit b0398b136c1de5656031a33ad31909902f8d4073
1 change: 1 addition & 0 deletions src/test/java/com/oltpbenchmark/api/AbstractTestCase.java
Original file line number Diff line number Diff line change
Expand Up @@ -175,6 +175,7 @@ private int findAvailablePort() throws IOException {
}

try (ServerSocket testSocket = new ServerSocket(port)) {
assert testSocket != null;
return port;
} catch (BindException e) {
// This port is already in use. Continue to next port.
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,6 @@

import com.oltpbenchmark.api.AbstractTestWorker;
import com.oltpbenchmark.api.Procedure;
import com.oltpbenchmark.api.Worker;
import com.oltpbenchmark.benchmarks.auctionmark.util.UserId;

import java.io.IOException;
Expand Down Expand Up @@ -61,9 +60,9 @@ public void testUniqueSellers() throws Exception {
Set<UserId> all_users = new HashSet<UserId>();
Set<UserId> worker_users = new TreeSet<UserId>();
Integer last_num_users = null;
for (Worker w : this.workers) {
AuctionMarkWorker worker = (AuctionMarkWorker) w;
for (var w : this.workers) {
assertNotNull(w);
AuctionMarkWorker worker = (AuctionMarkWorker) w;

// Get the uninitialized profile
AuctionMarkProfile profile = worker.getProfile();
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@

package com.oltpbenchmark.benchmarks.auctionmark.util;

import static junit.framework.Assert.assertFalse;
import static org.junit.Assert.assertFalse;
import static org.junit.Assert.assertEquals;
import static org.junit.Assert.assertNotNull;
import static org.junit.Assert.assertTrue;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -122,6 +122,7 @@ public void testPop() {
expected[i] = rng.astring(1, 32);
} // FOR

@SuppressWarnings("rawtypes")
Collection<String>[] collections = new Collection[]{
CollectionUtil.addAll(new ListOrderedSet<String>(), expected),
CollectionUtil.addAll(new HashSet<String>(), expected),
Expand Down