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.

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

LDEV-4648 support partially escaped sequence #2227

Closed
wants to merge 236 commits into from

Conversation

hideo67
Copy link
Contributor

@hideo67 hideo67 commented Sep 8, 2023

@CLAassistant
Copy link

CLAassistant commented Sep 8, 2023

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
3 out of 4 committers have signed the CLA.

✅ michaeloffner
✅ zspitzer
✅ hideo67
❌ isapir
You have signed the CLA already but the status is still pending? Let us recheck it.

isapir and others added 26 commits March 9, 2024 16:05
# Conflicts:
#	core/src/main/java/lucee/transformer/bytecode/util/SystemExitScanner.java
#	loader/build.xml
#	loader/pom.xml
#	loader/src/main/java/lucee/loader/engine/CFMLEngineFactory.java
remove references to "javax" that can be avoided
# Conflicts:
#	core/src/main/java/lucee/transformer/cfml/Data.java
#	core/src/main/java/lucee/transformer/cfml/evaluator/impl/Loop.java
#	core/src/main/java/lucee/transformer/cfml/tag/CFMLTransformer.java
#	loader/build.xml
#	loader/pom.xml
@hideo67
Copy link
Contributor Author

hideo67 commented Mar 17, 2024

I merged 6.1 into this PR branch, and changed the implementation to use apache commons.

@zspitzer
Copy link
Member

Can you refile this the PR, since the merges it's become impossibly confusing to review?

@hideo67
Copy link
Contributor Author

hideo67 commented Mar 18, 2024

I'm closing this since I refiled it as #2335

@hideo67 hideo67 closed this Mar 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants