Merge branch 'topic/vadim/fixes' into 'master'

Fix documentation extraction for subprogram as compilation unit

Closes ada_language_server#1613

See merge request eng/ide/gnatdoc!162

Author: godunko

source/gnatdoc-comments-extractor-trailing.adb

@@ -945,7 +945,7 @@ package body GNATdoc.Comments.Extractor.Trailing is
          end loop;
 
          if State /= None then
-            raise Program_Error;
+            Apply;
          end if;
       end;
    end Process;

testsuite/extractor/subprograms_gnat/monitor_loop.ads

@@ -0,0 +1,19 @@
+--
+--  Copyright (C) 2018-2021, AdaCore
+--
+--  SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
+--
+
+procedure Monitor_Loop (Timeout : Duration);
+--  Drive process control engine, deliver events, watch child processes, etc.
+--
+--  Timeout to run. Don't wait if zero. If there are some events then procedure
+--  will return before timeout expires.
+--
+--  Note: This procedure is NOT thread-safe! In multi-tasking environment
+--  create a dedicated task to drive the engine.
+--
+--  In single task application you should call this to drive process engine.
+--
+--  In Glib enabled application the engine is integrated it the Glib event
+--  loop and this procedure shouldn't be used.

testsuite/extractor/subprograms_gnat/test.out

@@ -562,3 +562,40 @@ Documentation of the subprogram.
 <EndName subprograms_gnat.ads:263:5-263:21>
 <Id "Subprograms_GNAT" subprograms_gnat.ads:263:5-263:21>
 <PragmaNodeList subprograms_gnat.ads:263:22-263:22>
+<CompilationUnit monitor_loop.ads:7:1-7:45>
+<AdaNodeList monitor_loop.ads:7:1-7:1>
+<LibraryItem monitor_loop.ads:7:1-7:45>
+<PrivateAbsent monitor_loop.ads:7:1-7:1>
+<SubpDecl ["Monitor_Loop"] monitor_loop.ads:7:1-7:45>
+**************************
+\/ RAW <<CALLABLE>> ()
+Drive process control engine, deliver events, watch child processes, etc.
+
+Timeout to run. Don't wait if zero. If there are some events then procedure
+will return before timeout expires.
+
+Note: This procedure is NOT thread-safe! In multi-tasking environment
+create a dedicated task to drive the engine.
+
+In single task application you should call this to drive process engine.
+
+In Glib enabled application the engine is integrated it the Glib event
+loop and this procedure shouldn't be used.
+\/ PARAMETER timeout (Timeout)
+\/ SNIPPET ada ()
+procedure Monitor_Loop (Timeout : Duration)
+\/ DESCRIPTION  ()
+Drive process control engine, deliver events, watch child processes, etc.
+
+Timeout to run. Don't wait if zero. If there are some events then procedure
+will return before timeout expires.
+
+Note: This procedure is NOT thread-safe! In multi-tasking environment
+create a dedicated task to drive the engine.
+
+In single task application you should call this to drive process engine.
+
+In Glib enabled application the engine is integrated it the Glib event
+loop and this procedure shouldn't be used.
+**************************
+<PragmaNodeList monitor_loop.ads:7:45-7:45>

testsuite/extractor/subprograms_gnat/test.yaml

@@ -1 +1,4 @@
-extractor_source: subprograms_gnat.ads
+description: Check documentation extraction for subprogram declarations.
+extractor_sources:
+  - subprograms_gnat.ads
+  - monitor_loop.ads