Author: jani
Date: Mon Sep 30 11:57:33 2013
New Revision: 1527521

URL: http://svn.apache.org/r1527521
Log:
error in old xrm extract <title not extracted, corrected
setSourceKey in merge may not create new keys (throw instead)

Modified:
    openoffice/branches/l10n40/main/l10ntools/source/gConSrcWrap.cxx
    openoffice/branches/l10n40/main/l10ntools/source/gConTreeWrap.cxx
    openoffice/branches/l10n40/main/l10ntools/source/gConUlfWrap.cxx
    openoffice/branches/l10n40/main/l10ntools/source/gConXcsWrap.cxx
    openoffice/branches/l10n40/main/l10ntools/source/gConXcuWrap.cxx
    openoffice/branches/l10n40/main/l10ntools/source/gConXhpWrap.cxx
    openoffice/branches/l10n40/main/l10ntools/source/gConXrmWrap.cxx
    openoffice/branches/l10n40/main/l10ntools/source/gConXrmlex.l
    openoffice/branches/l10n40/main/l10ntools/source/gL10nMem.cxx
    openoffice/branches/l10n40/main/l10ntools/source/gL10nMem.hxx
    openoffice/branches/l10n40/main/l10ntools/source/gLang.hxx
    
openoffice/branches/l10n40/main/languages/source/templates/readlicense_oo.pot

