From 52ca07e2102458540513fd389b72776cbe417f93 Mon Sep 17 00:00:00 2001 From: Haroon Qureshi Date: Mon, 15 Dec 2025 06:03:03 -0800 Subject: [PATCH] Decoder `decode` function should take a non-const pointer to mjResource. This is because mjcReadResource requires a non-const pointer. PiperOrigin-RevId: 844737328 Change-Id: I88d217aa85a66023f6b222437403c136448cabfe --- include/mujoco/mjplugin.h | 2 +- test/plugin/decoder/decoder_test.cc | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/include/mujoco/mjplugin.h b/include/mujoco/mjplugin.h index a9e5127c..610297bc 100644 --- a/include/mujoco/mjplugin.h +++ b/include/mujoco/mjplugin.h @@ -69,7 +69,7 @@ typedef struct mjpResourceProvider mjpResourceProvider; // function pointer types // return an mjSpec representing the decoded resource. -typedef mjSpec* (*mjfDecode)(const mjResource* resource); +typedef mjSpec* (*mjfDecode)(mjResource* resource); // return true if the given resource can be decoded. typedef int (*mjfCanDecode)(const mjResource* resource); diff --git a/test/plugin/decoder/decoder_test.cc b/test/plugin/decoder/decoder_test.cc index 581b0db2..58113b14 100644 --- a/test/plugin/decoder/decoder_test.cc +++ b/test/plugin/decoder/decoder_test.cc @@ -39,7 +39,7 @@ static mjSpec* MakeSimpleSpec() { } // Always returns a simple mjSpec, ignoring the resource. -mjSpec* FakeDecode(const mjResource* resource) { return MakeSimpleSpec(); } +mjSpec* FakeDecode(mjResource* resource) { return MakeSimpleSpec(); } // Can decode any resource that has a .fakeformat extension. int FakeCanDecode(const mjResource* resource) {