[Ada] Style cleanups related to Current_Condition
authorPiotr Trojanek <trojanek@adacore.com>
Fri, 15 May 2020 18:40:03 +0000 (20:40 +0200)
committerPierre-Marie de Rodat <derodat@adacore.com>
Wed, 8 Jul 2020 14:55:49 +0000 (10:55 -0400)
gcc/ada/

* einfo.ads (Current_Value): Fix typo in comment.
* exp_ch2.adb (Expand_Current_Value): Remove unnecessary "Start
of processing ..." comment.
* exp_util.adb (Set_Entity_Current_Value): Fix unbalanced paren
in comment.
(Get_Current_Value_Condition): Fix layout in comment.
* sem_ch5.adb (Analyze_Cond_Then): Replace commented condition
with pragma Assert.

gcc/ada/einfo.ads
gcc/ada/exp_ch2.adb
gcc/ada/exp_util.adb
gcc/ada/sem_ch5.adb

index 7138b9d59de4e5172f53d4eb5b24924e717f9511..7bb1f16d98de13fec758e3510a406cd04bab038d 100644 (file)
@@ -822,7 +822,7 @@ package Einfo is
 --       Defined in all object entities. Set in E_Variable, E_Constant, formal
 --       parameters and E_Loop_Parameter entities if we have trackable current
 --       values. Set non-Empty if the (constant) current value of the variable
---       is known, This value is valid only for references from the same
+--       is known. This value is valid only for references from the same
 --       sequential scope as the entity. The sequential scope of an entity
 --       includes the immediate scope and any contained scopes that are package
 --       specs, package bodies, blocks (at any nesting level) or statement
index b8ac2cde2645604e9fcda558a0d57c9e4118a3cb..407ffcb434ccd1d6c6df392b6ab23ca7315b29c1 100644 (file)
@@ -122,8 +122,6 @@ package body Exp_Ch2 is
       Val : Node_Id;
       Op  : Node_Kind;
 
-   --  Start of processing for Expand_Current_Value
-
    begin
       if True
 
index d93788b8e5ba884183293a5976ac3f84ca338ef7..236d9ce6158bf6e877bfe331d33cdbffa1cdba88 100644 (file)
@@ -6418,7 +6418,7 @@ package body Exp_Util is
             if Loc < Sloc (CV) then
                return;
 
-               --  After end of IF statement
+            --  After end of IF statement
 
             elsif Loc >= Sloc (CV) + Text_Ptr (UI_To_Int (End_Span (CV))) then
                return;
@@ -12741,7 +12741,7 @@ package body Exp_Util is
                --  they occur at the same level. If the second one is nested,
                --  then the decision is neither right nor wrong (it would be
                --  equally OK to leave the outer one in place, or take the new
-               --  inner one. Really we should record both, but our data
+               --  inner one). Really we should record both, but our data
                --  structures are not that elaborate.
 
                if Nkind (Current_Value (Ent)) not in N_Subexpr then
index 4028d0232ab9b29fb19c8c1c7ef32cf6a4bb5504..4664df83aba3b53b22703a31910edd6a5acf8c02 100644 (file)
@@ -1833,7 +1833,7 @@ package body Sem_Ch5 is
 
             --  If condition is False, analyze THEN with expansion off
 
-            else -- Is_False (Expr_Value (Cond))
+            else pragma Assert (Is_False (Expr_Value (Cond)));
                Expander_Mode_Save_And_Set (False);
                In_Deleted_Code := True;
                Analyze_Statements (Tstm);