Ticket #2264 (closed: wontfix)

Opened 10 years ago

Last modified 5 years ago

Put size_t's instead of int where appropriate.

Reported by: Janik Zikovsky Owned by: Janik Zikovsky
Priority: minor Milestone: Iteration 28
Component: Infrastructure Keywords:
Cc: Blocked By:
Blocking: Tester: Stuart Campbell

Description (last modified by Peter Peterson) (diff)

On linux a first round of offending files can be found using:

ack-grep for\\s\*.\\s\*u?int

Change History

comment:1 Changed 10 years ago by Janik Zikovsky

  • Status changed from new to accepted

comment:2 Changed 10 years ago by Janik Zikovsky

  • Status changed from accepted to verify
  • Resolution set to wontfix

Tried it, saw that WAY too many places need size_t and everything has to chagne. Gave up.

comment:3 Changed 10 years ago by Peter Peterson

  • Status changed from verify to reopened
  • Resolution wontfix deleted
  • Description modified (diff)

comment:4 Changed 10 years ago by Peter Peterson

  • Priority changed from major to minor
  • Component changed from Mantid to Infrastructure

comment:5 Changed 10 years ago by Peter Peterson

(In [8878]) Getting rid of int in for loops for kernel. Really silly in the tests. Refs #2264.

comment:6 Changed 10 years ago by Janik Zikovsky

  • Milestone changed from Iteration 27 to Iteration 28

comment:7 Changed 10 years ago by Janik Zikovsky

  • Status changed from reopened to accepted

comment:8 Changed 10 years ago by Janik Zikovsky

  • Status changed from accepted to assigned

comment:9 Changed 10 years ago by Janik Zikovsky

  • Status changed from assigned to accepted

comment:10 Changed 10 years ago by Janik Zikovsky

  • Status changed from accepted to verify
  • Resolution set to wontfix

Too big of a job.

comment:11 Changed 9 years ago by Stuart Campbell

  • Status changed from verify to verifying
  • Tester set to Stuart Campbell

comment:12 Changed 9 years ago by Stuart Campbell

  • Status changed from verifying to closed

I agree it's a big job... :-)

comment:13 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 3111

Note: See TracTickets for help on using tickets.