Skip to content
This repository has been archived by the owner on May 29, 2024. It is now read-only.

Bug Fixes #1330

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

pushkarSingh9900
Copy link

PR Checklist:

  • My submission is formatted according to the guidelines in the contributing guide
  • My addition is on refer on the language README.md file
  • My addition does not have a spelling problem
  • My submission has a proper and user-friendly description of the algorithm
  • My submission has the time complexity of the algorithm
  • My submission has sample input-output of the program (NOT FOR PYTHON)

What kind of change does this PR introduce? (check at least one)

  • Bugfix
  • New algorithm
  • Optimization in previous algorithms
  • Code style update
  • Refactor
  • Documentation
  • Other, please describe:

Briefly describe the changes in this PR
When i ran this code in my system, it has multiple errors in the Java language of algorithms. So, i fixed them.

Other information:

Copy link

welcome bot commented May 9, 2024

Thanks for opening this pull request! Please check out our contributing guidelines.

Copy link

@MdialloC19 MdialloC19 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why do you change the scope of class, by making them default scope. they may be cause bug for some use.
Apart from that nothing has been reported. Good Work

@pushkarSingh9900
Copy link
Author

Right, i will change it back to public.
Thanks

Copy link

@MdialloC19 MdialloC19 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Everything is ok

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants