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

Remove the assert on AppendDelayed #867

Closed
vgvassilev opened this issue Apr 24, 2024 · 0 comments · Fixed by #872
Closed

Remove the assert on AppendDelayed #867

vgvassilev opened this issue Apr 24, 2024 · 0 comments · Fixed by #872
Milestone
v1.5

Comments

@vgvassilev
Copy link
Owner

PR #766 inserted an assert which breaks the incremental processing case. Relax the assert:

clad/tools/ClangPlugin.h

Line 275 in 962e5d9

assert(!m_HasMultiplexerProcessedDelayedCalls);

@vgvassilev vgvassilev added this to the v1.5 milestone Apr 24, 2024
vgvassilev added a commit that referenced this issue Apr 26, 2024
@vgvassilev
Add clang-repl tests to make sure we do not break repls.
92fc173 
Fixes #854, #867.
@vgvassilev vgvassilev mentioned this issue Apr 26, 2024
Merged
vgvassilev added a commit that referenced this issue Apr 26, 2024
@vgvassilev
Add clang-repl tests to make sure we do not break repls.
77ed5b4 
Fixes #854, fixes #867.
vgvassilev added a commit that referenced this issue Apr 26, 2024
@vgvassilev
Add clang-repl tests to make sure we do not break repls.
9ff24d9 
Fixes #854, fixes #867.
vgvassilev added a commit that referenced this issue Apr 26, 2024
@vgvassilev
Add clang-repl tests to make sure we do not break repls.
eaead8b 
Fixes #854, fixes #867.
vgvassilev added a commit that referenced this issue Apr 26, 2024
@vgvassilev
Add clang-repl tests to make sure we do not break repls.
2f0ca61 
Fixes #854, fixes #867.
vgvassilev added a commit that referenced this issue Apr 26, 2024
@vgvassilev
Add clang-repl tests to make sure we do not break repls.
989395d 
Fixes #854, fixes #867.
vgvassilev added a commit that referenced this issue Apr 27, 2024
@vgvassilev
Add clang-repl tests to make sure we do not break repls.
509e151 
Fixes #854, fixes #867.
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
v1.5
Development

Successfully merging a pull request may close this issue.

Add clang-repl tests to make sure we do not break repls. vgvassilev/clad
1 participant
@vgvassilev