Replaced struct ip_addr by typedef ip_addr_t to make changing the actual implementation behind the typedef easier.

src/netif/ppp/ppp.c

@@ -794,7 +794,7 @@ pppifOutputOverEthernet(int pd, struct pbuf *p)
 
 /* Send a packet on the given connection. */
 static err_t
-pppifOutput(struct netif *netif, struct pbuf *pb, struct ip_addr *ipaddr)
+pppifOutput(struct netif *netif, struct pbuf *pb, ip_addr_t *ipaddr)
 {
   int pd = (int)netif->state;
   PPPControl *pc = &pppControl[pd];

src/netif/ppp/ppp.h

@@ -336,7 +336,7 @@ struct ppp_settings {
 };
 
 struct ppp_addrs {
-  struct ip_addr our_ipaddr, his_ipaddr, netmask, dns1, dns2;
+  ip_addr_t our_ipaddr, his_ipaddr, netmask, dns1, dns2;
 };
 
 /*****************************

src/netif/ppp/ppp_oe.c

@@ -386,7 +386,7 @@ pppoe_dispatch_disc_pkt(struct netif *netif, struct pbuf *pb)
   char devname[6];
   char *error;
   u8_t *ac_cookie;
-  size_t ac_cookie_len;
+  u16_t ac_cookie_len;
 #ifdef PPPOE_SERVER
   u8_t *hunique;
   size_t hunique_len;
@@ -549,9 +549,9 @@ breakbreak:;
       sc->sc_state = PPPOE_STATE_PADO_SENT;
       pppoe_send_pado(sc);
       break;
-  #endif /* PPPOE_SERVER */
+#endif /* PPPOE_SERVER */
     case PPPOE_CODE_PADR:
-  #ifdef PPPOE_SERVER
+#ifdef PPPOE_SERVER
       /*
        * get sc from ac_cookie if IFF_PASSIVE
        */
@@ -587,10 +587,10 @@ breakbreak:;
       sc->sc_state = PPPOE_STATE_SESSION;
       pppoe_linkstatus_up(sc); /* notify upper layers */
       break;
-  #else
+#else
       /* ignore, we are no access concentrator */
       goto done;
-  #endif /* PPPOE_SERVER */
+#endif /* PPPOE_SERVER */
     case PPPOE_CODE_PADO:
       if (sc == NULL) {
         /* be quiet if there is not a single pppoe instance */