In lwip_netconn_do_connect() it call setup_tcp() before tcp_connect(), but in lwip_netconn_do_connected() it calls setup_tcp() again for TCP, this seems unnecesary, or did I miss something?

lwip_netconn_do_connect(void *m)
{
  ...
  ...
      case NETCONN_TCP:
          ...
          setup_tcp(msg->conn); <---- first calls setup_tcp here!
          err = tcp_connect(msg->conn->pcb.tcp, API_EXPR_REF(msg->msg.bc.ipaddr),
                            msg->msg.bc.port, lwip_netconn_do_connected);

  ...
}

lwip_netconn_do_connected(void *arg, struct tcp_pcb *pcb, err_t err)
{
  ...
  if ((NETCONNTYPE_GROUP(conn->type) == NETCONN_TCP) && (err == ERR_OK)) {
    setup_tcp(conn);  <---- on successfully connected, setup_tcp is called again, is necessary?
  }
  ...
}