Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Make normal library mode (not header-only) #128

Open
YarikTH opened this issue Aug 24, 2023 · 0 comments
Open

Make normal library mode (not header-only) #128

YarikTH opened this issue Aug 24, 2023 · 0 comments

Comments

@YarikTH
Copy link
Owner

YarikTH commented Aug 24, 2023

Description

There is not so much non template code in the library but there is some. According to the compile time benchmarks it allows to spare ~100 ms per cpp in which ureact is included (See #126).

Need to repeat fmtlib approach, where cpp part is in header files, that are just included in header-only mode, while they are included from the library cpp file in library mode. The only difference is - header-only. It keeps ureact's compatibility with godbolt.

Need to make sure that make amalgamated works correctly.
YarikTH added a commit that referenced this issue Aug 24, 2023
@YarikTH
#128 Add initial version of UREACT_HEADER_ONLY and UREACT_FUNC defines
614b6d6
YarikTH added a commit that referenced this issue Aug 24, 2023
@YarikTH
#128 Extract node_id_vector definition in graph_interface.inl
c61f5c5
YarikTH added a commit that referenced this issue Aug 24, 2023
@YarikTH
#128 Extract transaction related definitions in transaction.inl
daa924b
YarikTH added a commit that referenced this issue Aug 24, 2023
@YarikTH
#128 Extract context related definitions in context.inl
e44b4ad
YarikTH added a commit that referenced this issue Aug 24, 2023
@YarikTH
#128 Extract node_base definition in node_base.inl
25b5bc8
YarikTH added a commit that referenced this issue Aug 24, 2023
@YarikTH
#128 Extract observer related definitions in observer.inl
ac6fdb5
YarikTH added a commit that referenced this issue Aug 24, 2023
@YarikTH
#128 Add detail::make_react_graph() as preparation for hiding react_g…
68f6006 
…raph's implementation
YarikTH added a commit that referenced this issue Aug 24, 2023
@YarikTH
#128 Move ureact/detail/graph_impl.hpp into `ureact/detail/graph_im…
32ad4dc 
…pl.inl`
YarikTH added a commit that referenced this issue Aug 24, 2023
@YarikTH
#128 Extract react_graph related definitions in graph_impl.inl
34c5aba
YarikTH added a commit that referenced this issue Aug 26, 2023
@YarikTH
#128 Add initial version of UREACT_HEADER_ONLY and UREACT_FUNC defines
d79b7f8
YarikTH added a commit that referenced this issue Aug 26, 2023
@YarikTH
#128 Add checks for *.inl headers in ureact_self_contained_headers_test
89c0866
YarikTH added a commit that referenced this issue Aug 26, 2023
@YarikTH
#128 Extract node_id_vector definition in graph_interface.inl
f148f52
YarikTH added a commit that referenced this issue Aug 26, 2023
@YarikTH
#128 Extract transaction related definitions in transaction.inl
a0ead64
YarikTH added a commit that referenced this issue Aug 26, 2023
@YarikTH
#128 Extract context related definitions in context.inl
2e2fba3
YarikTH added a commit that referenced this issue Aug 26, 2023
@YarikTH
#128 Extract node_base definition in node_base.inl
462d19b
YarikTH added a commit that referenced this issue Aug 26, 2023
@YarikTH
# #128 Extract observer related definitions in observer.inl
854b253
YarikTH added a commit that referenced this issue Aug 26, 2023
@YarikTH
#128 Add detail::make_react_graph() as preparation for hiding react_g…
f8382b6 
…raph's implementation
YarikTH added a commit that referenced this issue Aug 26, 2023
@YarikTH
#128 Move ureact/detail/graph_impl.hpp into `ureact/detail/graph_im…
0280f89 
…pl.inl`
YarikTH added a commit that referenced this issue Aug 26, 2023
@YarikTH
#128 Extract react_graph related definitions in graph_impl.inl
ff4b126
YarikTH added a commit that referenced this issue Aug 26, 2023
@YarikTH
#128 Add initial version of UREACT_HEADER_ONLY and UREACT_FUNC defines
d080775
YarikTH added a commit that referenced this issue Aug 26, 2023
@YarikTH
#128 Add checks for *.inl headers in ureact_self_contained_headers_test
146d682
YarikTH added a commit that referenced this issue Aug 26, 2023
@YarikTH
#128 Extract node_id_vector definition in graph_interface.inl
42b031f
YarikTH added a commit that referenced this issue Aug 26, 2023
@YarikTH
#128 Extract transaction related definitions in transaction.inl
9648059
YarikTH added a commit that referenced this issue Aug 26, 2023
@YarikTH
#128 Extract context related definitions in context.inl
d71fcd5
YarikTH added a commit that referenced this issue Aug 26, 2023
@YarikTH
#128 Extract node_base definition in node_base.inl
1de7650
YarikTH added a commit that referenced this issue Aug 26, 2023
@YarikTH
# #128 Extract observer related definitions in observer.inl
3baecd6
YarikTH added a commit that referenced this issue Aug 26, 2023
@YarikTH
#128 Add detail::make_react_graph() as preparation for hiding react_g…
c084f6b 
…raph's implementation
YarikTH added a commit that referenced this issue Aug 26, 2023
@YarikTH
#128 Move ureact/detail/graph_impl.hpp into `ureact/detail/graph_im…
c104bc5 
…pl.inl`
YarikTH added a commit that referenced this issue Aug 26, 2023
@YarikTH
#128 Extract react_graph related definitions in graph_impl.inl
94f13f2
YarikTH added a commit that referenced this issue Aug 26, 2023
@YarikTH
#128 Extract observer related definitions in observer.inl
404ce9b
YarikTH added a commit that referenced this issue Aug 26, 2023
@YarikTH
#128 Add detail::make_react_graph() as preparation for hiding react_g…
24683fa 
…raph's implementation
YarikTH added a commit that referenced this issue Aug 26, 2023
@YarikTH
#128 Move ureact/detail/graph_impl.hpp into `ureact/detail/graph_im…
1569d54 
…pl.inl`
YarikTH added a commit that referenced this issue Aug 26, 2023
@YarikTH
#128 Extract react_graph related definitions in graph_impl.inl
e8bf664
YarikTH added a commit that referenced this issue Aug 26, 2023
@YarikTH
#128 Move ureact/detail/graph_impl.hpp into `ureact/detail/graph_im…
31970d8 
…pl.inl`
YarikTH added a commit that referenced this issue Aug 26, 2023
@YarikTH
#128 Extract react_graph related definitions in graph_impl.inl
7463e7d
YarikTH added a commit that referenced this issue Aug 26, 2023
@YarikTH
#128 Mark non-template public classes and functions as UREACT_API
01162d6 
UREACT_API, UREACT_EXPORT and UREACT_SHARED are ported from `{fmt}`
YarikTH added a commit that referenced this issue Sep 10, 2023
@YarikTH
#128 fix generate_amalgamated_file.py after extracting some inline files
808902c 
* conditional includes starts with "#   include" and regexp pattern used to discover includes was not ready for it
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@YarikTH