]> git.ipfire.org Git - thirdparty/gcc.git/commitdiff
[multiple changes]
authorArnaud Charlet <charlet@gcc.gnu.org>
Wed, 31 Aug 2011 09:24:32 +0000 (11:24 +0200)
committerArnaud Charlet <charlet@gcc.gnu.org>
Wed, 31 Aug 2011 09:24:32 +0000 (11:24 +0200)
2011-08-31  Robert Dewar  <dewar@adacore.com>

* exp_ch5.adb, exp_ch7.ads, sem_ch5.adb, put_scos.adb, s-rannum.adb,
a-rbtgbo.adb, exp_intr.adb, a-cbdlli.adb, a-cbdlli.ads: Minor
reformatting.

2011-08-31  Hristian Kirtchev  <kirtchev@adacore.com>

* exp_util.adb (Find_Protection_Type): Do not look for fields _object
if the corresponding type is malformed due to restriction violations.

2011-08-31  Robert Dewar  <dewar@adacore.com>

* s-ransee.ads, s-ransee.adb: Minor reformatting.

2011-08-31  Hristian Kirtchev  <kirtchev@adacore.com>

* exp_ch4.adb (Expand_N_Allocator): Correct faulty condition which
would cause the generation of Set_Finalize_Address if the target is a
VM and the designated type is not derived from [Limited_]Controlled.

2011-08-31  Arnaud Charlet  <charlet@adacore.com>

* comperr.adb, comperr.ads, gnat1drv.adb (Delete_SCIL_Files): New
subprogram.
(Compiler_Abort, Gnat1drv): Call Delete_SCIL_Files in codepeer mode in
case of a compilation error.

2011-08-31  Hristian Kirtchev  <kirtchev@adacore.com>

* init.c (__gnat_error_handler): Standardize the stack overflow or
erroneous memory access message.
* seh_init.c (__gnat_SEH_error_handler): Standardize the stack overflow
or erroneous memory access message.

From-SVN: r178368

19 files changed:
gcc/ada/ChangeLog
gcc/ada/a-cbdlli.adb
gcc/ada/a-cbdlli.ads
gcc/ada/a-rbtgbo.adb
gcc/ada/comperr.adb
gcc/ada/comperr.ads
gcc/ada/exp_ch4.adb
gcc/ada/exp_ch5.adb
gcc/ada/exp_ch7.ads
gcc/ada/exp_intr.adb
gcc/ada/exp_util.adb
gcc/ada/gnat1drv.adb
gcc/ada/init.c
gcc/ada/put_scos.adb
gcc/ada/s-rannum.adb
gcc/ada/s-ransee.adb
gcc/ada/s-ransee.ads
gcc/ada/seh_init.c
gcc/ada/sem_ch5.adb

index 24abfaed4dbfec1377226968ebb6d57e6bdbff3f..3240bcdfda5284ce3e817a8216ef8e7f8871a4cc 100644 (file)
@@ -1,3 +1,38 @@
+2011-08-31  Robert Dewar  <dewar@adacore.com>
+
+       * exp_ch5.adb, exp_ch7.ads, sem_ch5.adb, put_scos.adb, s-rannum.adb,
+       a-rbtgbo.adb, exp_intr.adb, a-cbdlli.adb, a-cbdlli.ads: Minor
+       reformatting.
+
+2011-08-31  Hristian Kirtchev  <kirtchev@adacore.com>
+
+       * exp_util.adb (Find_Protection_Type): Do not look for fields _object
+       if the corresponding type is malformed due to restriction violations.
+
+2011-08-31  Robert Dewar  <dewar@adacore.com>
+
+       * s-ransee.ads, s-ransee.adb: Minor reformatting.
+
+2011-08-31  Hristian Kirtchev  <kirtchev@adacore.com>
+
+       * exp_ch4.adb (Expand_N_Allocator): Correct faulty condition which
+       would cause the generation of Set_Finalize_Address if the target is a
+       VM and the designated type is not derived from [Limited_]Controlled.
+
+2011-08-31  Arnaud Charlet  <charlet@adacore.com>
+
+       * comperr.adb, comperr.ads, gnat1drv.adb (Delete_SCIL_Files): New
+       subprogram.
+       (Compiler_Abort, Gnat1drv): Call Delete_SCIL_Files in codepeer mode in
+       case of a compilation error.
+
+2011-08-31  Hristian Kirtchev  <kirtchev@adacore.com>
+
+       * init.c (__gnat_error_handler): Standardize the stack overflow or
+       erroneous memory access message.
+       * seh_init.c (__gnat_SEH_error_handler): Standardize the stack overflow
+       or erroneous memory access message.
+
 2011-08-31  Robert Dewar  <dewar@adacore.com>
 
        * sem_ch4.adb: Minor reformatting.
