Skip to content

Commit

Permalink
Removed deprecated xjson.hpp
Browse files Browse the repository at this point in the history
  • Loading branch information
JohanMabille committed Sep 10, 2021
1 parent 74d1885 commit 17e6307
Show file tree
Hide file tree
Showing 6 changed files with 8 additions and 30 deletions.
1 change: 0 additions & 1 deletion CMakeLists.txt
Expand Up @@ -129,7 +129,6 @@ set(XEUS_HEADERS
${XEUS_INCLUDE_DIR}/xeus/xhistory_manager.hpp
${XEUS_INCLUDE_DIR}/xeus/xinput.hpp
${XEUS_INCLUDE_DIR}/xeus/xinterpreter.hpp
${XEUS_INCLUDE_DIR}/xeus/xjson.hpp
${XEUS_INCLUDE_DIR}/xeus/xkernel.hpp
${XEUS_INCLUDE_DIR}/xeus/xkernel_configuration.hpp
${XEUS_INCLUDE_DIR}/xeus/xlogger.hpp
Expand Down
2 changes: 1 addition & 1 deletion example/src/main.cpp
Expand Up @@ -26,7 +26,7 @@ int main(int argc, char* argv[])
interpreter_ptr interpreter = interpreter_ptr(new custom::custom_interpreter());

// Create kernel instance and start it
xeus::xkernel kernel(config, xeus::get_user_name(), std::move(interpreter), make_xserver);
xeus::xkernel kernel(config, xeus::get_user_name(), std::move(interpreter), xeus::make_xserver_zmq);
kernel.start();

return 0;
Expand Down
21 changes: 0 additions & 21 deletions include/xeus/xjson.hpp

This file was deleted.

6 changes: 3 additions & 3 deletions include/xeus/xserver_zmq.hpp
Expand Up @@ -78,9 +78,9 @@ namespace xeus
};

XEUS_API
std::unique_ptr<xserver> make_xserver(zmq::context_t& context,
const xconfiguration& config,
nl::json::error_handler_t eh = nl::json::error_handler_t::strict);
std::unique_ptr<xserver> make_xserver_zmq(zmq::context_t& context,
const xconfiguration& config,
nl::json::error_handler_t eh = nl::json::error_handler_t::strict);
}

#endif
6 changes: 3 additions & 3 deletions src/xserver_zmq.cpp
Expand Up @@ -209,9 +209,9 @@ namespace xeus
(void)m_heartbeat_controller.recv(response);
}

std::unique_ptr<xserver> make_xserver(zmq::context_t& context,
const xconfiguration& config,
nl::json::error_handler_t eh)
std::unique_ptr<xserver> make_xserver_zmq(zmq::context_t& context,
const xconfiguration& config,
nl::json::error_handler_t eh)
{
return std::make_unique<xserver_zmq>(context, config, eh);
}
Expand Down
2 changes: 1 addition & 1 deletion test/main.cpp
Expand Up @@ -21,7 +21,7 @@ int main(int argc, char* argv[])

using interpreter_ptr = std::unique_ptr<test_kernel::test_interpreter>;
interpreter_ptr interpreter = interpreter_ptr(new test_kernel::test_interpreter());
xeus::xkernel kernel(config, xeus::get_user_name(), std::move(interpreter), xeus::make_xserver);
xeus::xkernel kernel(config, xeus::get_user_name(), std::move(interpreter), xeus::make_xserver_zmq);
std::cout << "starting kernel" << std::endl;
kernel.start();

Expand Down

0 comments on commit 17e6307

Please sign in to comment.