Menu

#9 Restructure src/marsyas a bit

open
nobody
None
5
2012-09-17
2010-06-10
No

Currently, the src/marsyas folder contains hundreds of header and cpp-files (due to all the different MarSystems available). It makes it sometimes hard to find the things you are looking for (in a IDE or in a file manager).

I think it would be better for the overview to separate the marsyas core stuff (realvec, MarSystem, MarSystemManager, MarControl, ...) from the actual MarSystems. This would also make it clearer to newcomers, so they can see the difference between the core concepts and example MarSystem implementations.

For example with this layout:
/src/marsyas/core/
/src/marsyas/marsystems/

Another possibility:
/src/marsyas-core
/src/marsystems

any thoughts?

Discussion

  • Luis Gustavo Martins

    I don't have any objections. In fact, I agree and my vote goes to the first form you propose (i.e. /src/marsyas/core, etc).

     
  • Thijs Koerselman

    Yes I would very much like this as well. I prefer the first form. I think that by separating the core from the marsystem, it not only cleans up the directory structure, but also makes it more apparent what parts of the core need to be exposed as a public interface.

    I think MarSystems could be viewed as a sort of "client side code", and should only "see" a minimal part of the core implementation. This will naturally improve dependencies and thus maintainance.

     

Anonymous
Anonymous

Add attachments
Cancel