Skip to content

added public cleanAPlist() function. #4020

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

Closed
wants to merge 6 commits into from
Closed

added public cleanAPlist() function. #4020

wants to merge 6 commits into from

Conversation

NayanKaran
Copy link
Contributor

cleaning AP list without calling destructor.

@NayanKaran
Copy link
Contributor Author

@igrr can you please help me to commit this?

@devyte
Copy link
Collaborator

devyte commented Jan 6, 2018

@bapidebabrata I can help with this, I need to look at wifimulti again anyways as part of another issue, and I'm familiar with the code.
It might take me a couple of days to get to it, though, I'm currently looking at something else.

@earlephilhower
Copy link
Collaborator

This conflicts and looks like it's missing bits to actually do anything. Closing.

@devyte
Copy link
Collaborator

devyte commented Jan 26, 2019

This just exposes a private method, which allows clearing the content. I wanted to remove the public methods and make the private ones public +rename, but it's not really necessary.

@devyte devyte reopened this Jan 26, 2019
@earlephilhower
Copy link
Collaborator

Closing as #4107 is identical and doesn't have the linefeed changes.

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 this pull request may close these issues.

4 participants