index cf2422748d203425c63d3304ad696a564a5b4a2b..a8a7c5eafbc94c4721137cd50d915dcdf84182c7 100644 (file)
@@ -1046,6 +1046,7 @@ package body Ada.Containers.Bounded_Doubly_Linked_Lists is
             Process (Cursor'(Container'Unrestricted_Access, Node));
             Node := Container.Nodes (Node).Next;
          end loop;
+
       exception
          when others =>
             B := B - 1;
@@ -1055,8 +1056,9 @@ package body Ada.Containers.Bounded_Doubly_Linked_Lists is
       B := B - 1;
    end Iterate;
 
-   function Iterate (Container : List)
-     return List_Iterator_Interfaces.Reversible_Iterator'class
+   function Iterate
+     (Container : List)
+      return List_Iterator_Interfaces.Reversible_Iterator'class
    is
    begin
       if Container.Length = 0 then
@@ -1066,8 +1068,10 @@ package body Ada.Containers.Bounded_Doubly_Linked_Lists is
       end if;
    end Iterate;
 
-   function Iterate (Container : List; Start : Cursor)
-     return List_Iterator_Interfaces.Reversible_Iterator'class
+   function Iterate
+     (Container : List;
+      Start     : Cursor)
+      return List_Iterator_Interfaces.Reversible_Iterator'class
    is
       It : constant Iterator := (Container'Unrestricted_Access, Start.Node);
    begin
index 32e992fa60dbac0290b3c4ddfa333a135a121b30..0443c304a8a829733123dd1483a252854b072fdc 100644 (file)
@@ -44,8 +44,7 @@ package Ada.Containers.Bounded_Doubly_Linked_Lists is
    pragma Pure;
    pragma Remote_Types;
 
-   type List (Capacity : Count_Type) is tagged private
-   with
+   type List (Capacity : Count_Type) is tagged private with
       Constant_Indexing => Constant_Reference,
       Variable_Indexing => Reference,
       Default_Iterator  => Iterate,
@@ -59,6 +58,7 @@ package Ada.Containers.Bounded_Doubly_Linked_Lists is
    Empty_List : constant List;
 
    No_Element : constant Cursor;
+
    function Has_Element (Position : Cursor) return Boolean;
 
    package List_Iterator_Interfaces is new
@@ -140,10 +140,13 @@ package Ada.Containers.Bounded_Doubly_Linked_Lists is
 
    procedure Reverse_Elements (Container : in out List);
 
-   function Iterate (Container : List)
+   function Iterate
+     (Container : List)
       return List_Iterator_Interfaces.Reversible_Iterator'class;
 
-   function Iterate (Container : List; Start : Cursor)
+   function Iterate
+     (Container : List;
+      Start     : Cursor)
       return List_Iterator_Interfaces.Reversible_Iterator'class;
 
    procedure Swap
index afa98f89c033cfabf95b937055c558e61e43fac8..f420438efac6808eaef5286b6ce898d2f419154e 100644 (file)
@@ -63,8 +63,11 @@ package body Ada.Containers.Red_Black_Trees.Generic_Bounded_Operations is
       Tree.Last   := 0;
       Tree.Root   := 0;
       Tree.Length := 0;
+
+      --  Why are the following commented out with no explanation ???
       --  Tree.Busy
       --  Tree.Lock
+
       Tree.Free   := -1;
    end Clear_Tree;
 
@@ -76,7 +79,6 @@ package body Ada.Containers.Red_Black_Trees.Generic_Bounded_Operations is
      (Tree : in out Tree_Type'Class;
       Node : Count_Type)
    is
-
       --  CLR p. 274
 
       X : Count_Type;
@@ -143,7 +145,7 @@ package body Ada.Containers.Red_Black_Trees.Generic_Bounded_Operations is
             end if;
 
             if (Left (N (W))  = 0 or else Color (N (Left (N (W)))) = Black)
-                  and then
+                 and then
                (Right (N (W)) = 0 or else Color (N (Right (N (W)))) = Black)
             then
                Set_Color (N (W), Red);
index da6c8a688ed31f596f47411ee92caf1822f8a087..676995fef1c6d2c230dddf8bcb6cc3f0d2c2e2a2 100644 (file)
@@ -6,7 +6,7 @@
 --                                                                          --
 --                                 B o d y                                  --
 --                                                                          --
---          Copyright (C) 1992-2009, Free Software Foundation, Inc.         --
+--          Copyright (C) 1992-2011, Free Software Foundation, Inc.         --
 --                                                                          --
 -- GNAT is free software;  you can  redistribute it  and/or modify it under --
 -- terms of the  GNU General Public License as published  by the Free Soft- --
 --  error is detected. Calls to these routines cause termination of the
 --  current compilation with appropriate error output.
 
-with Atree;    use Atree;
-with Debug;    use Debug;
-with Errout;   use Errout;
-with Gnatvsn;  use Gnatvsn;
-with Namet;    use Namet;
-with Opt;      use Opt;
-with Osint;    use Osint;
-with Output;   use Output;
-with Sinput;   use Sinput;
-with Sprint;   use Sprint;
-with Sdefault; use Sdefault;
-with Targparm; use Targparm;
-with Treepr;   use Treepr;
-with Types;    use Types;
+with Atree;         use Atree;
+with Debug;         use Debug;
+with Errout;        use Errout;
+with Gnatvsn;       use Gnatvsn;
+with Lib;           use Lib;
+with Namet;         use Namet;
+with Opt;           use Opt;
+with Osint;         use Osint;
+with Output;        use Output;
+with Sinfo;         use Sinfo;
+with Sinput;        use Sinput;
+with Sprint;        use Sprint;
+with Sdefault;      use Sdefault;
+with System.OS_Lib; use System.OS_Lib;
+with Targparm;      use Targparm;
+with Treepr;        use Treepr;
+with Types;         use Types;
 
 with Ada.Exceptions; use Ada.Exceptions;
 
@@ -144,6 +147,10 @@ package body Comperr is
          end if;
       end if;
 
+      if CodePeer_Mode then
+         Delete_SCIL_Files;
+      end if;
+
       --  If any errors have already occurred, then we guess that the abort
       --  may well be caused by previous errors, and we don't make too much
       --  fuss about it, since we want to let programmer fix the errors first.
@@ -422,9 +429,40 @@ package body Comperr is
          Source_Dump;
          raise Unrecoverable_Error;
       end if;
-
    end Compiler_Abort;
 
+   -----------------------
+   -- Delete_SCIL_Files --
+   -----------------------
+
+   procedure Delete_SCIL_Files is
+      Main    : Node_Id;
+      Success : Boolean;
+      pragma Unreferenced (Success);
+   begin
+      --  If parsing was not successful, no Main_Unit is available, so return
+      --  immediately.
+
+      if Main_Source_File = No_Source_File then
+         return;
+      end if;
+
+      --  Retrieve unit name, and remove old versions of SCIL/<unit>.scil and
+      --  SCIL/<unit>__body.scil
+
+      Main := Unit (Cunit (Main_Unit));
+
+      if Nkind (Main) = N_Subprogram_Body then
+         Get_Name_String (Chars (Defining_Unit_Name (Specification (Main))));
+      else
+         Get_Name_String (Chars (Defining_Unit_Name (Main)));
+      end if;
+
+      Delete_File ("SCIL/" & Name_Buffer (1 .. Name_Len) & ".scil", Success);
+      Delete_File
+        ("SCIL/" & Name_Buffer (1 .. Name_Len) & "__body.scil", Success);
+   end Delete_SCIL_Files;
+
    -----------------
    -- Repeat_Char --
    -----------------
index 04a60621897cd2fc2e9ade46a1285ad861c5d597..a45faf16245f0cffa7c01ba2586b512fd6b8ad99 100644 (file)
@@ -6,7 +6,7 @@
 --                                                                          --
 --                                 S p e c                                  --
 --                                                                          --
---          Copyright (C) 1992-2007, Free Software Foundation, Inc.         --
+--          Copyright (C) 1992-2011, Free Software Foundation, Inc.         --
 --                                                                          --
 -- GNAT is free software;  you can  redistribute it  and/or modify it under --
 -- terms of the  GNU General Public License as published  by the Free Soft- --
@@ -50,6 +50,9 @@ package Comperr is
    --  end exception (with possible message stored in TSD.Current_Excep,
    --  and negative (an unused value) for a GCC abort.
 
+   procedure Delete_SCIL_Files;
+   --  Delete SCIL files associated with the main unit
+
    ------------------------------
    -- Use of gnat_bug.box File --
    ------------------------------
index 1a1159b2a19826f752ebc71aee9fee1926f72e23..ab966963a693884dda21595f7ffdae0dc9f74b9d 100644 (file)
@@ -3949,13 +3949,13 @@ package body Exp_Ch4 is
                      --  Types derived from [Limited_]Controlled are the only
                      --  ones considered since they have fields Prev and Next.
 
-                     if VM_Target /= No_VM
-                       and then Is_Controlled (T)
-                     then
-                        Insert_Action (N,
-                          Make_Attach_Call
-                            (Obj_Ref => New_Copy_Tree (Init_Arg1),
-                             Ptr_Typ => PtrT));
+                     if VM_Target /= No_VM then
+                        if Is_Controlled (T) then
+                           Insert_Action (N,
+                             Make_Attach_Call
+                               (Obj_Ref => New_Copy_Tree (Init_Arg1),
+                                Ptr_Typ => PtrT));
+                        end if;
 
                      --  Default case, generate:
 
index 5203885712d23c8be816011d212f5366d65543f8..0f184552a95d169d7f4dfec4e5ebfebbe843b8ef 100644 (file)
@@ -3249,7 +3249,7 @@ package body Exp_Ch5 is
                    Subtype_Mark  => New_Occurrence_Of (Iter_Type, Loc),
                    Name          => Relocate_Node (Name (I_Spec)));
 
-               --  Create declaration for cursor.
+               --  Create declaration for cursor
 
                Decl2 :=
                  Make_Object_Declaration (Loc,
index 8a0be81bac22e3ef14b84836e3c53fb3f37409e3..8ea71916e26bae5d833f0a9f09ebf5d11bbd380e 100644 (file)
@@ -41,33 +41,34 @@ package Exp_Ch7 is
    --  that take care of finalization management at run-time.
 
    --  Support of exceptions from user finalization procedures
-   --
+
    --  There is a specific mechanism to handle these exceptions, continue
-   --  finalization and then raise PE.
-   --  This mechanism is used by this package but also by exp_intr for
-   --  Ada.Unchecked_Deallocation.
+   --  finalization and then raise PE. This mechanism is used by this package
+   --  but also by exp_intr for Ada.Unchecked_Deallocation.
+
    --  There are 3 subprograms to use this mechanism, and the type
    --  Finalization_Exception_Data carries internal data between these
    --  subprograms:
    --
-   --  1. Build_Object_Declaration: create the variables for the next two
-   --  subprograms.
-   --  2. Build_Exception_Handler: create the exception handler for a call to
-   --  a user finalization procedure.
-   --  3. Build_Raise_Stmt: create the code to potentially raise a PE exception
-   --  if am exception was raise in a user finalization procedure.
+   --    1. Build_Object_Declaration: create the variables for the next two
+   --       subprograms.
+   --    2. Build_Exception_Handler: create the exception handler for a call
+   --       to a user finalization procedure.
+   --    3. Build_Raise_Stmt: create code to potentially raise a PE exception
+   --       if an exception was raise in a user finalization procedure.
+
    type Finalization_Exception_Data is record
-      Loc         : Source_Ptr;
+      Loc : Source_Ptr;
       --  Sloc for the added nodes
 
-      Abort_Id    : Entity_Id;
+      Abort_Id : Entity_Id;
       --  Boolean variable set to true if the finalization was triggered by
       --  an abort.
 
-      E_Id        : Entity_Id;
+      E_Id : Entity_Id;
       --  Variable containing the exception occurrence raised by user code
 
-      Raised_Id   : Entity_Id;
+      Raised_Id : Entity_Id;
       --  Boolean variable set to true if an exception was raised in user code
    end record;
 
index 163258293142eb53391deaf0fc2f350c63d253fc..2d47846747461be34dff78311945f3dd5d2ff777 100644 (file)
@@ -964,19 +964,15 @@ package body Exp_Intr is
            Make_Block_Statement (Loc,
              Handled_Statement_Sequence =>
                Make_Handled_Sequence_Of_Statements (Loc,
-                 Statements => New_List (
-                   Make_Final_Call (
-                     Obj_Ref => Deref,
-                     Typ     => Desig_T)),
+                 Statements         => New_List (
+                   Make_Final_Call (Obj_Ref => Deref, Typ => Desig_T)),
                  Exception_Handlers => New_List (
                    Build_Exception_Handler (Finalizer_Data)))));
 
          --  For .NET/JVM, detach the object from the containing finalization
          --  collection before finalizing it.
 
-         if VM_Target /= No_VM
-           and then Is_Controlled (Desig_T)
-         then
+         if VM_Target /= No_VM and then Is_Controlled (Desig_T) then
             Prepend_To (Final_Code,
               Make_Detach_Call (New_Copy_Tree (Arg)));
          end if;
index d712570d920edb0207de04fbb590e2fd81713ac4..bc323a8afd444b4ece9168d181b1d5d24a58af0f 100644 (file)
@@ -2313,6 +2313,15 @@ package body Exp_Util is
          Typ := Corresponding_Record_Type (Typ);
       end if;
 
+      --  Since restriction violations are not considered serious errors, the
+      --  expander remains active, but may leave the corresponding record type
+      --  malformed. In such cases, component _object is not available so do
+      --  not look for it.
+
+      if not Analyzed (Typ) then
+         return Empty;
+      end if;
+
       Comp := First_Component (Typ);
       while Present (Comp) loop
          if Chars (Comp) = Name_uObject then
index 3c2b5f4df987e6ebd1f418b1063e35332a7f508f..98998fff9f06884ffc4830a27fb6b54e39f51032 100644 (file)
@@ -842,6 +842,10 @@ begin
             Tree_Gen;
          end if;
 
+         if CodePeer_Mode then
+            Comperr.Delete_SCIL_Files;
+         end if;
+
          Errout.Finalize (Last_Call => True);
          Exit_Program (E_Errors);
       end if;
index 9601dc690d177f16415a7636f3288b52d59ab292..0e6fb11745c2d337bd8322f8ef02058a653bfd64 100644 (file)
@@ -358,7 +358,7 @@ __gnat_error_handler (int sig, siginfo_t *si, void *ucontext)
          ((volatile char *)
           ((long) si->si_addr & - getpagesize ()))[getpagesize ()];
          exception = &storage_error;
-         msg = "stack overflow (or erroneous memory access)";
+         msg = "stack overflow or erroneous memory access";
        }
       break;
 
