Skip to content

Commit d0a53db

Browse files
committed
libnativehelper: retire AFileDescriptor_{get,set}FD
Superceded by AFileDescriptor_{get,set}Fd. Bug: 185256332 Test: m Change-Id: I7ccc81fda92aa48afabc14be09e042113e2e335b
1 parent 9606a1e commit d0a53db

4 files changed

+0
-35
lines changed

file_descriptor_jni.c

-11
Original file line numberDiff line numberDiff line change
@@ -41,18 +41,7 @@ JNIEXPORT int AFileDescriptor_getFd(JNIEnv* env, jobject fileDescriptor) {
4141
return (*env)->GetIntField(env, fileDescriptor, JniConstants_FileDescriptor_descriptor(env));
4242
}
4343

44-
JNIEXPORT int AFileDescriptor_getFD(JNIEnv* env, jobject fileDescriptor) {
45-
// Method deprecated pending removal (b/185256332).
46-
return AFileDescriptor_getFd(env, fileDescriptor);
47-
}
48-
4944
JNIEXPORT void AFileDescriptor_setFd(JNIEnv* env, jobject fileDescriptor, int fd) {
5045
EnsureArgumentIsFileDescriptor(env, fileDescriptor);
5146
(*env)->SetIntField(env, fileDescriptor, JniConstants_FileDescriptor_descriptor(env), fd);
5247
}
53-
54-
55-
JNIEXPORT void AFileDescriptor_setFD(JNIEnv* env, jobject fileDescriptor, int fd) {
56-
// Method deprecated pending removal (b/185256332).
57-
AFileDescriptor_setFd(env, fileDescriptor, fd);
58-
}

include/android/file_descriptor_jni.h

-6
Original file line numberDiff line numberDiff line change
@@ -62,9 +62,6 @@ jobject AFileDescriptor_create(JNIEnv* env) __INTRODUCED_IN(31);
6262
*/
6363
int AFileDescriptor_getFd(JNIEnv* env, jobject fileDescriptor) __INTRODUCED_IN(31);
6464

65-
// Deprecated equivalent of AFileDescriptor_getFd (b/185256332).
66-
int AFileDescriptor_getFD(JNIEnv* env, jobject fileDescriptor) __INTRODUCED_IN(31);
67-
6865
/**
6966
* Sets the Unix file descriptor represented by the given java.io.FileDescriptor.
7067
*
@@ -82,9 +79,6 @@ int AFileDescriptor_getFD(JNIEnv* env, jobject fileDescriptor) __INTRODUCED_IN(3
8279
*/
8380
void AFileDescriptor_setFd(JNIEnv* env, jobject fileDescriptor, int fd) __INTRODUCED_IN(31);
8481

85-
// Deprecated equivalent of AFileDescriptor_setFd (b/185256332).
86-
void AFileDescriptor_setFD(JNIEnv* env, jobject fileDescriptor, int fd) __INTRODUCED_IN(31);
87-
8882
__END_DECLS
8983

9084
/** @} */

libnativehelper.map.txt

-4
Original file line numberDiff line numberDiff line change
@@ -4,10 +4,6 @@ LIBNATIVEHELPER_S { # introduced=S
44
AFileDescriptor_create;
55
AFileDescriptor_getFd;
66
AFileDescriptor_setFd;
7-
# AFileDescriptor_getFD is deprecated pending removal (b/185256332).
8-
AFileDescriptor_getFD;
9-
# AFileDescriptor_setFD is deprecated pending removal (b/185256332).
10-
AFileDescriptor_setFD;
117

128
# JNI Invocation methods available to platform and apps.
139
JNI_CreateJavaVM;

libnativehelper_lazy.c

-14
Original file line numberDiff line numberDiff line change
@@ -41,8 +41,6 @@ enum MethodIndex {
4141
k_AFileDescriptor_create,
4242
k_AFileDescriptor_getFd,
4343
k_AFileDescriptor_setFd,
44-
k_AFileDescriptor_getFD, // Deprecated pending removal (b/185256332).
45-
k_AFileDescriptor_setFD, // Deprecated pending removal (b/185256332).
4644

4745
// JNI_Invocation API declared in jni.h.
4846
k_JNI_CreateJavaVM,
@@ -126,8 +124,6 @@ static void InitializeOnce() {
126124
BIND_SYMBOL(AFileDescriptor_create);
127125
BIND_SYMBOL(AFileDescriptor_getFd);
128126
BIND_SYMBOL(AFileDescriptor_setFd);
129-
BIND_SYMBOL(AFileDescriptor_getFD); // Deprecated pending removal (b/185256332).
130-
BIND_SYMBOL(AFileDescriptor_setFD); // Deprecated pending removal (b/185256332).
131127

132128
// JNI_Invocation API declared in jni.h.
133129
BIND_SYMBOL(JNI_CreateJavaVM);
@@ -208,16 +204,6 @@ void AFileDescriptor_setFd(JNIEnv* env, jobject fileDescriptor, int fd) {
208204
INVOKE_VOID_METHOD(AFileDescriptor_setFd, M, env, fileDescriptor, fd);
209205
}
210206

211-
int AFileDescriptor_getFD(JNIEnv* env, jobject fileDescriptor) {
212-
typedef int (*M)(JNIEnv*, jobject);
213-
INVOKE_METHOD(AFileDescriptor_getFD, M, env, fileDescriptor);
214-
}
215-
216-
void AFileDescriptor_setFD(JNIEnv* env, jobject fileDescriptor, int fd) {
217-
typedef void (*M)(JNIEnv*, jobject, int);
218-
INVOKE_VOID_METHOD(AFileDescriptor_setFD, M, env, fileDescriptor, fd);
219-
}
220-
221207
//
222208
// Forwarding for the JNI_Invocation API declarded in jni.h.
223209
//

0 commit comments

Comments
 (0)