[binutils-gdb] gdb: fix build errors in gdbsupport/thread-pool.h used with old gcc

[binutils-gdb] gdb: fix build errors in gdbsupport/thread-pool.h used with old gcc

[Tiezhu Yang]

https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;h=11d7dd3357997616427e22969ea8a929f5db59c9

commit 11d7dd3357997616427e22969ea8a929f5db59c9
Author: Tiezhu Yang <yangtiezhu@loongson.cn>
Date:   Thu Apr 14 10:37:30 2022 +0800

    gdb: fix build errors in gdbsupport/thread-pool.h used with old gcc
    
    When I build gdb with gcc 8.3, there exist the following build errors,
    rename the typedef to task_t to fix them.
    
      CXX      thread-pool.o
    In file included from /home/loongson/gdb.git/gdbsupport/thread-pool.cc:21:
    /home/loongson/gdb.git/gdbsupport/../gdbsupport/thread-pool.h: In member function ‘std::future<void> gdb::thread_pool::post_task(std::function<void()>&&)’:
    /home/loongson/gdb.git/gdbsupport/../gdbsupport/thread-pool.h:69:44: error: declaration of ‘task’ shadows a previous local [-Werror=shadow=local]
         std::packaged_task<void ()> task (std::move (func));
                                                ^~~~
    /home/loongson/gdb.git/gdbsupport/../gdbsupport/thread-pool.h:102:39: note: shadowed declaration is here
       typedef std::packaged_task<void ()> task;
                                           ^~~~
    /home/loongson/gdb.git/gdbsupport/../gdbsupport/thread-pool.h: In member function ‘std::future<_Res> gdb::thread_pool::post_task(std::function<T()>&&)’:
    /home/loongson/gdb.git/gdbsupport/../gdbsupport/thread-pool.h:80:41: error: declaration of ‘task’ shadows a previous local [-Werror=shadow=local]
         std::packaged_task<T ()> task (std::move (func));
                                             ^~~~
    /home/loongson/gdb.git/gdbsupport/../gdbsupport/thread-pool.h:102:39: note: shadowed declaration is here
       typedef std::packaged_task<void ()> task;
                                           ^~~~
    
    Signed-off-by: Tiezhu Yang <yangtiezhu@loongson.cn>

Diff:
---
 gdbsupport/thread-pool.cc | 2 +-
 gdbsupport/thread-pool.h  | 4 ++--
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/gdbsupport/thread-pool.cc b/gdbsupport/thread-pool.cc
index efd8b799713..464e32fe79e 100644
--- a/gdbsupport/thread-pool.cc
+++ b/gdbsupport/thread-pool.cc
@@ -171,7 +171,7 @@ thread_pool::thread_function ()
 
   while (true)
     {
-      optional<task> t;
+      optional<task_t> t;
 
       {
 	/* We want to hold the lock while examining the task list, but
diff --git a/gdbsupport/thread-pool.h b/gdbsupport/thread-pool.h
index 3243346771f..5e203fd896c 100644
--- a/gdbsupport/thread-pool.h
+++ b/gdbsupport/thread-pool.h
@@ -99,13 +99,13 @@ private:
   size_t m_thread_count = 0;
 
   /* A convenience typedef for the type of a task.  */
-  typedef std::packaged_task<void ()> task;
+  typedef std::packaged_task<void ()> task_t;
 
   /* The tasks that have not been processed yet.  An optional is used
      to represent a task.  If the optional is empty, then this means
      that the receiving thread should terminate.  If the optional is
      non-empty, then it is an actual task to evaluate.  */
-  std::queue<optional<task>> m_tasks;
+  std::queue<optional<task_t>> m_tasks;
 
   /* A condition variable and mutex that are used for communication
      between the main thread and the worker threads.  */