search

G-Node / nix

Neuroscience information exchange format
https://readthedocs.org/projects/nixio/
Other
66 stars 36 forks source link

Backend code separation #584

Closed gicmo closed 8 years ago

gicmo commented 8 years ago

Currently the backend code is intertwined with the fronted code, everything living in src, i.e. the hdf5 backend is in src/hdf5 and the headers are in include/nix/hdf5 (and are also installed). With pull request #583 the hdf5 backend headers are not installed anymore (along with the separation of hdf5 backend code from the h5 C++ wrapper). The file backend currently lives in src/file It would be cleaner if all the backends all the backends would live in backend/[name] (instead of src/[name]).

stoewer commented 8 years ago

Sounds good :+1:

jgrewe commented 8 years ago

closed via pr #583

jgrewe commented 8 years ago

ups, not yet entirely done...

jgrewe commented 8 years ago

I wonder whether we should keep the HDF5, FS suffixes in our backend naming scheme. They may help to distinguish between back and front ends but it looks like done twice...

jgrewe commented 8 years ago

apparently finalized with pr #604 #603