Skip to content

Commit 6449744

Browse files
committed
Remove extern C
Replace with 'namespace rcljava' Signed-off-by: Jacob Perron <[email protected]>
1 parent 2644593 commit 6449744

File tree

3 files changed

+10
-11
lines changed

3 files changed

+10
-11
lines changed

rcljava/src/main/cpp/convert.hpp

Lines changed: 5 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -21,9 +21,9 @@
2121

2222
#ifndef MAIN__CPP__CONVERT_HPP_
2323
#define MAIN__CPP__CONVERT_HPP_
24-
#ifdef __cplusplus
25-
extern "C" {
26-
#endif
24+
25+
namespace rcljava
26+
{
2727

2828
jobject
2929
convert_rmw_request_id_to_java(JNIEnv * env, rmw_request_id_t * request_id)
@@ -83,7 +83,6 @@ convert_rmw_request_id_from_java(JNIEnv * env, jobject jrequest_id)
8383
return request_id;
8484
}
8585

86-
#ifdef __cplusplus
87-
}
88-
#endif
86+
} // namespace rcljava
87+
8988
#endif // MAIN__CPP__CONVERT_HPP_

rcljava/src/main/cpp/org_ros2_rcljava_action_ActionServerImpl.cpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -213,7 +213,7 @@ Java_org_ros2_rcljava_action_ActionServerImpl_nativeCreateActionServer(
213213
jobject jtaken_msg = convert_to_java(taken_msg, jrequest_msg); \
214214
destroy_ros_message(taken_msg); \
215215
assert(jtaken_msg != nullptr); \
216-
jobject jheader = convert_rmw_request_id_to_java(env, &header); \
216+
jobject jheader = rcljava::convert_rmw_request_id_to_java(env, &header); \
217217
return jheader; \
218218
} \
219219
destroy_ros_message(taken_msg); \
@@ -231,7 +231,7 @@ Java_org_ros2_rcljava_action_ActionServerImpl_nativeCreateActionServer(
231231
convert_from_java_signature convert_from_java = \
232232
reinterpret_cast<convert_from_java_signature>(jresponse_from_java_converter_handle); \
233233
void * response_msg = convert_from_java(jresponse_msg, nullptr); \
234-
rmw_request_id_t * request_id = convert_rmw_request_id_from_java(env, jrequest_id); \
234+
rmw_request_id_t * request_id = rcljava::convert_rmw_request_id_from_java(env, jrequest_id); \
235235
rcl_ret_t ret = rcl_action_send_ ## Type ## _response( \
236236
action_server, request_id, response_msg); \
237237
destroy_ros_message_signature destroy_ros_message = \

rcljava/src/main/cpp/org_ros2_rcljava_executors_BaseExecutor.cpp

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -297,7 +297,7 @@ Java_org_ros2_rcljava_executors_BaseExecutor_nativeTakeRequest(
297297

298298
assert(jtaken_msg != nullptr);
299299

300-
jobject jheader = convert_rmw_request_id_to_java(env, &header);
300+
jobject jheader = rcljava::convert_rmw_request_id_to_java(env, &header);
301301
return jheader;
302302
}
303303

@@ -325,7 +325,7 @@ Java_org_ros2_rcljava_executors_BaseExecutor_nativeSendServiceResponse(
325325

326326
void * response_msg = convert_from_java(jresponse_msg, nullptr);
327327

328-
rmw_request_id_t * request_id = convert_rmw_request_id_from_java(env, jrequest_id);
328+
rmw_request_id_t * request_id = rcljava::convert_rmw_request_id_from_java(env, jrequest_id);
329329

330330
rcl_ret_t ret = rcl_send_response(service, request_id, response_msg);
331331

@@ -387,7 +387,7 @@ Java_org_ros2_rcljava_executors_BaseExecutor_nativeTakeResponse(
387387

388388
assert(jtaken_msg != nullptr);
389389

390-
jobject jheader = convert_rmw_request_id_to_java(env, &header);
390+
jobject jheader = rcljava::convert_rmw_request_id_to_java(env, &header);
391391
return jheader;
392392
}
393393

0 commit comments

Comments
 (0)