This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [PATCH] Be lazy about refreshing the windows in tui_show_frame_info (PR tui/13378)
- From: Pedro Alves <palves at redhat dot com>
- To: Patrick Palka <patrick at parcs dot ath dot cx>, gdb-patches at sourceware dot org
- Date: Tue, 30 Jun 2015 15:27:31 +0100
- Subject: Re: [PATCH] Be lazy about refreshing the windows in tui_show_frame_info (PR tui/13378)
- Authentication-results: sourceware.org; auth=none
- References: <1435372525-1374-2-git-send-email-patrick at parcs dot ath dot cx> <1435631532-32504-1-git-send-email-patrick at parcs dot ath dot cx>
On 06/30/2015 03:32 AM, Patrick Palka wrote:
> This revised patch makes sure that tui_set_locator_info returns 1 when the
> locator is first constructed, just in case none of the later checks trigger
> for some reason.
I have a couple questions below, but I'm fine with this approach.
> @@ -302,21 +306,36 @@ tui_set_locator_info (struct gdbarch *gdbarch,
> {
> struct tui_gen_win_info *locator = tui_locator_win_info_ptr ();
> struct tui_locator_element *element;
> + int locator_changed_p = 0;
>
> /* Allocate the locator content if necessary. */
> if (locator->content_size <= 0)
> {
> locator->content = tui_alloc_content (1, LOCATOR_WIN);
> locator->content_size = 1;
> + locator_changed_p = 1;
> }
>
> element = &locator->content[0]->which_element.locator;
> +
> + if (procname != NULL)
> + locator_changed_p |= strncmp (element->proc_name, procname,
> + MAX_LOCATOR_ELEMENT_LEN) != 0;
Can't element->proc_name be NULL here?
For the string fields, do we also need to compare
whether we go from NULL <-> non-NULL ?
locator_changed_p |= ((fullname == NULL) != (element->full_name == NULL));
etc.?
Thanks,
Pedro Alves