ser-tcp.c: Small signed->unsigned cleanup.
authorPedro Alves <palves@redhat.com>
Tue, 26 Mar 2013 20:29:47 +0000 (20:29 +0000)
committerPedro Alves <palves@redhat.com>
Tue, 26 Mar 2013 20:29:47 +0000 (20:29 +0000)
commit2c619be216c9fb729c2696fa10957732290b041a
tree2a32eabaa3134e38ff8499bca0fdb3f1c3c8f6cf
parent1b493192488ccb9886ffe709864e5f5c0588b004
ser-tcp.c: Small signed->unsigned cleanup.

The "set tcp connect-timeout" variable is unsigned:

  /* Timeout period for connections, in seconds.  */

  static unsigned int tcp_retry_limit = 15;

And used like:

  /* Check for timeout.  */
  if (*polls > tcp_retry_limit * POLL_INTERVAL)
    {
      errno = ETIMEDOUT;
      return -1;
    }

Which made me stop and look over why is it that 'polls' is signed.
What I found is there's really no reason.

gdb/
2013-03-26  Pedro Alves  <palves@redhat.com>

* ser-tcp.c (wait_for_connect): Make 'polls' parameter unsigned.
(net_open): Make 'polls' local unsigned.
gdb/ChangeLog
gdb/ser-tcp.c