Skip to content

Conversation

@tyeth
Copy link
Member

@tyeth tyeth commented Oct 15, 2025

Draft PR to be able to leave comments on the changes

#include "Wippersnapper_Networking.h"

// // Global WS instance - defined as platform-specific type in Wippersnapper.cpp
// extern Wippersnapper_WiFi WS; ///< Global member variable for callbacks
Copy link
Member Author

@tyeth tyeth Oct 15, 2025

Choose a reason for hiding this comment

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

Drop this, but leave comment: global WS now is a Wippersnapper_WiFi which comes via Wippersnapper_Networking.h, but needs to be extern'd in each cpp file.

Copy link
Member Author

Choose a reason for hiding this comment

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

resolved in #851

*/

#include "Wippersnapper.h"
#include "Wippersnapper_Networking.h"
Copy link
Member Author

Choose a reason for hiding this comment

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

should be via Wippersnapper.h now so drop this include

Copy link
Member Author

Choose a reason for hiding this comment

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

resolved in #851

Copy link
Member Author

@tyeth tyeth Oct 15, 2025

Choose a reason for hiding this comment

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

all these need fixing up like esp32 (other Arch builds not tested)

Copy link
Member Author

Choose a reason for hiding this comment

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

resolved in #851

@tyeth
Copy link
Member Author

tyeth commented Dec 17, 2025

Linking issue rearing it's ugly head again in #851 due to calling pingBroker and runNetFSM from displays/controller.cpp
Will need to reuse something like this.

tyeth added a commit to tyeth/Adafruit_Wippersnapper_Arduino that referenced this pull request Dec 17, 2025
@tyeth tyeth mentioned this pull request Dec 17, 2025
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.

1 participant