Opened 16 years ago

Closed 15 years ago

#894 closed Bugs (fixed)

mutli_array/types.hpp 64-bit types?

Reported by: nobody Owned by: nobody
Milestone: Component: None
Version: None Severity: Showstopper
Keywords: Cc:

Description

multi_array/types.hpp currently:

// needed typedefs
typedef std::size_t size_type;
typedef int index;

Should /index/ be a 64-bit quantity?
Should it be std::ptrdiff_t or 
std::size_t?

I caught this because of warnings
from MS Visual Studio 8 (2005).

Boost is really great!  

Thanks!

Sincerely,
  Mike Albert

  m_albert137@yahoo.com

Change History (2)

comment:1 by nobody, 16 years ago

Logged In: NO 

I don't think it can be size_t because index can be negative. However, I think there are some instances in which multi_array assigns 64-bit unsigned quantities to 64-bit(?) signed quantities. You should submit the specific warnings and code

comment:2 by Ronald Garcia, 15 years ago

Resolution: Nonefixed
Severity: Showstopper
Status: assignedclosed

index is already typedef'd to std::ptrdiff_t in CVS

Note: See TracTickets for help on using tickets.