Conflicting type definition depending on include order

[dok-net]

This is in release 2.6.2.
For some reason that shouldn't matter here, if Arduino.h gets (indirectly) included in Schedule.h, this redefinition bug is raised by the compiler:

cores\esp8266\sntp-lwip2.cpp:

typedef struct __tzrule_struct
{
  char ch;
  int m;
  int n;
  int d;
  int s;
  time_t change;
  int offset;
} __tzrule_type;

tools\sdk\libc\xtensa-lx106-elf\include\time.h:

typedef struct __tzrule_struct
{
  char ch;
  int m;
  int n;
  int d;
  int s;
  time_t change;
  long offset; /* Match type of _timezone. */
} __tzrule_type;

@d-a-v I was asked by @devyte to refer this one to you.

[d-a-v]

I suggest removing the struct(s) from sntp-lwip2 and use instead

#include <time.h>

(in case of issue, try with)

#include <../include/time.h>

(I can of course try and deal with it because in the core we shouldn't have that kind of redeclaration)

[dok-net]

@d-a-v This changes #include <sys/time.h> to #include <time.h>...