Does debian/patches/compilation_fixes really fix only "compilation on
Debian systems"?
I see several problems with the __fls function:
- You can't use "__" in your own identifiers; such names are reserved.
- The implementation uses x86 assembly.
- The unsigned long -> int typecast looks redundant to me.
Typo in src/optimize/lbfgs.f: THE THE -> THE.
Typo in src/optimize/lbfgsb.f: easible -> feasible.
--
Jakub Wilk
--
To UNSUBSCRIBE, email to debian-mentors-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org
Archive: http://lists.debian.org/20130108010649.ga2...@jwilk.net
