Skip to content

Commit

Permalink
target/breakpoints: make breakpoint_watchpoint_remove_all static
Browse files Browse the repository at this point in the history
Fixes below Sparse tool warning.
warning: symbol 'breakpoint_watchpoint_remove_all' was not declared.
Should it be static?

Signed-off-by: Erhan Kurubas <erhan.kurubas@espressif.com>
Change-Id: I4af1d5aa54abcb45f746b877513ba0b5fccbeb47
Reviewed-on: https://review.openocd.org/c/openocd/+/7955
Reviewed-by: Tomas Vanek <vanekt@fbl.cz>
Reviewed-by: Antonio Borneo <borneo.antonio@gmail.com>
Tested-by: jenkins
Reviewed-by: Jan Matyas <jan.matyas@codasip.com>
Reviewed-by: Marek Vrbka <marek.vrbka@codasip.com>
  • Loading branch information
erhankur authored and borneoa committed Nov 11, 2023
1 parent 0f26118 commit a908008
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion src/target/breakpoints.c
Original file line number Diff line number Diff line change
Expand Up @@ -425,7 +425,7 @@ static int watchpoint_remove_all_internal(struct target *target)
return retval;
}

int breakpoint_watchpoint_remove_all(struct target *target, enum breakpoint_watchpoint bp_wp)
static int breakpoint_watchpoint_remove_all(struct target *target, enum breakpoint_watchpoint bp_wp)
{
assert(bp_wp == BREAKPOINT || bp_wp == WATCHPOINT);
int retval = ERROR_OK;
Expand Down

0 comments on commit a908008

Please sign in to comment.