Voyager source released under GPL

	Bottom Previous Topic Next Topic
Register To Post

« 1 (2)

Chris

Re: Voyager source released under GPL

Posted on: 2017/10/23 16:33 #21

Amigans Defender

@salass00

Quote:

Well I can see two differences:

1) The SEC_TO_AMIGA_EPOCH value you are using in your code is lower than in mine.

Only by ten days.

Quote:

2) You are not converting the time to local time. AmigaDOS expects date stamps to always be in local time unlike other systems like Windows and Linux where these are stored as GMT.

Again, this would only be a few hours. The results I'm geting aren't even in the right year.

Quote:

However if it is as xenic reported that the dates are different each time the same archive is extracted then there clearly is a serious problem somewhere else in the code.

I've fixed that particular bug. The structure had changed and I hadn't noticed as the variable names were the same.

Chris

Re: Voyager source released under GPL

Posted on: 2017/10/23 18:39 #22

Amigans Defender

A-ha! Turns out my original swapping of high and low LONGs was actually correct, it doesn't need any other bytes swapped.

It's ten days out so I'll use salass00's epoch value and then evaluate whether it needs timezone correction.

xenic

Re: Voyager source released under GPL

Posted on: 2017/10/23 21:43 #23

Just can't stay away

@Chris
Thanks to you & salass00 for taking the time to uncover the problem and hopefully fix it.

Amiga X1000 with 2GB memory & OS 4.1FE + Radeon HD 5450

Chris

Re: Voyager source released under GPL

Posted on: 2017/10/24 0:10 #24

Amigans Defender

@xenic

Well, it's fixed but I'm going to try to merge in the latest LZMA changes before I release it.*

* which may well un-fix it again

Chris

Re: Voyager source released under GPL

Posted on: 2017/10/28 18:34 #25

Amigans Defender

I've decided against the LZMA changes beyond a minor update I'd apparently already merged, as they are more significant than expected and I can't be bothered with the fallout right now.

The version with fixed timestamps will be available from AmiUpdate tomorrow. (It will offer it today, but it's the old one still - didn't want to forget to change it

)

Register To Post	« 1 (2)
	Top Previous Topic Next Topic

Currently Active Users Viewing This Thread: 3 ( 0 members and 3 Anonymous Users )