-
Notifications
You must be signed in to change notification settings - Fork 5
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
wrapper function added #16
Conversation
@bhbam please also make the changes for the other classes in the All of these have the same issue ( |
@bhbam can you also please edit the description for the pull request, just to describe the problem it is trying to fix. This will be useful in case we are trying to look back at the history and remember how we solved this |
@bhbam It looks like you made the pull request, but then pushed two commits that reverted all of your changes, so nothing is getting changed here. Can you please reset to before the last two commits so that we can review the changes |
Sorry I want to push the changes to new branch (not master) so I tried to revert the change. Now I am pushed to new branch and created pull request. |
Closing this pull request without merging. It is superceded by #17 |
The wrapper function was added to prevent initializing the configuration over and over and segmentation fault error solved