@@ -644,7 +644,7 @@ __gnat_error_handler (int sig, siginfo_t *si ATTRIBUTE_UNUSED, void *ucontext)
         that this is quite acceptable, since a "real" SIGSEGV can only
         occur as the result of an erroneous program.  */
       exception = &storage_error;
-      msg = "stack overflow (or erroneous memory access)";
+      msg = "stack overflow or erroneous memory access";
       break;
 
     case SIGBUS:
@@ -824,7 +824,7 @@ __gnat_error_handler (int sig, siginfo_t *reason, void *uc ATTRIBUTE_UNUSED)
                 the stack into a guard page, not an attempt to
                 write to .text or something.  */
          exception = &storage_error;
-         msg = "SIGSEGV: (stack overflow or erroneous memory access)";
+         msg = "SIGSEGV: stack overflow or erroneous memory access";
        }
       else
        {
@@ -1022,7 +1022,7 @@ __gnat_error_handler (int sig, siginfo_t *si, void *ucontext ATTRIBUTE_UNUSED)
          ((volatile char *)
           ((long) si->si_addr & - getpagesize ()))[getpagesize ()];
          exception = &storage_error;
-         msg = "stack overflow (or erroneous memory access)";
+         msg = "stack overflow or erroneous memory access";
        }
       break;
 
