Skip to content

Commit 3ec726f

Browse files
authored
Remove local build fixes that were merged with ECH feature branch (#1395)
1 parent 0e1bae5 commit 3ec726f

File tree

3 files changed

+2
-27
lines changed

3 files changed

+2
-27
lines changed

android/lint.xml

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -28,10 +28,4 @@
2828
<issue id="Assert">
2929
<ignore path="**/common/src/main/java/org/conscrypt/OpenSSLCipherChaCha20.java" />
3030
</issue>
31-
32-
<!-- Workaround for "Unexpected failure during lint analysis". -->
33-
<issue id="LintError">
34-
<ignore regexp=".*module-info\.class.*"/>
35-
</issue>
36-
3731
</lint>

common/src/jni/main/cpp/conscrypt/native_crypto.cc

Lines changed: 2 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -7835,12 +7835,7 @@ static int sslSelect(JNIEnv* env, int type, jobject fdObject, AppData* appData,
78357835
if (fds[1].revents & POLLIN) {
78367836
char token;
78377837
do {
7838-
// TEMP - fixes build error
7839-
int foo = 0;
7840-
foo = read(appData->fdsEmergency[0], &token, 1);
7841-
if (foo > 0) {
7842-
CONSCRYPT_LOG_VERBOSE("FOO: %d", foo);
7843-
}
7838+
(void)read(appData->fdsEmergency[0], &token, 1);
78447839
} while (errno == EINTR);
78457840
}
78467841
}
@@ -7871,12 +7866,7 @@ static void sslNotify(AppData* appData) {
78717866
char token = '*';
78727867
do {
78737868
errno = 0;
7874-
// TEMP - fixes build error
7875-
int foo = 0;
7876-
foo = write(appData->fdsEmergency[1], &token, 1);
7877-
if (foo > 0) {
7878-
CONSCRYPT_LOG_VERBOSE("FOO: %d", foo);
7879-
}
7869+
(void)write(appData->fdsEmergency[1], &token, 1);
78807870
} while (errno == EINTR);
78817871
errno = errnoBackup;
78827872
#endif

openjdk/build.gradle

Lines changed: 0 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -350,15 +350,6 @@ def testInterop = tasks.register("testInterop", Test) {
350350
}
351351
check.dependsOn testInterop
352352

353-
// Added to see results of new ECH tests when running tests from the command line
354-
tasks.withType(Test).configureEach {
355-
testLogging {
356-
exceptionFormat "full"
357-
events "started", "skipped", "passed", "failed"
358-
showStandardStreams true
359-
}
360-
}
361-
362353
jacocoTestReport {
363354
additionalSourceDirs.from files("$rootDir/openjdk/src/test/java", "$rootDir/common/src/main/java")
364355
executionData tasks.withType(Test)

0 commit comments

Comments
 (0)