lzxpress test

Matthieu Patou mat at samba.org
Sat Nov 27 07:03:19 MST 2010


On 27/11/2010 13:18, Matthias Dieter Wallnöfer wrote:
> Thanks for fixing this, ekacnet!
>
You're welcome I didn't even noticed that there was a discussion on the 
list about this.

Brad: does the fix works for you or it sill didn't produce the same 
output as the MS-code ?

Matthieu
> Matthias
>
> Matthias Dieter Wallnöfer wrote:
>> Tridge,
>>
>> (Jelmer was the wrong author - I first had only a look at the 
>> copyright and not at "git blame")
>>
>> could you please look into this and make the test also big-endian safe?
>>
>> Cheers,
>> Matthias
>>
>> Matthias Dieter Wallnöfer wrote:
>>> Jelmer,
>>>
>>> it would be nice if you could have a look at the lzxpress test 
>>> (../../lib/compression/testsuite.c). It seems not to be big-endian 
>>> compliant - host tridge: 
>>> http://build.samba.org/?function=View+Build;host=tridge;tree=samba_4_0_test;compiler=gcc. 
>>>
>>>
>>> Most probably the "fixed_out" array has to be reordered for 
>>> big-endian platforms in order to pass the test.
>>>
>>> Cheers,
>>> Matthias
>>>
>>
>>
>


-- 
Matthieu Patou
Samba Team        http://samba.org
Private repo      http://git.samba.org/?p=mat/samba.git;a=summary




More information about the samba-technical mailing list