This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [PATCH v2] Associate target_ops with target_fileio file descriptors
- From: Pedro Alves <palves at redhat dot com>
- To: Gary Benson <gbenson at redhat dot com>, gdb-patches at sourceware dot org
- Date: Tue, 24 Mar 2015 13:22:39 +0000
- Subject: Re: [PATCH v2] Associate target_ops with target_fileio file descriptors
- Authentication-results: sourceware.org; auth=none
- References: <20150318184538 dot GA32134 at blade dot nx> <1426867612-32278-1-git-send-email-gbenson at redhat dot com>
On 03/20/2015 04:06 PM, Gary Benson wrote:
> +
> + /* Search for closed handles to reuse. */
> + if (lowest_closed_fd >= 0)
> + {
> + for (index = lowest_closed_fd;
> + VEC_iterate (fileio_fh_t, fileio_fhandles, index, fh);
> + index++)
> + if (is_closed_fileio_fh (fh->fd))
> + break;
You don't really need 'index'. If this walks using
lowest_closed_fd instead, like:
for (;
VEC_iterate (fileio_fh_t, fileio_fhandles,
lowest_closed_fd, fh);
lowest_closed_fd++)
if (is_closed_fileio_fh (fh->fd))
break;
Then the next time around we skip slots we already know are
open.
> + }
> +
> + /* Push a new handle if no closed handles were found. */
> + if (index == total)
> + {
> + fh = VEC_safe_push (fileio_fh_t, fileio_fhandles, NULL);
> + lowest_closed_fd = -1;
> + }
> +
> + /* Fill in the handle and return its index. */
> + fh->t = t;
> + fh->fd = fd;
> +
> + return index;
> +}
> +
Otherwise looks good. Please push.
Thanks,
Pedro Alves