Skip to content

Commit 83e91cc

Browse files
l46kokcopybara-github
authored andcommitted
Fix extension class visibility to address Kotlin compilation warnings
PiperOrigin-RevId: 862324115
1 parent ff8cd30 commit 83e91cc

File tree

6 files changed

+10
-6
lines changed

6 files changed

+10
-6
lines changed

extensions/src/main/java/dev/cel/extensions/CelComprehensionsExtensions.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -41,7 +41,7 @@
4141
import java.util.Optional;
4242

4343
/** Internal implementation of CEL two variable comprehensions extensions. */
44-
final class CelComprehensionsExtensions
44+
public final class CelComprehensionsExtensions
4545
implements CelCompilerLibrary, CelInternalRuntimeLibrary, CelExtensionLibrary.FeatureSet {
4646

4747
private static final String MAP_INSERT_FUNCTION = "cel.@mapInsert";

extensions/src/main/java/dev/cel/extensions/CelListsExtensions.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -51,9 +51,10 @@
5151
import java.util.Set;
5252

5353
/** Internal implementation of CEL lists extensions. */
54-
final class CelListsExtensions
54+
public final class CelListsExtensions
5555
implements CelCompilerLibrary, CelInternalRuntimeLibrary, CelExtensionLibrary.FeatureSet {
5656

57+
/** Supported functions for Lists extension library. */
5758
@SuppressWarnings({"unchecked"}) // Unchecked: Type-checker guarantees casting safety.
5859
public enum Function {
5960
// Note! Creating dependencies on the outer class may cause circular initialization issues.

extensions/src/main/java/dev/cel/extensions/CelMathExtensions.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -57,7 +57,7 @@
5757
*/
5858
@SuppressWarnings({"rawtypes", "unchecked"}) // Use of raw Comparables.
5959
@Immutable
60-
final class CelMathExtensions
60+
public final class CelMathExtensions
6161
implements CelCompilerLibrary, CelRuntimeLibrary, CelExtensionLibrary.FeatureSet {
6262

6363
private static final String MATH_NAMESPACE = "math";

extensions/src/main/java/dev/cel/extensions/CelProtoExtensions.java

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -31,7 +31,8 @@
3131

3232
/** Internal implementation of CEL proto extensions. */
3333
@Immutable
34-
final class CelProtoExtensions implements CelCompilerLibrary, CelExtensionLibrary.FeatureSet {
34+
public final class CelProtoExtensions
35+
implements CelCompilerLibrary, CelExtensionLibrary.FeatureSet {
3536

3637
private static final String PROTO_NAMESPACE = "proto";
3738
private static final CelExpr ERROR = CelExpr.newBuilder().setConstant(Constants.ERROR).build();
@@ -148,4 +149,6 @@ private static boolean isTargetInNamespace(CelExpr target) {
148149
return target.exprKind().getKind().equals(CelExpr.ExprKind.Kind.IDENT)
149150
&& target.ident().name().equals(PROTO_NAMESPACE);
150151
}
152+
153+
CelProtoExtensions() {}
151154
}

extensions/src/main/java/dev/cel/extensions/CelRegexExtensions.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -37,7 +37,7 @@
3737

3838
/** Internal implementation of CEL regex extensions. */
3939
@Immutable
40-
final class CelRegexExtensions
40+
public final class CelRegexExtensions
4141
implements CelCompilerLibrary, CelRuntimeLibrary, CelExtensionLibrary.FeatureSet {
4242

4343
private static final String REGEX_REPLACE_FUNCTION = "regex.replace";

extensions/src/main/java/dev/cel/extensions/CelSetsExtensions.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -41,7 +41,7 @@
4141
* rewrite the AST into a map to achieve a O(1) lookup.
4242
*/
4343
@Immutable
44-
final class CelSetsExtensions
44+
public final class CelSetsExtensions
4545
implements CelCompilerLibrary, CelRuntimeLibrary, CelExtensionLibrary.FeatureSet {
4646

4747
private static final String SET_CONTAINS_OVERLOAD_DOC =

0 commit comments

Comments
 (0)