From: Bruno Haible <bruno@clisp.org>
Date: Mon, 4 Feb 2008 23:22:38 +0000 (+0000)
Subject: Don't throw away a prev_msgid information too early.
X-Git-Tag: v0.18~462
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=0b9ce53958cad74193f0861c3380466c229f5d9f;p=thirdparty%2Fgettext.git

Don't throw away a prev_msgid information too early.
---

diff --git a/gettext-tools/src/ChangeLog b/gettext-tools/src/ChangeLog
index 441cd2269..61624e2d9 100644
--- a/gettext-tools/src/ChangeLog
+++ b/gettext-tools/src/ChangeLog
@@ -1,3 +1,10 @@
+2008-02-04  Bruno Haible  <bruno@clisp.org>
+
+	* msgmerge.c (message_merge): Store the prev_msgid field also if the
+	resulting message is not fuzzy: it might be marked as fuzzy later.
+	(match_domain): Remove the prev_msgid fields from the non-fuzzy
+	messages.
+
 2008-01-29  Bruno Haible  <bruno@clisp.org>
 
 	* write-po.c (wrap): Change the severity of the "should not contain
diff --git a/gettext-tools/src/msgmerge.c b/gettext-tools/src/msgmerge.c
index d65c37de1..7edef4106 100644
--- a/gettext-tools/src/msgmerge.c
+++ b/gettext-tools/src/msgmerge.c
@@ -1,5 +1,5 @@
 /* GNU gettext - internationalization aids
-   Copyright (C) 1995-1998, 2000-2007 Free Software Foundation, Inc.
+   Copyright (C) 1995-1998, 2000-2008 Free Software Foundation, Inc.
    This file was written by Peter Miller <millerp@canb.auug.org.au>
 
    This program is free software: you can redistribute it and/or modify
@@ -1172,11 +1172,13 @@ message_merge (message_ty *def, message_ty *ref, bool force_fuzzy,
      Do so only when --previous is specified, for backward compatibility.
      Since the "previous msgid" represents the original msgid that led to
      the current msgstr,
-       - we can omit it if the resulting message is not fuzzy,
+       - we can omit it if the resulting message is not fuzzy (but do this
+         in a later pass, since result->is_fuzzy is not finalized at this
+         point),
        - otherwise, if the corresponding message from the definition file
          was translated (not fuzzy), we use that message's msgid,
        - otherwise, we use that message's prev_msgid.  */
-  if (keep_previous && result->is_fuzzy)
+  if (keep_previous)
     {
       result->prev_msgctxt = prev_msgctxt;
       result->prev_msgid = prev_msgid;
@@ -1493,6 +1495,20 @@ this message should not define plural forms"));
 	  }
       }
   }
+
+  /* Now that mp->is_fuzzy is finalized for all messages, remove the previous
+     msgid information from all messages that are not fuzzy.  */
+  for (j = 0; j < resultmlp->nitems; j++)
+    {
+      message_ty *mp = resultmlp->item[j];
+
+      if (!mp->is_fuzzy)
+	{
+	  mp->prev_msgctxt = NULL;
+	  mp->prev_msgid = NULL;
+	  mp->prev_msgid_plural = NULL;
+	}
+    }
 }
 
 static msgdomain_list_ty *