@@ -1421,7 +1421,7 @@ __gnat_handle_vms_condition (int *sigargs, void *mechargs)
        else
          {
            exception = &storage_error;
-           msg = "stack overflow (or erroneous memory access)";
+           msg = "stack overflow or erroneous memory access";
          }
        __gnat_adjust_context_for_raise (SS$_ACCVIO, (void *)mechargs);
        break;
index 32427df781736069414c71ef758e8c17f03053c6..1ff3cb3aefd839dfdc8bd457cb838f621b740814 100644 (file)
@@ -82,6 +82,7 @@ procedure Put_SCOs is
 
    procedure Write_SCO_Initiate (SU : SCO_Unit_Index) is
       SUT : SCO_Unit_Table_Entry renames SCO_Unit_Table.Table (SU);
+
    begin
       if Current_SCO_Unit /= SU then
          Write_Info_Initiate ('C');
@@ -126,7 +127,7 @@ begin
                T            : SCO_Table_Entry renames SCO_Table.Table (Start);
                Continuation : Boolean;
 
-               Ctr          : Nat;
+               Ctr : Nat;
                --  Counter for statement entries
 
             begin
index d0b14fdf9db89274360b430b6374bc31bc11ae2e..4e5e1d5579703fda58ea33b44e282a3ef859ba27 100644 (file)
@@ -87,6 +87,7 @@
 ------------------------------------------------------------------------------
 
 with Ada.Unchecked_Conversion;
