Ticket #18741

Improve const-correctness

Eröffnet am: 2009-09-17 02:27 Letztes Update: 2009-12-12 00:18

Auswertung:
Verantwortlicher:
(Keine)
Status:
Offen
Komponente:
(Keine)
Meilenstein:
(Keine)
Priorität:
1 - Unterste
Schweregrad:
5 - Mittel
Lösung:
Postponed
Datei:
Keine
Vote
Score: 0
No votes
0.0% (0/0)
0.0% (0/0)

Details

Would you like to apply the advices from a well-known article to more places in your sources?

I suggest to add the key word "const" to the type specifiers for parameters like the following.

Ticket-Verlauf (3/9 Historien)

2009-09-17 02:27 Aktualisiert von: elfring
  • New Ticket "Improve const-correctness" created
2009-09-26 11:41 Aktualisiert von: maya
Kommentar

I think it is good. Do you have an idea for list up the points of replace?

2009-09-26 18:37 Aktualisiert von: elfring
Kommentar

I'm sorry - I do not really understand your reply on this issue.

I have pointed already to a few places in the source files which might need further clarification about the handling of more data structures in a read-only way.

2009-09-29 12:09 Aktualisiert von: maya
  • Typ Update from Fehler to Funktionsanfragen
Kommentar

I'm grateful for your suggestion, but I guess your meaning isn't correct only the "Example" points.

Do you have any idea for find *all of the place* that need to add const?

2009-09-29 18:45 Aktualisiert von: elfring
Kommentar

I do not know the source files good enough to definitely say in all places where data structures are used that do not need to be modified by a passed pointer. I have got the impression that there are still more update candidates in the source code.

I hope to get a clarification on the interface design from the core software developers.

2009-10-06 22:28 Aktualisiert von: maya
  • Priorität Update from 5 - Mittel to 3
2009-10-06 22:28 Aktualisiert von: maya
  • Lösung Update from Keine to Postponed
2009-12-10 20:44 Aktualisiert von: maya
  • Komponente Update from Tera Term to (Keine)
  • Priorität Update from 3 to 1 - Unterste
Kommentar

Disputed points and side-effect are too many, and team members doesn't have enough time to fix this issue.

2009-12-12 00:18 Aktualisiert von: elfring
Kommentar

How can the concerns about potential side effects be clarified?

Dateianhangliste

Keine Anhänge

Bearbeiten

You are not logged in. I you are not logged in, your comment will be treated as an anonymous post. » Anmelden