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

late is not correct for _next in chain_of_responsibility #10

Closed
mortezasojodi opened this issue Jan 11, 2023 · 1 comment · Fixed by #11
Closed

late is not correct for _next in chain_of_responsibility #10

mortezasojodi opened this issue Jan 11, 2023 · 1 comment · Fixed by #11

Comments

@mortezasojodi
Copy link

mortezasojodi commented Jan 11, 2023

when you define _next late, for last Logger we get error (or any logger that we dont need _next ) because this variable is not initilized.
we must change it to Nullable inctance instead, like below:
Logger? _next;

@scottt2
Copy link
Owner

scottt2 commented Jan 12, 2023

Hey @mortezasojodi ! Thanks for the report. I should be able to get to fixing this by the weekend or if you want to go ahead and file a PR with the fix. I am always happy to pull in contributor work!

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 a pull request may close this issue.

2 participants