chenlin/Mujoco_WASM
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Replace printf with mju_error in WASM bindings.

Browse Source 
PiperOrigin-RevId: 860099912
Change-Id: Ia6a24da9301099aa976bb7a38beb9da15074a43d
This commit is contained in:
Google DeepMind
2026-01-23 07:25:39 -08:00
committed by Copybara-Service
parent 2582a83ac3
commit c55c9a7bf1
2 changed files with 6 additions and 12 deletions
Download Patch File Download Diff File Expand all files Collapse all files
wasm/codegen/generated/bindings.cc
+3 -6
View File
@@ -8005,8 +8005,7 @@ std::unique_ptr<MjModel> mj_loadXML_wrapper(std::string filename) {
char error[1000]; char error[1000];
mjModel *model = mj_loadXML(filename.c_str(), nullptr, error, sizeof(error)); mjModel *model = mj_loadXML(filename.c_str(), nullptr, error, sizeof(error));
if (!model) { if (!model) {
printf("Loading error: %s\n", error); mju_error("Loading error: %s\n", error);
return nullptr;
} }
return std::unique_ptr<MjModel>(new MjModel(model)); return std::unique_ptr<MjModel>(new MjModel(model));
} }
@@ -8020,8 +8019,7 @@ void mj_saveModel_wrapper(const MjModel& m, const StringOrNull& filename, const
std::unique_ptr<MjModel> mj_loadModel_wrapper(std::string filename, const MjVFS& vfs) { std::unique_ptr<MjModel> mj_loadModel_wrapper(std::string filename, const MjVFS& vfs) {
mjModel *model = mj_loadModel(filename.c_str(), vfs.get()); mjModel *model = mj_loadModel(filename.c_str(), vfs.get());
if (!model) { if (!model) {
printf("mj_loadModel: failed to load from mjb"); mju_error("Failed to load from mjb");
return nullptr;
} }
return std::unique_ptr<MjModel>(new MjModel(model)); return std::unique_ptr<MjModel>(new MjModel(model));
} }
@@ -8030,8 +8028,7 @@ std::unique_ptr<MjSpec> parseXMLString_wrapper(const std::string &xml) {
char error[1000]; char error[1000];
mjSpec *ptr = mj_parseXMLString(xml.c_str(), nullptr, error, sizeof(error)); mjSpec *ptr = mj_parseXMLString(xml.c_str(), nullptr, error, sizeof(error));
if (!ptr) { if (!ptr) {
printf("Could not create Spec from XML string: %s\n", error); mju_error("Could not create Spec from XML string: %s\n", error);
return nullptr;
} }
return std::unique_ptr<MjSpec>(new MjSpec(ptr)); return std::unique_ptr<MjSpec>(new MjSpec(ptr));
} }
wasm/codegen/templates/bindings.cc
+3 -6
View File
@@ -523,8 +523,7 @@ std::unique_ptr<MjModel> mj_loadXML_wrapper(std::string filename) {
char error[1000]; char error[1000];
mjModel *model = mj_loadXML(filename.c_str(), nullptr, error, sizeof(error)); mjModel *model = mj_loadXML(filename.c_str(), nullptr, error, sizeof(error));
if (!model) { if (!model) {
printf("Loading error: %s\n", error); mju_error("Loading error: %s\n", error);
return nullptr;
} }
return std::unique_ptr<MjModel>(new MjModel(model)); return std::unique_ptr<MjModel>(new MjModel(model));
} }
@@ -538,8 +537,7 @@ void mj_saveModel_wrapper(const MjModel& m, const StringOrNull& filename, const
std::unique_ptr<MjModel> mj_loadModel_wrapper(std::string filename, const MjVFS& vfs) { std::unique_ptr<MjModel> mj_loadModel_wrapper(std::string filename, const MjVFS& vfs) {
mjModel *model = mj_loadModel(filename.c_str(), vfs.get()); mjModel *model = mj_loadModel(filename.c_str(), vfs.get());
if (!model) { if (!model) {
printf("mj_loadModel: failed to load from mjb"); mju_error("Failed to load from mjb");
return nullptr;
} }
return std::unique_ptr<MjModel>(new MjModel(model)); return std::unique_ptr<MjModel>(new MjModel(model));
} }
@@ -548,8 +546,7 @@ std::unique_ptr<MjSpec> parseXMLString_wrapper(const std::string &xml) {
char error[1000]; char error[1000];
mjSpec *ptr = mj_parseXMLString(xml.c_str(), nullptr, error, sizeof(error)); mjSpec *ptr = mj_parseXMLString(xml.c_str(), nullptr, error, sizeof(error));
if (!ptr) { if (!ptr) {
printf("Could not create Spec from XML string: %s\n", error); mju_error("Could not create Spec from XML string: %s\n", error);
return nullptr;
} }
return std::unique_ptr<MjSpec>(new MjSpec(ptr)); return std::unique_ptr<MjSpec>(new MjSpec(ptr));
} }