EEPROM variable declared static instead of extern

[jamessynge]

warning: 'EEPROM' defined but not used [-Wunused-variable]

This warning occurs because EEPROM.h declares the EEPROM variable as follows:

static EEPROMClass EEPROM;

This means that the EEPROM variable is local (not exported) from every file that includes EEPROM.h.

I'm developing a program using the Arduino framework that has lots of source files. I do unit testing on a linux host, so I find it convenient to have a platform.h header file that includes either the Arduino framework header files that are used by the source file, or my minimal host versions of those Arduino files for execution on host. Among those files is EEPROM.h, so for every .cpp files that includes my platform.h file but doesn't actually use EEPROM I see the warning above. The fix is quite simple, and I'm happy to provide a PR if the team is interested.

[jamessynge]

I went ahead and created a PR. FWIW, I found discussions of this issue as far back as 2015. I realize it is "just" a warning, but it can make it hard to see the real errors if there are lots of warnings too.