Le Thu, Jun 26, 2014 at 05:31:03PM +0300, Plamen Aleksandrov a écrit :
> I just look at the patch provided by Aleksandar. I believe unaligned access 
> is still possible after applying it.
> 
> The reason is that an auto array of uint8_t is still only guaranteed to be 
> 1-byte aligned. These discussions confirm this:
> http://stackoverflow.com/questions/4009463/alignment-of-char-arrays
> http://bytes.com/topic/c/answers/811633-alignment-stack-arrays
> 
> So I updated the patch to fix this issue.

Hi,

Aleksandar had an extended discussion with Upstream on GitHub.  On my side I
would prefer for that issue to be solved upstream than locally.  Can you see if
your updated patch would be useful in the context of the following pull request 
?

    https://github.com/samtools/htslib/pull/99

Have a nice day,

-- 
Charles Plessy
Tsurumi, Kanagawa, Japan


-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org

Reply via email to