freenode/#clim - IRC Chatlog
Search
16:40:31
slyrus
yeah, so it uses the foreign memory allocation routines but I'm not sure it actually calls any foreign code. I'm curious to know if that really makes pngload any faster or not.
16:41:30
slyrus
it looks like there's an option in pngload to turn it off. It would be great if somebody were to benchmark things and show that static-vectors was worth the headache.
16:49:40
slyrus
it does seem that it would be pretty straightforward to 1) run pngload with and without static vectors and benchmark reading some PNG images and 2) rip the static-vectors stuff out of opticl. So far it has just been a source of incompatibility for me, but I do think pngload is better than png-read.
17:12:07
slyrus
at a minimum it would be nice if pngload would only attempt to use static-vectors on supported platforms.
17:18:41
jackdaniel
clim-backend package exports symbols, which are meant to be implemented by backends (which are not part of clim nor clime package)
17:21:23
slyrus
that seems silly. I'd prefer to see the symbols exported from some package, but OK, whatever.
17:23:07
jackdaniel
climi is our implementation package, that's where we implement things. clim exports CLIM API, clime exports our official extensions, climb exports our requirements for backends (if they go beyond what is defined in clim package)
17:23:56
jackdaniel
and while I understand that backends have their own packages to avoid name clashes between themself I don't see much purpose in adding new packages for anything else
19:00:17
jackdaniel
for me yes, I've added 'approve' to PR (I'll let merging to Nisar when he is satisfied with the changes too)