'SenderContext' defines a non-default destructor but does not define a copy constructor, a copy assignment operator, a move constructor or a move assignment operator

[tiziano88]

From clang-tidy:

/home/tzn/.cache/bazel/_bazel_tzn/dab993785bc8bd52220ddfe5340ed840/sandbox/linux-sandbox/5480/execroot/oak/cc/crypto/hpke/sender_context.h:33:7: error: class 'SenderContext' defines a non-default destructor but does not define a copy constructor, a copy assignment operator, a move constructor or a move assignment operator [cppcoreguidelines-special-member-functions,-warnings-as-errors]

@ipetr0v could you take a quick look and fix that?

[ipetr0v]

Is this error produced by an internal clang-tidy or by the one added in #4634 ?

I'm trying to run bazel build --config=clang-tidy //cc/... locally and I don't get this error

[tiziano88]

To reproduce you can add cppcoreguidelines-special-member-functions to the list of checks in https://github.com/project-oak/oak/blob/main/.clang-tidy and then run just clang-tidy

[ipetr0v]

clang-tidy wants me to use reinterpret_cast here:

oak/cc/crypto/hpke/jni/context_jni.cc

Lines 36 to 37 in 1ef0a9d

	oak::crypto::SenderContext* sender_context =
	(oak::crypto::SenderContext*)(env->GetLongField(obj, fid));

But then says

error: integer to pointer cast pessimizes optimization opportunities [performance-no-int-to-ptr,-warnings-as-errors]

And we have to cast jlong into a pointer because of JNI

[ipetr0v]

Currently had to add -performance-no-int-to-ptr to https://github.com/project-oak/oak/blob/main/.clang-tidy

[ipetr0v]

Also about the error: consider replacing 'long' with 'int64' [google-runtime-int,-warnings-as-errors]. I'm not sure whether it's fine to do on Android JNI (needs additional double-checking).

oak/cc/crypto/hpke/jni/hpke_jni.cc

Lines 53 to 55 in 1a95821

	jobject sender_context =
	env->NewObject(sender_context_class, sender_context_constructor,
	serialized_encapsulated_public_key, (long)native_sender_context->release());

cc @conradgrobler @k-naliuka

[tiziano88]

FYI I don't think this is urgent or is blocking anything, just something that should be cleaned up at some point.