#467 Brace init formatting bad

[Beep6581]

Hello

Is there a way to prevent astyle from fudging brace inits?

@@ -350,20 +354,20 @@ ToneCurveParams::ToneCurveParams() :
     method("Blend"),
     expcomp(0),
     curve{

-        DCT_Linear
-    },
-    curve2{
-        DCT_Linear
-    },
-    curveMode(ToneCurveParams::TcMode::STD),
-    curveMode2(ToneCurveParams::TcMode::STD),
-    brightness(0),
-    black(0),
-    contrast(0),
-    saturation(0),
-    shcompr(50),
-    hlcompr(0),
-    hlcomprthresh(33)
+    DCT_Linear
+},
+curve2{
+    DCT_Linear
+},
+curveMode(ToneCurveParams::TcMode::STD),
+          curveMode2(ToneCurveParams::TcMode::STD),
+          brightness(0),
+          black(0),
+          contrast(0),
+          saturation(0),
+          shcompr(50),
+          hlcompr(0),
+          hlcomprthresh(33)
 {
 }

style=1tbs
indent=spaces=4
indent-switches
break-blocks
pad-oper
convert-tabs
pad-header
unpad-paren

https://github.com/Beep6581/RawTherapee/issues/3715#issuecomment-346268169

[Beep6581]

Bump

[Jim Pattee]

You can disable formatting with
http://astyle.sourceforge.net/astyle.html#_Disable_Formatting

It may or may not be a viable solution depending on how many need to be disabled.

If you request a fix I need to see the entire statement.
This looks like an unusual format.

[Beep6581]

Jim apologies for the late reply, here you go:
https://github.com/Beep6581/RawTherapee/blob/dev/rtengine/procparams.cc#L307

P.S. Do you use GitHub? If you do, I would prefer to report there.

[Jim Pattee]

This will probably be fixed in the next release.
There are other similar problems with initializers.

Currently, there is no account on GitHub.

[André Simon]

fixed in recent release (3.6.4)

[w-peuker]

@saalen Isn't 3.6.5 the last release ?

[André Simon]

yes, but this bug was fixed before ;)

[w-peuker]

Thanks for the clarification.