--nextPart3411836.U50GroeXm0
Content-Type: text/plain;
  charset="iso-8859-1"
Content-Transfer-Encoding: quoted-printable
Content-Disposition: inline

Hi,

> (If kcfg is not relevant to k-c-d, please drop them with my apologies.)

(the opposite, actually)

> kcalc fails to build due to the additions of _helper methods, since the
> code relies on variable declarations from previous settings.
> Unfortunately, things are not as simple as just adding copies of the
> declarations to each setting, since the code also appears in the ctor.

Yes, I know about it.

> Is there a way to fix this without making many copies of the same
> variable with different names?

No(t yet).
Given that adding accessors for the default value, as done originally in a=
=20
recent patch by Albert, can have two down-sides
a) not being 100% sure <code> snippets are totally independent one from eac=
h=20
other
b) avoid people getting "grown" generated cpp/h files for a feature they do=
n't=20
use
I propose, with Albert's approval, the following: explicitely adding in the=
=20
=2Ekcfgc file the option DefaultValueGetters=3Dtrue for eanbling it (defaul=
t:=20
false). This way, nothing should change in the generated cpp/h files wrt KD=
E=20
4.3, and only users of that would enable it.
What do you (all) think about it?

> (Also, who do I yell at for changing the code generation w/o checking
> that it didn't break existing users? ;-) </friendly ribbing>)

Given that the feature was committed by Albert, who left for akademy the da=
y=20
after, I fixed it on my own making it working (modulo <code> snippets for=20
default values depending on other ones); so I guess I could (but I won't) t=
ake=20
your yell.
Although, given that akademy people had a crappy (at most) internet working=
=20
(so had difficulties in reaching him), and that after all trunk is open sin=
ce=20
few days (and the breakage is there since not even half a week), what about=
=20
being a bit more patient, instead to "have to yell" at first? </friendly=20
ribbing>

=2D-=20
Pino Toscano


--nextPart3411836.U50GroeXm0
Content-Type: application/pgp-signature; name=signature.asc 
Content-Description: This is a digitally signed message part.

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.9 (GNU/Linux)

iD8DBQBKU/YwTNH2piB/L3oRArk4AKC6OZbzjCbcOlRQh3rh9LKEUvI19gCgmFhr
2iRMb1KKQPrvmaursrijNaY=
=v8ed
-----END PGP SIGNATURE-----

--nextPart3411836.U50GroeXm0--