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

TLS_NAT_CONST typo fixes #920

Merged
merged 2 commits into from
Oct 25, 2023
Merged

TLS_NAT_CONST typo fixes #920

merged 2 commits into from
Oct 25, 2023

Conversation

DrDet
Copy link
Contributor

@DrDet DrDet commented Oct 25, 2023 •
edited
Loading

  • keep old incorrect behaviour in tl-compiler for backward-compatibility
  • maintain correct and incorrect tlo format in vkext just in case

relates to #913

@DrDet DrDet added the small fix When it's not an huge enhancement label Oct 25, 2023
@DrDet DrDet self-assigned this Oct 25, 2023
@DrDet DrDet force-pushed the dvaksman/vkext/fix-tl-nat-expr-parsing branch from 199b8d5 to 72d5617 Compare October 25, 2023 10:54
@DrDet DrDet changed the title [vkext] fix legacy typo in TL nat const parsing [tl-compiler] keep old behaviour for backward compatibility Oct 25, 2023
@DrDet DrDet changed the title [tl-compiler] keep old behaviour for backward compatibility [tl-compiler] revert to old behaviour for backward compatibility Oct 25, 2023
@DrDet DrDet force-pushed the dvaksman/vkext/fix-tl-nat-expr-parsing branch from 64227ef to 51d6959 Compare October 25, 2023 11:16
@DrDet DrDet changed the title [tl-compiler] revert to old behaviour for backward compatibility [tl-compiler] keep old behaviour for backward compatibility Oct 25, 2023
@DrDet DrDet changed the title [tl-compiler] keep old behaviour for backward compatibility TLS_NAT_CONST typo fixes Oct 25, 2023
@DrDet DrDet merged commit 51cf776 into master Oct 25, 2023
5 checks passed
@DrDet DrDet deleted the dvaksman/vkext/fix-tl-nat-expr-parsing branch October 25, 2023 11:29
@Danil42Russia Danil42Russia added this to the next milestone Nov 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@tolk-vm tolk-vm tolk-vm approved these changes

Assignees

@DrDet DrDet

Labels
small fix When it's not an huge enhancement
Projects
None yet
Milestone
10.11.2023
Development

Successfully merging this pull request may close these issues.
3 participants
@DrDet @tolk-vm @Danil42Russia