From: Vincent Celier <celier@adacore.com>
Date: Mon, 1 Aug 2011 10:15:08 +0000 (+0000)
Subject: a-stzfix.adb, [...] (Replace_Slice): Fixed computation when High is above Source... 
X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=a3a16b218709b07f98329f42bc805d1c7731f71b;p=gcc.git

a-stzfix.adb, [...] (Replace_Slice): Fixed computation when High is above Source length.

2011-08-01  Vincent Celier  <celier@adacore.com>

	* a-stzfix.adb, a-stwifi.adb (Replace_Slice): Fixed computation when
	High is above Source length.

From-SVN: r177007
---

diff --git a/gcc/ada/ChangeLog b/gcc/ada/ChangeLog
index f8cebbace6c..e9de3368258 100644
--- a/gcc/ada/ChangeLog
+++ b/gcc/ada/ChangeLog
@@ -1,3 +1,8 @@
+2011-08-01  Vincent Celier  <celier@adacore.com>
+
+	* a-stzfix.adb, a-stwifi.adb (Replace_Slice): Fixed computation when
+	High is above Source length.
+
 2011-08-01  Robert Dewar  <dewar@adacore.com>
 
 	* a-ztexio.ads, a-textio.ads, a-witeio.ads: Fix comment.
diff --git a/gcc/ada/a-stwifi.adb b/gcc/ada/a-stwifi.adb
index c4229062424..31278505b8a 100644
--- a/gcc/ada/a-stwifi.adb
+++ b/gcc/ada/a-stwifi.adb
@@ -447,30 +447,40 @@ package body Ada.Strings.Wide_Fixed is
       High   : Natural;
       By     : Wide_String) return Wide_String
    is
-      Result_Length : Natural;
-
    begin
       if Low > Source'Last + 1 or else High < Source'First - 1 then
          raise Index_Error;
-      else
-         Result_Length :=
-           Source'Length - Natural'Max (High - Low + 1, 0) + By'Length;
+      end if;
 
+      if High >= Low then
          declare
-            Result : Wide_String (1 .. Result_Length);
+            Front_Len : constant Integer :=
+              Integer'Max (0, Low - Source'First);
+            --  Length of prefix of Source copied to result
+
+            Back_Len  : constant Integer :=
+              Integer'Max (0, Source'Last - High);
+            --  Length of suffix of Source copied to result
+
+            Result_Length : constant Integer :=
+              Front_Len + By'Length + Back_Len;
+            --  Length of result
+
+            Result    : Wide_String (1 .. Result_Length);
 
          begin
-            if High >= Low then
-               Result :=
-                  Source (Source'First .. Low - 1) & By &
-                  Source (High + 1 .. Source'Last);
-            else
-               Result := Source (Source'First .. Low - 1) & By &
-                         Source (Low .. Source'Last);
-            end if;
+            Result (1 .. Front_Len) :=
+              Source (Source'First .. Low - 1);
+            Result (Front_Len + 1 .. Front_Len + By'Length) :=
+              By;
+            Result (Front_Len + By'Length + 1 .. Result'Length) :=
+              Source (High + 1 .. Source'Last);
 
             return Result;
          end;
+
+      else
+         return Insert (Source, Before => Low, New_Item => By);
       end if;
    end Replace_Slice;
 
diff --git a/gcc/ada/a-stzfix.adb b/gcc/ada/a-stzfix.adb
index 077a65c0ecd..67f5482f95b 100644
--- a/gcc/ada/a-stzfix.adb
+++ b/gcc/ada/a-stzfix.adb
@@ -449,30 +449,40 @@ package body Ada.Strings.Wide_Wide_Fixed is
       High   : Natural;
       By     : Wide_Wide_String) return Wide_Wide_String
    is
-      Result_Length : Natural;
-
    begin
       if Low > Source'Last + 1 or else High < Source'First - 1 then
          raise Index_Error;
-      else
-         Result_Length :=
-           Source'Length - Natural'Max (High - Low + 1, 0) + By'Length;
+      end if;
 
+      if High >= Low then
          declare
+            Front_Len : constant Integer :=
+                          Integer'Max (0, Low - Source'First);
+            --  Length of prefix of Source copied to result
+
+            Back_Len  : constant Integer :=
+                          Integer'Max (0, Source'Last - High);
+            --  Length of suffix of Source copied to result
+
+            Result_Length : constant Integer :=
+                              Front_Len + By'Length + Back_Len;
+            --  Length of result
+
             Result : Wide_Wide_String (1 .. Result_Length);
 
          begin
-            if High >= Low then
-               Result :=
-                  Source (Source'First .. Low - 1) & By &
-                  Source (High + 1 .. Source'Last);
-            else
-               Result := Source (Source'First .. Low - 1) & By &
-                         Source (Low .. Source'Last);
-            end if;
+            Result (1 .. Front_Len) :=
+              Source (Source'First .. Low - 1);
+            Result (Front_Len + 1 .. Front_Len + By'Length) :=
+              By;
+            Result (Front_Len + By'Length + 1 .. Result'Length) :=
+              Source (High + 1 .. Source'Last);
 
             return Result;
          end;
+
+      else
+         return Insert (Source, Before => Low, New_Item => By);
       end if;
    end Replace_Slice;