Ticket #18742

Check type selection for sizes

Open Date: 2009-09-17 02:58 Last Update: 2009-12-12 00:18

Reporter:
Owner:
(None)
Status:
Open
Component:
(None)
MileStone:
(None)
Priority:
1 - Lowest
Severity:
5 - Medium
Resolution:
Postponed
File:
None
Vote
Score: 0
No votes
0.0% (0/0)
0.0% (0/0)

Details

How do you think about to replace any "int" and "long" for parameters like "pos", "len" or "destlen" by the data type "size_t"?

Examples:

Ticket History (3/9 Histories)

2009-09-17 02:58 Updated by: elfring
  • New Ticket "Check type selection for sizes" created
2009-09-26 11:40 Updated by: maya
Comment

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

2009-09-26 18:33 Updated by: elfring
Comment

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

I have pointed already to places in the source files which might need further clarification on better data type selection.

2009-09-29 12:09 Updated by: maya
Comment

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 change data type?

2009-09-29 18:50 Updated by: elfring
Comment

I do not know the source files good enough to definitely say in all places where negative values are really acceptable for integral data types. I have got the impression that there are still more update candidates in the source code where unsigned types will improve software correctness.

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

2009-10-06 22:28 Updated by: maya
  • Priority Update from 5 - Medium to 3
2009-10-06 22:28 Updated by: maya
  • Resolution Update from None to Postponed
2009-12-10 20:44 Updated by: maya
  • Component Update from Tera Term to (None)
  • Priority Update from 3 to 1 - Lowest
Comment

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
Comment

How can the concerns about potential side effects be clarified?

Attachment File List

No attachments

Edit

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