80789 - change hashing algorithm for cfg file

Status: VERIFIED FIXED

(SeaMonkey :: General, defect)

Description

[Chip Clark]

Change the hashing algorithm for the cfg file from the current +/- 7 to 
intentially break from 4.x cfg hashed files.

Comment 1

[Rodney Velasco]

eClient issue, assigning QA to trix and adding dean pinckert to Cc.

Comment 2

[Tim Beauchamp]

Would not this best be accomplished via a versioning scheme?
That way you can selectively handle compatability issues
between versions and not just fail.  I the need arose, you
could even give a conversion hint.  Just one man's opinion. 

Comment 3

[Chip Clark]

Attachment: changes algorith from 7 to 13 to intentionally break existing cfg files

Comment 4

[Alec Flett]

I have to agree with the versioning thing. There are a finite number of prime
numbers between 0 and 255...
but sr=alecf anyway.

Comment 5

[Brian Nesse (gone)]

Comment on attachment 49811 [details] [diff] [review]
changes algorith from 7 to 13 to intentionally break existing cfg files

:) Let's just hope we don't have to change it everytime we ship a new release. r=bnesse.

Comment 6

[Chip Clark]

Mitesh checked the patch into the tree for chipc

Comment 7

[Rodney Velasco]

Chip, apparently this wasn't checked in by Mitesh.  This needs to get checked in
to test the new hash tool. Grega, can we get PDT approval for this, this is
vital for testing out the factory product.

Comment 8

[grega]

Approved.  PDT+ added

Comment 9

[Brian Nesse (gone)]

Checked into branch.

Comment 10

[Rodney Velasco]

this has been verified on the branch and trunk on all supported platforms, new
hash tool hashes CFG with new algorithm, browser now reads new algorithm.