samskalicky commented on a change in pull request #20523:
URL: https://github.com/apache/incubator-mxnet/pull/20523#discussion_r689995091
##########
File path: include/mxnet/c_api.h
##########
@@ -233,10 +233,11 @@ MXNET_DLL const char *MXGetLastError();
/*!
* \brief Load library dynamically
* \param path to the library .so file
- * \param 0 for quiet, 1 for verbose
+ * \param verbose 0 for quiet, 1 for verbose
+ * \param lib handle to opened library
* \return 0 when success, -1 when failure happens.
*/
-MXNET_DLL int MXLoadLib(const char *path, unsigned verbose);
+MXNET_DLL int MXLoadLib(const char *path, unsigned verbose, void** lib);
Review comment:
Thanks for the review @szha! No this change isnt required, but without
it we cannot return a handle to the opened library. Its not required per-se
since opening the same library using ctypes or some other API will return the
same handle anyway (with linux doing the mapping). So we can remove this change
if that is important, either way is fine to me.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
