Merge pull request #9 from rflamand/fix/raffla/fix_web_binding_for_colormanager

fedorov · web-flow · commit 26a69abdef06 · 2026-02-03T14:29:52.000+01:00
Update ColorManager constructor with proper type
diff --git a/wasm/src/ColorManager.hpp b/wasm/src/ColorManager.hpp
@@ -22,7 +22,7 @@ class ColorManager {
   /// </summary>
   ColorManager(FrameInfo frameInfo,
                const val &iccProfile,
-               int outputType = DCM_ICC_OUTPUT_SRGB) {
+               DcmIccOutputType outputType = DCM_ICC_OUTPUT_SRGB) {
 
     this->frameInfo = frameInfo;
 
diff --git a/wasm/src/jslib.cpp b/wasm/src/jslib.cpp
@@ -30,7 +30,7 @@ EMSCRIPTEN_BINDINGS(FrameInfo) {
 
 EMSCRIPTEN_BINDINGS(ColorManager) {
   class_<ColorManager>("ColorManager")
-    .constructor<FrameInfo, const val, int>()
+    .constructor<FrameInfo, const val, DcmIccOutputType>()
     .function("getFrameInfo", &ColorManager::getFrameInfo)
     .function("transform", &ColorManager::transform)
   ;

Original file line number	Diff line number	Diff line change
`@@ -30,7 +30,7 @@ EMSCRIPTEN_BINDINGS(FrameInfo) {`
`30`	`30`
`31`	`31`	`EMSCRIPTEN_BINDINGS(ColorManager) {`
`32`	`32`	`class_<ColorManager>("ColorManager")`
`33`		`- .constructor<FrameInfo, const val, int>()`
	`33`	`+ .constructor<FrameInfo, const val, DcmIccOutputType>()`
`34`	`34`	`.function("getFrameInfo", &ColorManager::getFrameInfo)`
`35`	`35`	`.function("transform", &ColorManager::transform)`
`36`	`36`	`;`