+
 with System.Random_Seed;
 
 with Interfaces; use Interfaces;
@@ -480,7 +481,7 @@ package body System.Random_Numbers is
 
    procedure Reset (Gen : Generator) is
       X : constant Unsigned_32 :=
-           Unsigned_32'Mod (Unsigned_64 (Random_Seed.Get_Seed) * 64);
+            Unsigned_32'Mod (Unsigned_64 (Random_Seed.Get_Seed) * 64);
       --  Why * 64 ???
 
    begin
index dec22dbf4dc47dd6d78ec0d6941fc247f4b67b5f..ad0833a26f21d589d6ac1e5ec923ceee27b10692 100644 (file)
@@ -29,6 +29,8 @@
 --                                                                          --
 ------------------------------------------------------------------------------
 
+--  Version used on all systems except Ravenscar where Calendar is unavailable
+
 with Ada.Calendar; use Ada.Calendar;
 
 package body System.Random_Seed is
index 7a2dedd7e120aef5ff8d3647bde2070cc5506e2d..ffae8323c04e9434801ef37c91533f69a30be5b6 100644 (file)
 
 --  This package provide a seed for pseudo-random number generation using
 --  the clock.
+
 --  There are two separate implementations of this package:
 --  o  one based on Ada.Calendar
 --  o  one based on Ada.Real_Time
