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

improve scala compile support #1470 #1473

Closed
wants to merge 3 commits into from

Conversation

sinodzh
Copy link

@sinodzh sinodzh commented May 12, 2023

What this PR does / why we need it?

improve scala compile support #1470

Summary of your change

add function put in JSONObject

Please indicate you've done the following:

  • Made sure tests are passing and test coverage is added if needed.
  • Made sure commit message follow the rule of Conventional Commits specification.
  • Considered the docs impact and opened a new docs issue or PR with docs changes if needed.

@CLAassistant
Copy link

CLAassistant commented May 12, 2023

CLA assistant check
All committers have signed the CLA.

@wenshao
Copy link
Member

wenshao commented May 12, 2023

你在执行下mvn test修复下单测的问题

@sinodzh
Copy link
Author

sinodzh commented May 14, 2023

你在执行下mvn test修复下单测的问题
#507
#518
fastjson2需要支持非string的key的put是吧,如果是的话,那还不能我这样改。

@wenshao
Copy link
Member

wenshao commented May 14, 2023

你在执行下mvn test修复下单测的问题
#507
#518
fastjson2需要支持非string的key的put是吧,如果是的话,那还不能我这样改。

是的,要支持non-string key

@sinodzh
Copy link
Author

sinodzh commented May 15, 2023

你在执行下mvn test修复下单测的问题
#507
#518
fastjson2需要支持非string的key的put是吧,如果是的话,那还不能我这样改。

是的,要支持non-string key

那non-string key,是一个jsonobject实例里都是一个类型的key,还是可以多个类型的key,,比如"a",1,2.0l 这样的key

@sinodzh sinodzh closed this May 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants