Chris--A
commented
8 years ago Hi, this error is new to me! (this isn't my lib, I just created a repo so it can be used through the Arduino library manager). However I have taken control of its updates and now take responsibility, so I thank you for raising the issue.
This looks like a simple error involving multiple names. Its not specific to this lib or the GSM lib.
The usual way for this to be resolved in C++ is to use namespaces by wrapping the include of one of the libs in a namespace giving it an extra level of indirection. Or as others may have mentioned, pick one of the libraries and change the name so there is no conflict (not what you want).
Researching this, I see many years of users asking that this be fixed, and not expecting to be told fix it yourself by editing a library. What then for the next release, fix it again?
Here is how you can implement it so you will not have to change any library code with future updates. And yes you can be the first to have a working solution without touching the library!
First wrap the include in a namespace in your sketch file:
namespace KP{ //You can use any name you like (as long as it is unique)
#include <Keypad.h>
};Next step, create a cpp file in your sketch directory (from the IDE use the menu Sketch->Show Sketch folder).
Inside the new cpp file add this code:
namespace KP{ //Same namespace name you used in your sketch
#include <Key.cpp>
#include <Keypad.cpp>
};Now in your sketch, you need to access the library components with the fully qualified name. Here is a sample:
namespace KP{
#include <Keypad.h>
};
const byte ROWS = 4; //four rows
const byte COLS = 3; //three columns
char keys[ROWS][COLS] = {
{'1','2','3'},
{'4','5','6'},
{'7','8','9'},
{'*','0','#'}
};
byte rowPins[ROWS] = {5, 4, 3, 2}; //connect to the row pinouts of the kpd
byte colPins[COLS] = {8, 7, 6}; //connect to the column pinouts of the kpd
KP::Keypad kpd = KP::Keypad( makeKeymap(keys), rowPins, colPins, ROWS, COLS );
void setup() {}
void loop() {
switch (kpd.key[0].kstate) { // Report active key state : IDLE, PRESSED, HOLD, or RELEASED
case KP::PRESSED:
break;
case KP::HOLD:
break;
case KP::RELEASED:
break;
case KP::IDLE:
break;
}
}Notice the KP:: in front of the Keypad library components. Also #defines aren't specific to any namespace so makeKeymap does not need the extra indirection.
Resolving these conflicts is not an easy task, even for people familiar with C++, which probably is why there is no real solution to the problem (until now!).
Let me know how you go, and I'll help you through it if you get stuck.
DouglasBair
Take a blank Arduino sketch, include and . try to compile. I receive this error: "C:\Program Files (x86)\Arduino\libraries\GSM\src/GSM3MobileAccessProvider.h:37:36: note: previous declaration 'GSM3_NetworkStatus_t IDLE' enum GSM3_NetworkStatus_t { ERROR, IDLE, CONNECTING, GSM_READY, GPRS_READY, TRANSPARENT_CONNECTED, OFF};"
Researching this, I see many years of users asking that this be fixed, and not expecting to be told fix it yourself by editing a library. What then for the next release, fix it again? I've already check the forum and the issues. Many 'smart alecs' say its fixed, where? This happens in v1.6.5, 1.6.6, and in (.org's) 1.7.9.
When will Arduino.cc fix this?
And, yes, I'm a newbie. So if I'm expecting too much or have entered the lib names incorrectly, let me know. Doug