Modified: openoffice/branches/l10n40/main/l10ntools/source/gConSrcWrap.cxx
URL: 
http://svn.apache.org/viewvc/openoffice/branches/l10n40/main/l10ntools/source/gConSrcWrap.cxx?rev=1527521&r1=1527520&r2=1527521&view=diff
==============================================================================
--- openoffice/branches/l10n40/main/l10ntools/source/gConSrcWrap.cxx (original)
+++ openoffice/branches/l10n40/main/l10ntools/source/gConSrcWrap.cxx Mon Sep 30 
11:57:33 2013
@@ -212,7 +212,7 @@ void convert_src::setNL(char *syyText, b
     sKey += "." + msCmd + "." + msTextName;
     if (msValue.size() && msValue != "-")
     {
-      mcMemory.setSourceKey(miLineNo, msSourceFile, sKey, msValue);
+      mcMemory.setSourceKey(miLineNo, msSourceFile, sKey, msValue, 
mbMergeMode);
       if (mbMergeMode)
         insertLanguagePart(sKey, msTextName);
     }

Modified: openoffice/branches/l10n40/main/l10ntools/source/gConTreeWrap.cxx
URL: 
http://svn.apache.org/viewvc/openoffice/branches/l10n40/main/l10ntools/source/gConTreeWrap.cxx?rev=1527521&r1=1527520&r2=1527521&view=diff
==============================================================================
--- openoffice/branches/l10n40/main/l10ntools/source/gConTreeWrap.cxx (original)
+++ openoffice/branches/l10n40/main/l10ntools/source/gConTreeWrap.cxx Mon Sep 
30 11:57:33 2013
@@ -130,7 +130,7 @@ void convert_tree::setString(char *yytex
     case STATE_VAL_TITLE:
          std::string sText = copySourceSpecial(yytext, 1);
          sText.erase(sText.size()-1);
-         mcMemory.setSourceKey(miLineNo, msSourceFile, msId, sText);
+         mcMemory.setSourceKey(miLineNo, msSourceFile, msId, sText, 
mbMergeMode);
          break;
   }
   meStateVal = STATE_VAL_NONE;
@@ -152,7 +152,7 @@ void convert_tree::setState(char *yytext
 /**********************   I M P L E M E N T A T I O N   **********************/
 void convert_tree::setValue(char *yytext)
 {
-  mcMemory.setSourceKey(miLineNo, msSourceFile, msId, msCollector);
+  mcMemory.setSourceKey(miLineNo, msSourceFile, msId, msCollector, 
mbMergeMode);
   copySourceSpecial(yytext, 2);
 
   meStateTag = STATE_TAG_NONE;

Modified: openoffice/branches/l10n40/main/l10ntools/source/gConUlfWrap.cxx
URL: 
http://svn.apache.org/viewvc/openoffice/branches/l10n40/main/l10ntools/source/gConUlfWrap.cxx?rev=1527521&r1=1527520&r2=1527521&view=diff
==============================================================================
--- openoffice/branches/l10n40/main/l10ntools/source/gConUlfWrap.cxx (original)
+++ openoffice/branches/l10n40/main/l10ntools/source/gConUlfWrap.cxx Mon Sep 30 
11:57:33 2013
@@ -90,6 +90,8 @@ void convert_ulf::setValue(char *syyText
   nL = sText.rfind("\"");
   sText.erase(nL);
 
+  mcMemory.setSourceKey(miLineNo, msSourceFile, msKey, sText, mbMergeMode);
+
   if (mbMergeMode)
   {
     // prepare to read all languages
@@ -97,11 +99,8 @@ void convert_ulf::setValue(char *syyText
     for (; mcMemory.getMergeLang(sLang, sText);)
     {
       // Prepare tag
-      sText = "\"\n" + sLang + " = \"" + sText;
+      sText = "\n" + sLang + " = \"" + sText + "\"";
       writeSourceFile(sText);
     }
   }
-  else
-    mcMemory.setSourceKey(miLineNo, msSourceFile, msKey, sText);
-
 }

Modified: openoffice/branches/l10n40/main/l10ntools/source/gConXcsWrap.cxx
URL: 
http://svn.apache.org/viewvc/openoffice/branches/l10n40/main/l10ntools/source/gConXcsWrap.cxx?rev=1527521&r1=1527520&r2=1527521&view=diff
==============================================================================
--- openoffice/branches/l10n40/main/l10ntools/source/gConXcsWrap.cxx (original)
+++ openoffice/branches/l10n40/main/l10ntools/source/gConXcsWrap.cxx Mon Sep 30 
11:57:33 2013
@@ -157,6 +157,6 @@ void convert_xcs::stopCollectData(char *
 #endif
   }
 
-  mcMemory.setSourceKey(miLineNo, msSourceFile, sKey, sText);
+  mcMemory.setSourceKey(miLineNo, msSourceFile, sKey, sText, mbMergeMode);
   mbCollectingData = false;
 }  

Modified: openoffice/branches/l10n40/main/l10ntools/source/gConXcuWrap.cxx
URL: 
http://svn.apache.org/viewvc/openoffice/branches/l10n40/main/l10ntools/source/gConXcuWrap.cxx?rev=1527521&r1=1527520&r2=1527521&view=diff
==============================================================================
--- openoffice/branches/l10n40/main/l10ntools/source/gConXcuWrap.cxx (original)
+++ openoffice/branches/l10n40/main/l10ntools/source/gConXcuWrap.cxx Mon Sep 30 
11:57:33 2013
@@ -63,9 +63,6 @@ namespace XcuWrap
 /**********************   I M P L E M E N T A T I O N   **********************/
 void convert_xcu::execute()
 {
-  if (mbMergeMode)
-    throw l10nMem::showError("Merge not implemented");
-
   XcuWrap::yylex();
 }
 
@@ -163,7 +160,7 @@ void convert_xcu::stopCollectData(char *
     // locate key and extract it
     for (nL = 0; nL < (int)mcStack.size(); ++nL)
       useKey += (useKey.size() ? "." : "" ) + mcStack[nL];
-    mcMemory.setSourceKey(miLineNo, msSourceFile, useKey, useText);
+    mcMemory.setSourceKey(miLineNo, msSourceFile, useKey, useText, 
mbMergeMode);
   }
 
   if (mbMergeMode)

Modified: openoffice/branches/l10n40/main/l10ntools/source/gConXhpWrap.cxx
URL: 
http://svn.apache.org/viewvc/openoffice/branches/l10n40/main/l10ntools/source/gConXhpWrap.cxx?rev=1527521&r1=1527520&r2=1527521&view=diff
==============================================================================
--- openoffice/branches/l10n40/main/l10ntools/source/gConXhpWrap.cxx (original)
+++ openoffice/branches/l10n40/main/l10ntools/source/gConXhpWrap.cxx Mon Sep 30 
11:57:33 2013
@@ -274,7 +274,7 @@ void convert_xhp::closeTransTag(char *yy
   if (meExpectValue == VALUE_IS_VALUE || meExpectValue == VALUE_IS_VALUE_TAG)
   {
     if (msCollector.size() && msCollector != "-")
-      mcMemory.setSourceKey(miLineNo, msSourceFile, msKey, msCollector);
+      mcMemory.setSourceKey(miLineNo, msSourceFile, msKey, msCollector, 
mbMergeMode);
     msKey.clear();
     iType = 2;
   }

Modified: openoffice/branches/l10n40/main/l10ntools/source/gConXrmWrap.cxx
URL: 
http://svn.apache.org/viewvc/openoffice/branches/l10n40/main/l10ntools/source/gConXrmWrap.cxx?rev=1527521&r1=1527520&r2=1527521&view=diff
==============================================================================
--- openoffice/branches/l10n40/main/l10ntools/source/gConXrmWrap.cxx (original)
+++ openoffice/branches/l10n40/main/l10ntools/source/gConXrmWrap.cxx Mon Sep 30 
11:57:33 2013
@@ -146,6 +146,7 @@ void convert_xrm::stopCollectData(char *
   copySource(yytext);
   if (!mbNoCollectingData)
   {
+    mcMemory.setSourceKey(miLineNo, msSourceFile, msKey, sText, mbMergeMode);
     mbNoCollectingData = true;
     if (mbMergeMode)
     {
@@ -165,8 +166,6 @@ void convert_xrm::stopCollectData(char *
         writeSourceFile(sTagText);
       }
     }
-    else
-      mcMemory.setSourceKey(miLineNo, msSourceFile, msKey, sText);
     msKey.clear();
   }
   mbIsTag = false;

Modified: openoffice/branches/l10n40/main/l10ntools/source/gConXrmlex.l
URL: 
http://svn.apache.org/viewvc/openoffice/branches/l10n40/main/l10ntools/source/gConXrmlex.l?rev=1527521&r1=1527520&r2=1527521&view=diff
==============================================================================
--- openoffice/branches/l10n40/main/l10ntools/source/gConXrmlex.l (original)
+++ openoffice/branches/l10n40/main/l10ntools/source/gConXrmlex.l Mon Sep 30 
11:57:33 2013
@@ -75,7 +75,7 @@ SP    [ \t]*
 }
 
 
-
+"<title "    |
 "<h"[0-9]" " |
 "<p "        {
   LOCptr->setTag(yytext);
@@ -83,6 +83,7 @@ SP    [ \t]*
 
 
 
+"</title>"[ \t\r]*[\n]    |
 "</h"[0-9]">"[ \t\r]*[\n] |
 "</p>"[ \t\r]*[\n]        {
   LOCptr->stopCollectData(yytext);

Modified: openoffice/branches/l10n40/main/l10ntools/source/gL10nMem.cxx
URL: 
http://svn.apache.org/viewvc/openoffice/branches/l10n40/main/l10ntools/source/gL10nMem.cxx?rev=1527521&r1=1527520&r2=1527521&view=diff
==============================================================================
--- openoffice/branches/l10n40/main/l10ntools/source/gL10nMem.cxx (original)
+++ openoffice/branches/l10n40/main/l10ntools/source/gL10nMem.cxx Mon Sep 30 
11:57:33 2013
@@ -93,8 +93,8 @@ void l10nMem::setConvert(bool bC, bool b
      { l10nMem_impl::mcImpl->mcDb.setConvert(bC, bS); }
 void l10nMem::loadEntryKey(int iL, const std::string& sS, const std::string& 
sK, const std::string& sO, const std::string& sT, bool               bI)
      { l10nMem_impl::mcImpl->loadEntryKey(iL, sS, sK, sO, sT, bI); }
-void l10nMem::setSourceKey(int iL, const std::string& sF, const std::string& 
sK, const std::string& sT)
-     { l10nMem_impl::mcImpl->setSourceKey(iL, sF, sK, sT); }
+void l10nMem::setSourceKey(int iL, const std::string& sF, const std::string& 
sK, const std::string& sT, bool bM)
+     { l10nMem_impl::mcImpl->setSourceKey(iL, sF, sK, sT, bM); }
 void l10nMem::saveTemplates(const std::string& sT, bool bK, bool bF)
      { l10nMem_impl::mcImpl->saveTemplates(*this, sT, bK, bF); }
 void l10nMem::saveLanguages(const std::string& sT, bool bF)
@@ -194,12 +194,12 @@ void l10nMem_impl::loadEntryKey(int     
 void l10nMem_impl::setSourceKey(int                iLineNo,
                                 const std::string& sSourceFile,
                                 const std::string& sKey,
-                                const std::string& sMsgId)
+                                const std::string& sMsgId,
+                                bool               bMustExist)
 {
   std::string newText(sMsgId);
   int         i;
 
-
   // time to escape " and \ if contained in text or key
   for (i = 0; (i = newText.find("\\", i)) != (int)std::string::npos;)
   {
@@ -228,8 +228,11 @@ void l10nMem_impl::setSourceKey(int     
   }
   else
   {
-    // add key, if changed text, this is wrong but handled in reorganize
-    mcDb.addKey(iLineNo, sSourceFile, sKey, newText, l10nMem::ENTRY_ADDED);
+    if (bMustExist)
+      throw l10nMem::showError("key " + sKey + " does not exist");
+    else
+      // add key, if changed text, this is wrong but handled in reorganize
+      mcDb.addKey(iLineNo, sSourceFile, sKey, newText, l10nMem::ENTRY_ADDED);
   }
 }
 

Modified: openoffice/branches/l10n40/main/l10ntools/source/gL10nMem.hxx
URL: 
http://svn.apache.org/viewvc/openoffice/branches/l10n40/main/l10ntools/source/gL10nMem.hxx?rev=1527521&r1=1527520&r2=1527521&view=diff
==============================================================================
--- openoffice/branches/l10n40/main/l10ntools/source/gL10nMem.hxx (original)
+++ openoffice/branches/l10n40/main/l10ntools/source/gL10nMem.hxx Mon Sep 30 
11:57:33 2013
@@ -176,7 +176,8 @@ class l10nMem_impl
     void setSourceKey  (int                iLineNo,
                         const std::string& sFilename,
                         const std::string& sKey,
-                        const std::string& sMsgId);
+                        const std::string& sMsgId,
+                        bool               bMustExist);
 
     void saveTemplates (l10nMem& cMem,
                         const std::string& sTargetDir,

Modified: openoffice/branches/l10n40/main/l10ntools/source/gLang.hxx
URL: 
http://svn.apache.org/viewvc/openoffice/branches/l10n40/main/l10ntools/source/gLang.hxx?rev=1527521&r1=1527520&r2=1527521&view=diff
==============================================================================
--- openoffice/branches/l10n40/main/l10ntools/source/gLang.hxx (original)
+++ openoffice/branches/l10n40/main/l10ntools/source/gLang.hxx Mon Sep 30 
11:57:33 2013
@@ -81,7 +81,8 @@ class l10nMem
     void setSourceKey  (int                iLineNo,
                         const std::string& sFilename,
                         const std::string& sKey,
-                        const std::string& sText);
+                        const std::string& sText,
+                        bool               bMustExist);
 
     void saveTemplates (const std::string& sTargetDir,
                         bool               bKid,

Modified: 
openoffice/branches/l10n40/main/languages/source/templates/readlicense_oo.pot
URL: 
http://svn.apache.org/viewvc/openoffice/branches/l10n40/main/languages/source/templates/readlicense_oo.pot?rev=1527521&r1=1527520&r2=1527521&view=diff
==============================================================================
--- 
openoffice/branches/l10n40/main/languages/source/templates/readlicense_oo.pot 
(original)
+++ 
openoffice/branches/l10n40/main/languages/source/templates/readlicense_oo.pot 
Mon Sep 30 11:57:33 2013
@@ -31,6 +31,10 @@ msgstr ""
 "X-Generator: genLang\n"
 
 
+#: readme.xrm#TITLE
+msgid  "${PRODUCTNAME} Readme File"
+msgstr ""
+
 #: readme.xrm#WELCOME
 msgid  "${PRODUCTNAME} ${PRODUCTVERSION} ReadMe"
 msgstr ""


Reply via email to