Tíquete #18741

Improve const-correctness

: 2009-09-17 02:27 Última Atualização: 2009-12-12 00:18

Relator:
Dono:
(Nenhum)
Estado:
Aberto
Componente:
(Nenhum)
Marcos:
(Nenhum)
Prioridade:
1 - Lowest
Gravidade:
5 - Medium
Resolução:
Postponed
Arquivo:
Nenhum
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 History (3/9 Histories)

2009-09-17 02:27 Updated by: elfring
  • New Ticket "Improve const-correctness" created
2009-09-26 11:41 Updated by: (del#1144)
Comentário

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

2009-09-26 18:37 Updated by: elfring
Comentário

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 Updated by: (del#1144)
  • Tipo Update from Bugs to Feature Requests
Comentário

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 Updated by: elfring
Comentário

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 Updated by: (del#1144)
  • Prioridade Update from 5 - Medium to 3
2009-10-06 22:28 Updated by: (del#1144)
  • Resolução Update from Nenhum to Postponed
2009-12-10 20:44 Updated by: (del#1144)
  • Componente Update from Tera Term to (Nenhum)
  • Prioridade Update from 3 to 1 - Lowest
Comentário

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 Updated by: elfring
Comentário

How can the concerns about potential side effects be clarified?

Attachment File List

No attachments

Editar

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