Skip to content

update stream.go #332

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

update stream.go #332

wants to merge 1 commit into from

Conversation

vearne
Copy link

@vearne vearne commented Dec 6, 2018

增加限制json Marsharl递归层级,来防止循环引用,引起的栈溢出

@codecov
Copy link

codecov bot commented Dec 6, 2018

Codecov Report

Merging #332 into master will increase coverage by 0.02%.
The diff coverage is 31.81%.

Impacted file tree graph

@@            Coverage Diff            @@
##           master    #332      +/-   ##
=========================================
+ Coverage   81.48%   81.5%   +0.02%     
=========================================
  Files          41      41              
  Lines        5013    5035      +22     
=========================================
+ Hits         4085    4104      +19     
+ Misses        807     803       -4     
- Partials      121     128       +7
Impacted Files Coverage Δ
config.go 89.36% <ø> (ø) ⬆️
reflect_array.go 75% <0%> (-2.59%) ⬇️
reflect_slice.go 93.44% <0%> (-3.17%) ⬇️
reflect_map.go 85.78% <0%> (-1.72%) ⬇️
reflect_struct_encoder.go 87.5% <0%> (-1.49%) ⬇️
pool.go 100% <100%> (ø) ⬆️
stream.go 82.72% <50%> (-3.28%) ⬇️
reflect_struct_decoder.go 48.74% <0%> (+1.67%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d05f387...a43d14e. Read the comment docs.

@vearne
Copy link
Author

vearne commented Dec 6, 2018

处理循环引用所导致的栈溢出问题
类似下面的场景
http://vearne.cc/archives/1464

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
1 participant