ch1bo / integer-openssl

Implementation of the 'GHC.Integer' type using the OpenSSL's arbitrary size integer library.
BSD 3-Clause "New" or "Revised" License
6 stars 1 forks source link

Implementation mo1 #2

Closed oswald2 closed 5 years ago

oswald2 commented 5 years ago

Hallo Sebastian,

Die Änderungen von Florian und mir sind in diesem Branch erstmal drin. Ich hatte dieses Wochenende Zeit, ein bissl weiterzumachen und die Double Funktionen sind jetzt fertig und laufen auch bei den Tests durch. War ein bissl eine schwerere Geburt.

Was da jetzt drin ist:

Von Florian:

Florian hat ein paar Sachen im wrapper.c ausgebessert, ich hab abhängig von der openssl Version die Definition der Struct eingefügt. Bei mir am Ubuntu 14.04 gabs sonst eine doppelte Definition, während am 18.04 die struct nicht definiert war. Evtl muss man da noch nachbessern, hab jetzt erstmal ab 1.1.0 gesetzt, dass die Struktur da ist.

Beim decodeDoubleInteger ist die 32 Bit Version direkt von integer-simple übernommen und daher wahrscheinlich recht ineffektiv. Ich hatte noch keine Gelegenheit mir das auf einem 32 Bit System anzuschauen ob die Tests durchlaufen.

Ich glaub, das wärs erstmal.

lg, Michael