Skip to content

Conversation

@angeltears
Copy link
Contributor

在lz77中,可以看到仍在使用32位操作系统的long表示32字节,导致部分同学clone测试失败,另一方面在窗口长度上也有部分错误,compare_win会返回匹配的下标加一,所以导致匹配后,offset实际范围为0-4096这个时候winoff_bit取13字节更为合适,在解压函数也应该做相应的修改

@yourtion yourtion merged commit e39ae28 into yourtion:master Dec 6, 2018
@yourtion yourtion mentioned this pull request Dec 6, 2018
@yourtion
Copy link
Owner

yourtion commented Dec 6, 2018

@angeltears 我在Mac上用 cc 编译,结果还是不对哦

Compressing with LZ77
Compressing...Done
Uncompressing...Done
-> osize=5983, csize=1584, rsize=5983
Data was not properly restored
-> original[0]="/"
-> restored[0]=0x00

@yourtion yourtion mentioned this pull request Dec 7, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants