Skip to content

The use of bind in Ticker.h is prone to type inference failure #6129

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.

Sign up for GitHub

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

Merged
d-a-v merged 6 commits into from
May 25, 2019
Merged

The use of bind in Ticker.h is prone to type inference failure #6129

Changes from 1 commit
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
c6b9f13
std::bind has issues with type inference, use lambdas whereever possi…
dok-net May 22, 2019
362c557
Fix indentation.
dok-net May 22, 2019
416a638
Merge branch 'master' into refactorbindtolambda
dok-net May 23, 2019
43a88de
More descriptive placeholder name in lambda expression
dok-net May 24, 2019
e6d9ba2
Use formal argument names for remaining currying placeholders
dok-net May 24, 2019
d69a15d
Merge branch 'master' into refactorbindtolambda
d-a-v May 25, 2019
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Fix indentation.
  • Loading branch information
@dok-net
dok-net committed May 22, 2019
commit 362c5573a43e1065e608ad5e5d2cc5d82109ad73
30 changes: 15 additions & 15 deletions libraries/ESP8266mDNS/src/LEAmDNS.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -97,14 +97,14 @@ bool MDNSResponder::begin(const char* p_pcHostname) {
m_GotIPHandler = WiFi.onStationModeGotIP([this](const WiFiEventStationModeGotIP& pEvent) {
(void) pEvent;
// Ensure that _restart() runs in USER context
schedule_function([this]() { MDNSResponder::_restart(); });
schedule_function([this]() { MDNSResponder::_restart(); });
});

m_DisconnectedHandler = WiFi.onStationModeDisconnected([this](const WiFiEventStationModeDisconnected& pEvent) {
(void) pEvent;
// Ensure that _restart() runs in USER context
schedule_function([this]() { MDNSResponder::_restart(); });
});
schedule_function([this]() { MDNSResponder::_restart(); });
});

bResult = _restart();
}
Expand Down Expand Up @@ -137,10 +137,10 @@ bool MDNSResponder::begin(const char* p_pcHostname,
*/
bool MDNSResponder::close(void) {

m_GotIPHandler.reset(); // reset WiFi event callbacks.
m_DisconnectedHandler.reset();
m_GotIPHandler.reset(); // reset WiFi event callbacks.
m_DisconnectedHandler.reset();

_announce(false, true);
_announce(false, true);
_resetProbeStatus(false); // Stop probing

_releaseServiceQueries();
Expand All @@ -159,7 +159,7 @@ bool MDNSResponder::close(void) {
*/

bool MDNSResponder::end(void) {
return close();
return close();
}

/*
Expand Down Expand Up @@ -832,11 +832,11 @@ uint32_t MDNSResponder::answerCount(const MDNSResponder::hMDNSServiceQuery p_hSe

std::vector<MDNSResponder::MDNSServiceInfo> MDNSResponder::answerInfo (const MDNSResponder::hMDNSServiceQuery p_hServiceQuery) {
std::vector<MDNSResponder::MDNSServiceInfo> tempVector;
for (uint32_t i=0;i<answerCount(p_hServiceQuery);i++)
for (uint32_t i=0;i<answerCount(p_hServiceQuery);i++)
{
tempVector.emplace_back(*this,p_hServiceQuery,i);
tempVector.emplace_back(*this,p_hServiceQuery,i);
}
return tempVector;
return tempVector;
}

/*
Expand Down Expand Up @@ -1053,14 +1053,14 @@ const char* MDNSResponder::answerTxts(const MDNSResponder::hMDNSServiceQuery p_h
*/
bool MDNSResponder::setHostProbeResultCallback(MDNSResponder::MDNSHostProbeFn p_fnCallback) {

m_HostProbeInformation.m_fnHostProbeResultCallback = p_fnCallback;
m_HostProbeInformation.m_fnHostProbeResultCallback = p_fnCallback;

return true;
}

bool MDNSResponder::setHostProbeResultCallback(MDNSHostProbeFn1 pfn) {
using namespace std::placeholders;
return setHostProbeResultCallback([obj=std::ref(*this), pfn](const char* _1, bool _2) { pfn(obj, _1, _2); });
using namespace std::placeholders;
return setHostProbeResultCallback([obj=std::ref(*this), pfn](const char* _1, bool _2) { pfn(obj, _1, _2); });
}

/*
Expand Down Expand Up @@ -1088,8 +1088,8 @@ bool MDNSResponder::setServiceProbeResultCallback(const MDNSResponder::hMDNSServ

bool MDNSResponder::setServiceProbeResultCallback(const MDNSResponder::hMDNSService p_hService,
MDNSResponder::MDNSServiceProbeFn1 p_fnCallback) {
using namespace std::placeholders;
return setServiceProbeResultCallback(p_hService, [obj=std::ref(*this), p_fnCallback](const char* _1, const hMDNSService _2, bool _3) { p_fnCallback(obj, _1, _2, _3); });
using namespace std::placeholders;
return setServiceProbeResultCallback(p_hService, [obj=std::ref(*this), p_fnCallback](const char* _1, const hMDNSService _2, bool _3) { p_fnCallback(obj, _1, _2, _3); });
}


Expand Down