+
 --  This is required because Ada.Calendar cannot be used on ravenscar, but
---  Ada.Real_Time drags the tasking runtime on regular platforms.
+--  Ada.Real_Time drags in the whole tasking runtime on regular platforms.
 
 package System.Random_Seed is
 
index 610df54d6abafb3eb4abd407eb43db70cd0a1138..89c9ea48e090259aa574c68eec1c7c872ad93586 100644 (file)
@@ -99,7 +99,7 @@ __gnat_SEH_error_handler (struct _EXCEPTION_RECORD* ExceptionRecord,
        {
          /* otherwise it is a stack overflow  */
          exception = &storage_error;
-         msg = "stack overflow (or erroneous memory access)";
+         msg = "stack overflow or erroneous memory access";
        }
       break;
 
index ccd431fb65165668578e8f987cd50ca86bb098de..f8f00395c48bf09f8574c0a945b485824a2de10f 100644 (file)
@@ -2244,9 +2244,8 @@ package body Sem_Ch5 is
       Typ : Entity_Id;
 
    begin
-      --  In semantics mode, introduce loop variable so that
-      --  loop body can be properly analyzed. Otherwise this
-      --  is one after expansion.
+      --  In semantics mode, introduce loop variable so that loop body can be
+      --  properly analyzed. Otherwise this is one after expansion.
 
       if Operating_Mode = Check_Semantics then
          Enter_Name (Def_Id);
@@ -2335,7 +2334,7 @@ package body Sem_Ch5 is
             Error_Msg_N
               ("to iterate over the elements of an array, use OF", N);
 
-            --  Prevent cascaded errors.
+            --  Prevent cascaded errors
 
             Set_Ekind (Def_Id, E_Constant);
             Set_Etype (Def_Id, Etype (First_Index (Typ)));
@@ -2496,11 +2495,11 @@ package body Sem_Ch5 is
         or else not Expander_Active
       then
          if Present (Iter)
-           and then  Present (Iterator_Specification (Iter))
+           and then Present (Iterator_Specification (Iter))
          then
             declare
                Id : constant Entity_Id :=
-                  Defining_Identifier (Iterator_Specification (Iter));
+                      Defining_Identifier (Iterator_Specification (Iter));
             begin
                if Scope (Id) /= Current_Scope then
                   Enter_Name (Id);