Nueva estructura de directorios

[adrigm]

No me acaba de convencer la nueva estructura de directorios. Algunas cuestiones:

• ¿Por qué Core.hpp está dentro de su propia carpeta si se supone que es un archivo que incluye todos los archivos del Core?
• El log debería ser parte del Core ya que este es el módulo base de el que van a depender todos los demás módulos ya que en él se encuentran todas las tareas básicas del motor.
• Lo mismo con StringsUtils todo esto debe pertenecer al módulo Core que para eso es el Core, creo que se intentaba es dar la máxima modularidad posible y no creo que esto sea del todo conveniente en este proyecto.
• Los archivos cpp se encuentren en src/GDE/source/ y los archivos hpp se encuentran en src/GDE/include/GDE/ creo que es un poco incosistente que no se encuentre en la misma estructura.

Siento reabrir este debate de nuevo, pero creo que hay que dejar el sistema lo mejor posible antes de que esto avance más y sea más complicado.

[danigomez]

• mmm No entiendo el problema de que el header esté ahí, podrías explicarlo mejor??
• Creo que la idea de el módulo Core es que tenga lo que hace al funcionamiento del engine en si, Log y StringUtils son algo ajeno al engine en si, no hacen a su funcionamiento. Ese es el críterio que creo se uso para hacer la división de los módulos.
• Lo del módulo de Utils separado también me parece bien, en la medida que vayamos agregando más funcionalidades en Util va a estar bueno que esté separado en su propio módulo.
• Sobre lo que decís de los paths, estoy de acuerdo, deberiamos cambiar el path de include a src/GDE/include/ directamente.

[rickyah]

1. No entiendo qué problema hay. Core contiene los headers de core, y debería estar en su carpeta. No veo el problema, lo siento. Estaba pensando en más adelante crear un GDE.h que incluya el resto de headers de cada módulo, de cara a tener un único fichero para incluir todo (para facilitar la vida a los usuarios del engine)
2. (ver siguiente línea)
3. Ya lo había comentado aquí: https://github.com/genbetadev/Genbeta-Dev-Engine/issues/57#issuecomment-28474939
4. Eso ha sido para poder indicarle al compilador que use la carpeta include como un directorio donde buscar headers, pero a la vez poder referenciar los headers dentro del código con la cadena 'GDE' delante (e.g. #include <GDE/Core>) También es más fácil copiar los headers así: con copiar la carpeta include ya están colocados correctamente. Se puede cambiar pero entonces hay que quitar la cadena GDE de los includes (e.g. #include <Core>)

Y sobre esto:

creo que se intentaba es dar la máxima modularidad posible y no creo que esto sea del todo conveniente en este proyecto

Dar la máxima modularidad posible es siempre conveniente, en este y cualquier proyecto, salvo casos muy excepcionales.