Skip to content

Conversation

@stm2
Copy link

@stm2 stm2 commented Feb 29, 2016

No description provided.

@ennorehling
Copy link
Owner

Obwohl ich dir zustimme, dass das so schöner ist, ist const-correctness etwas, worum sich leider nicht jeder in der C-Welt schert. Die meisten scheinen sich zu wünschen, dass es das const keyword nicht gibt, der libxml2 Entwickler ist da ganz vorn mit dabei, aber auch CuTest hat solche Macken.

Es fragt sich, ob es wert ist, für so eine kleine Änderung am Code einen eigenen Fork zu verwalten. Wir kriegen das ganz sicher nicht in den Upstream, d.h. zukünftige Upgrades auf neue Versionen werden immer mit Mehrarbeit verbunden sein, wo wir eigene Patches einspielen müssen, damit der Eressea-Code wieder kompiliert. So etwas ist die Hölle, und es ist einfacher, im eigenen Code darum herum zu arbeiten mit einem Typecast an der richtigen Stelle.

@stm2
Copy link
Author

stm2 commented Feb 29, 2016

Ehrlich? Selbst wenn ich das akzeptiere, gibt es doch zwei gute Gründe für die Änderung:

  • Sie stellt keine Einschränkung dar, da sie keinen Code inkompatibel macht. Ich kann jetzt lediglich auch const pointer übergeben.
  • Es gibt da schon const char*-Parameter

@ennorehling
Copy link
Owner

  1. Wenn CuTest mein eigenes Projekt wäre, würde ich das auch so ändenr. Ich mag const.
  2. const char* ist eine Ausnahme, um die man auch als Hasser von const nicht herum kommt (weil alle String-Konstanten diesen Typ haben).
  3. Der Code type-castet jetzt schon überall CuAssertPtrEquals Argumente nach (void *).

Wie sagt man eigentlich type-casting auf Deutsch? Typ-Verzauberung? Ich kann diese Sprache mit ihren Schlangen und Kellern und was-auch-immer mit Informatik nicht in Einklang bringen.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants