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

跟着change.log看代码，bean-definition-and-bean-definition-registry ==> instantiation-strategy ，instantiation-strategy 多了好多东西啊 #35

Open
wzdlagrange opened this issue Nov 24, 2022 · 4 comments
Open

跟着change.log看代码，bean-definition-and-bean-definition-registry ==> instantiation-strategy ，instantiation-strategy 多了好多东西啊 #35

wzdlagrange opened this issue Nov 24, 2022 · 4 comments

Comments

@wzdlagrange
Copy link

No description provided.

@wzdlagrange wzdlagrange changed the title 跟着change.log 跟着change.log看代码，bean-definition-and-bean-definition-registry ==> instantiation-strategy ，instantiation-strategy 多了好多东西啊 Nov 24, 2022
@wzdlagrange
Copy link
Author

wzdlagrange commented Nov 24, 2022 •
edited
Loading

看得眼花了，是我对项目的定位错了吗？
我是想着跟着这个项目，来学习Spring的思想的。
两个分支内容跨度有点大，结果我理解起来就有些费劲了...

@Jiewyjson
Copy link

ioc篇里面就这个策略实例化的分支有点问题，对了很多下面篇幅的东西，其他分支还没发现什么问题。

@DerekYRC
Copy link
Owner

@Jiewyjson @wzdlagrange 失误把main分支的代码合并到instantiation-strategy分支了，我今晚改下，暂时可以跳到instantiation-strategy分支的be8f86f32b308662ff1709e3c85375266f3050af提交看看

@DerekYRC
Copy link
Owner

@wzdlagrange 改好了，再看下instantiation-strategy分支的代码

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
3 participants
@wzdlagrange @DerekYRC @Jiewyjson