review comments

StructurePlan.txt

@@ -1,11 +1,14 @@
 1 From AST (with clang lib) generate:
-generated.cpp
 generated.hpp 
 with proper members
-Have a cash system which will partially generate things
+Add it all to global context and expose it per module.
 
 Having as multiple .cpp and .hpp files does feel significantly better performance wise as it would allow to not recompile everything when something is changed and make it modular. In addition we would be able to remove need for registry if generated.hpp file would hold a reference to metadata. The downside is having a lot of files. BUT THREAAADING CAN BE IMPLEMENTED TO MAKE IT PARALLEL.
 
+Add supported list of attributes to our precompiler step, which will give warning based on if it is in there already or not.
+
+LLVM + clang; parse AST myself
+
 2 During compilation populate unordered map with type - members
 
 Different types and info we need to keep track of:
@@ -16,6 +19,7 @@ Different types and info we need to keep track of:
 	4 Functions: -//-, const, static, virtual and co, pointer to function, 
 	5 Template instances: our concept thing, member functions and variables, allignment, size, name with parameters, 
 	6 Enums: underlying type, scoped?, values
+	7 unions?
 
 
 If we implement inheritance reflected_member - we will incur 8*virtual_func_num hidden bytes
@@ -25,3 +29,13 @@ Need to figure out which modifiers exactly can be used in the future.
 Need to add a way to manually add reflections.
 
 buffer_string,dynamic string
+
+Need to include namespace into strings
+
+For choices: Hash function, hash structure, compile vs runtime containers
+
+For choices, where to store runtime reflection data - BSS vs Heap vs Stack
+
+Names are made from recursive parents. No runtime reflected namespaces objects.
+
+Had multiple parsing iterations.

VisualStudio22-All.bat

@@ -1 +1,2 @@
-rythe-build-system\premake5 --file=rythe-build-system/premake5.lua vs2022 --workspace-name=reflection-experiments
+rythe-build-system\premake5 --file=rythe-build-system/premake5.lua vs2022 --workspace-name=reflection-experiments
+pause

applications/reflection-experiments/.rythe_project

@@ -2,16 +2,9 @@ local fs = dofile("filesystem.lua")
 
 local project = {
     additional_types = { "test" },
-    dependencies = { "public rythe/rythe-standard-library", "private third_party/clang-c:header-only" },
+    dependencies = { "public rythe/rythe-standard-library", "private third_party/clang-c:header-only", 
+     "private third_party/xxhash", "public rythe/reflection-core"},
     additional_link_targets = { "libclang" },
 }
 
-function project.init(proj, context)
-    local targetDir = fs.sanitize(fs.parentPath(_MAIN_SCRIPT_DIR) .. "/build/" .. _ACTION) .. fs.getPathSeperator()
-    local srcDir = fs.sanitize(fs.parentPath(_MAIN_SCRIPT_DIR) .. "/" .. proj.location .. "/src/test.hpp")
-    proj.post_build = { "xcopy \"" .. srcDir .. "\" \"" .. targetDir .. "\" /i /r /y /s /c" }
-
-    return proj
-end
-
 return project

applications/reflection-experiments/src/source.cpp

@@ -1,4 +1,34 @@
+#include <unordered_set>
 
-int main() {
+#include <rsl/cli>
+
+#include "impl/reflection_parsers/ast_source_parser.h"
+#include "impl/reflection_context/reflection_context.h"
+#include "impl/reflection_context/reflection_registration_registry.h"
+//#include "target/test_generated.hpp"
+
+int main(int, char* argv[])
+{
+    rsl::cli_parser cmdl(argv);
+
+    // any particular reason for unordered_set?
+    std::unordered_set<std::string> folders;
+
+    for(auto& param : cmdl.params("fpath"))
+    {
+        std::cout << '\t' << param.first << " : " << param.second << '\n';
+        folders.insert(param.second);
+    }
+
+    auto parser = reflection_parsers::ast_source_parser();
+
+    parser.generate_reflection_files(folders);
+    reflection_registration_registry::instance().clear();
+    //register_reflection_file_0();
+    reflection_registration_registry::instance().run_all();
+    // lala
+    reflection_context::instance().print_classes();    
+
+
     return 0;
 }

applications/reflection-experiments/src/target/test.hpp

@@ -0,0 +1,34 @@
+class very_cool_target_class
+{
+public:
+    int int_variable;
+
+    float variable;
+
+    void void_function(int int_parameter);
+
+    class nested_class
+    {
+    public:
+        nested_class() : value(0) {}
+
+        void set_value(int v) { value = v; }
+
+        int get_value() const { return value; }
+    private:
+        int value;
+    };
+
+private:
+    char char_variable;
+};
+
+class target_class2
+{
+public:
+    int int_variable2;
+
+    void void_function2(int int_parameter);
+private:
+    char char_variable2;
+};

applications/reflection-experiments/src/target/test_generated.hpp

@@ -0,0 +1,72 @@
+#include "../runtime_reflection_containers/runtime_reflected_class.h"
+#include "../runtime_reflection_containers/runtime_reflected_function.h"
+#include "../runtime_reflection_containers/runtime_reflected_variable.h"
+#include "../reflection_id/reflection_id.h"
+#include "../reflection_context/reflection_registration_registry.h"
+#include "../reflection_context/reflection_context.h"
+void register_reflection_file_0()
+{
+    runtime_reflected_class class_14985410343605238495;
+    class_14985410343605238495.id = reflection_id(1644627251862529318ULL, 13483286640053009272ULL, 14985410343605238495ULL);
+    class_14985410343605238495.type_spelling = rsl::dynamic_string::from_string_length("very_cool_target_class");
+
+    runtime_reflected_variable variable_6501503083828029313;
+    variable_6501503083828029313.id = reflection_id(10933754376540684078ULL, 16350452351519940193ULL, 6501503083828029313ULL);
+    variable_6501503083828029313.offset = 0;
+    variable_6501503083828029313.type_spelling = rsl::dynamic_string::from_string_length("int");
+    class_14985410343605238495.add_variable(std::move(variable_6501503083828029313));
+
+    runtime_reflected_variable variable_126163732700301715;
+    variable_126163732700301715.id = reflection_id(5770744143431503129ULL, 12913254394975708779ULL, 126163732700301715ULL);
+    variable_126163732700301715.offset = 4;
+    variable_126163732700301715.type_spelling = rsl::dynamic_string::from_string_length("float");
+    class_14985410343605238495.add_variable(std::move(variable_126163732700301715));
+
+    runtime_reflected_variable variable_17337086075296791468;
+    variable_17337086075296791468.id = reflection_id(15779825311949492597ULL, 9221959329564397894ULL, 17337086075296791468ULL);
+    variable_17337086075296791468.offset = 8;
+    variable_17337086075296791468.type_spelling = rsl::dynamic_string::from_string_length("char");
+    class_14985410343605238495.add_variable(std::move(variable_17337086075296791468));
+
+    runtime_reflected_class class_3168970719907517523;
+    class_3168970719907517523.id = reflection_id(8627741348740311923ULL, 10691473971882747459ULL, 3168970719907517523ULL);
+    class_3168970719907517523.type_spelling = rsl::dynamic_string::from_string_length("nested_class");
+
+    runtime_reflected_variable variable_9809713342160364025;
+    variable_9809713342160364025.id = reflection_id(785375445157317512ULL, 16350452351519940193ULL, 9809713342160364025ULL);
+    variable_9809713342160364025.offset = 0;
+    variable_9809713342160364025.type_spelling = rsl::dynamic_string::from_string_length("int");
+    class_3168970719907517523.add_variable(std::move(variable_9809713342160364025));
+
+    class_14985410343605238495.add_class(std::move(class_3168970719907517523));
+
+    reflection_context::instance().add_class(std::move(class_14985410343605238495));
+
+    runtime_reflected_class class_862151746601595251;
+    class_862151746601595251.id = reflection_id(14404923345981690111ULL, 31932944099034707ULL, 862151746601595251ULL);
+    class_862151746601595251.type_spelling = rsl::dynamic_string::from_string_length("target_class2");
+
+    runtime_reflected_variable variable_13153249853553726320;
+    variable_13153249853553726320.id = reflection_id(4281340778281841492ULL, 16350452351519940193ULL, 13153249853553726320ULL);
+    variable_13153249853553726320.offset = 0;
+    variable_13153249853553726320.type_spelling = rsl::dynamic_string::from_string_length("int");
+    class_862151746601595251.add_variable(std::move(variable_13153249853553726320));
+
+    runtime_reflected_variable variable_6526241590453974317;
+    variable_6526241590453974317.id = reflection_id(10234543199726084352ULL, 9221959329564397894ULL, 6526241590453974317ULL);
+    variable_6526241590453974317.offset = 4;
+    variable_6526241590453974317.type_spelling = rsl::dynamic_string::from_string_length("char");
+    class_862151746601595251.add_variable(std::move(variable_6526241590453974317));
+
+    reflection_context::instance().add_class(std::move(class_862151746601595251));
+
+};
+
+struct reflection_file_registration_helper
+{
+    reflection_file_registration_helper()
+    {
+        reflection_registration_registry::instance().add(&register_reflection_file_0);
+    }
+};
+static reflection_file_registration_helper registration_instance;