U++ forum: Welcome to the forum

Search on this site

Search in forums

Home » Developing U++ » U++ Developers corner » Request for code audit

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

Request for code audit [message #2408]

Tue, 11 April 2006 23:32

mirek
Messages: 14266
Registered: November 2005

Ultimate Member

Well, it happened to me again...

I have noticed that current BlockStream implementation does not play well with BMPRaster (new virtual representation of Image source) seek/read patterns.

So I have spend two days reimplementing it - this is the most tricky code to implement, and unfortunately complexity cannot be reduced there in any way without harming the performance.

Well, I have run various tests to get it right, so I believe it should be OK by now, however if somebody keen and sharp would take time to review the code I would be happy.

So far it is made active just in _DEBUG mode, it is in "Core/BlockStream.cpp".

The motivation was to make it behave better when file is read "backwards" (by seeking and Getting chunks of data).

Mirek

Report message to a moderator

Re: Request for code audit [message #2411 is a reply to message #2408]

Wed, 12 April 2006 09:10

mr_ped
Messages: 826
Registered: November 2005
Location: Czech Republic - Praha

Experienced Contributor

"Core/BlockStream.cpp" .. which version should I look for? 604-dev1 is probably old I suppose?

(I'm not on that subversioning system of yours and I'm not going to join that, sorry, I will wait till you switch to SVN Smile