[Bug 2035671] Review Request: mrcpp - A general purpose numerical mathematics library based on multiresolution analysis

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



https://bugzilla.redhat.com/show_bug.cgi?id=2035671

stig.r.jensen@xxxxxxxxx <stig.r.jensen@xxxxxxxxx> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |stig.r.jensen@xxxxxxxxx



--- Comment #17 from stig.r.jensen@xxxxxxxxx <stig.r.jensen@xxxxxxxxx> ---
Hi,

Dan, I guess you can say it's unsupported because we haven't cared up to this
point, since we've only ever been concerned with little-endian so far.

The problem here is probably the likes of the CrossCorrelation.cpp binary
readings that Ben is pointing out, which are reading static data from files
that come shipped with the package. This is a small amount of data and happens
once at first encounter in the code, so it is not really critical. It can quite
easily be written in an endian-safe manner, but it has sort of stuck around
since the beginning of time...

The other part with the MWNode is data serialization which _is_ performance
critical, so I would be more hesitant to change it. This part is not meant (and
never expected) to be platform portable, though.


-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
https://bugzilla.redhat.com/show_bug.cgi?id=2035671
_______________________________________________
package-review mailing list -- package-review@xxxxxxxxxxxxxxxxxxxxxxx
To unsubscribe send an email to package-review-leave@xxxxxxxxxxxxxxxxxxxxxxx
Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: https://lists.fedoraproject.org/archives/list/package-review@xxxxxxxxxxxxxxxxxxxxxxx
Do not reply to spam on the list, report it: https://pagure.io/fedora-infrastructure




[Index of Archives]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite Conditions]     [KDE Users]

  Powered by Linux