U++ framework: U++ Core » Request: completion 64 bit support in Draw and Stream

Home » U++ Library support » U++ Core » Request: completion 64 bit support in Draw and Stream

Show: Today's Messages :: Polls :: Message Navigator
E-mail to friend

Request: completion 64 bit support in Draw and Stream [message #39004]

Wed, 06 February 2013 02:36

nixnixnix
Messages: 415
Registered: February 2007
Location: Kelowna, British Columbia

Senior Member

Hi,

I recently had a user complain that they couldn't save a TIF that they had opened in my software. The TIF is only 120MB but once as an Image it is approx 3000x3000 pixels and therefore around 3.6GB.

This means that the following function

void Image::Serialize(Stream& s)
{
	int version = 0;
	s / version;
	Size sz = GetSize();
	Point p = GetHotSpot();
	Size dots = GetDots();
	s % sz % p % dots;
	int len = sz.cx * sz.cy;
	if(s.IsLoading())
		if(len) {
			ImageBuffer b(sz);
			if(!s.GetAll(~b, len * sizeof(RGBA)))
				s.SetError();
			b.SetDots(dots);
			b.SetHotSpot(p);
			*this = b;
		}
		else
			Clear();
	else
		s.Put(~*this, len * sizeof(RGBA));
}

needs len to be declared as int64 leading to

int64 len = int64(sz.cx) * int64(sz.cy);

or something like that. Unfortunately, Stream::Put takes an int rather than an int64 and so the size is reported as less than zero and fails the ASSERT(size>=0) on Stream.h line 89.

I realise that I could re-write Image::Serialize to save Image in chunks but then you guys are so much better than me at this Smile

and it needs to be a permanent solution as we are all moving eventually to 64 bit and large memory items. Some of the code in Stream looks like it is intended for 64 bit.

I hope my detailed diagnostic helps.

Stream::GetAll will need upgrading as well to use int64. If no-one has time for this I can try to suggest a patch but I figure my code is unlikely to be the long-term solution.

Cheers,

Nick

Report message to a moderator

[Message index]

		Request: completion 64 bit support in Draw and Stream By: nixnixnix on Wed, 06 February 2013 02:36
		Re: Request: completion 64 bit support in Draw and Stream By: nixnixnix on Wed, 06 February 2013 15:13
		Re: Request: completion 64 bit support in Draw and Stream By: mirek on Thu, 07 February 2013 17:35
		Re: Request: completion 64 bit support in Draw and Stream By: nixnixnix on Thu, 07 February 2013 21:33
		Re: Request: completion 64 bit support in Draw and Stream By: koldo on Thu, 07 February 2013 23:03

Previous Topic:	Jsonize int64 surprise
Next Topic:	Gcc compile option proposal

Goto Forum:

-=] Back to Top [=-

[ Syndicate this forum (XML) ] [

] [

]

Current Time: Thu Sep 10 14:06:39 GMT+2 2026

Total time taken to generate the page: 0.00694 seconds