Re: Patch

[Chris Leick <c.leick@vollbio.de>]

Jens Seidel schrieb:
> Chris Leick wrote:
>   
> > Jens Seidel schrieb:
> >     
Neuer Versuch.

Die beiden Patches habe ich nun per diff -ru erstellt.

> > gnls.diff:
> > Translation of multiple parts of a single string may be difficult.
> >     
>
> Besser noch weiter erklären, z.B. an einem Beispiel. Das "may" würde ich
> auch herausnehmen. Aktuell können die Zeichenketten mit Sicherheit nicht in
> viele Sprachen übersetzt werden! Ich glaube, die Autoren wissen gar nichts
> vom Problem. Schreibe z.B. das zurzeit jedes Argument von paste() unabhängig
> von den anderen übersetzt wird (da ja eine Zeichenkette auch mehrfach an
> verschiedenen Stellen im Programm unterschiedlich verwendet werden könnte).
>   

Das habe ich nun einmal ausprobiert. »warning()« hängt die Strings 
einfach aneinander, z.B warnung("str1","str2") ergibt "str1str2". Paste 
fügt ein Leerzeichen ein paste("str1","str2") ergibt "str1 str2". Für 
den Übersetzer ist das kein Problem, er kann die Strings beliebig 
aufteilen und muß sich nicht einmal über das Leerzeichen Gedanken 
machen. Diesen Patch lasse ich daher mal weg.

-------------
Package: nlme
Version: 3.1.90-1
Severity: minor
Tags: Patch

Hi,

The following patch should fix some errors and oddities.


ReduceTranslation.diff:
R/corStruct.R:  stop("Initial value of nugget ratio must be between 0 
and 1")
R/corStruct.R:  stop("Initial value of nugget ratio must be in (0,1)")
R/corStruct.R:  stop("Initial value of nugget ratio must be between 0 
and 1")

This string need to be translated independently and it is possible to
reduce these to a single string to reduce translators work.


TypoFixes.diff:

Patch fixes a grammatical error (missing "of")

Greetings,
Chris

diff -ru nlme/R/corStruct.R nlme.ReduceTranslation/R/corStruct.R
--- nlme/R/corStruct.R	2008-02-11 15:05:14.000000000 +0100
+++ nlme.ReduceTranslation/R/corStruct.R	2009-05-11 10:04:08.000000000 +0200
@@ -2144,7 +2144,7 @@
 	}
       }
       if ((val[2] <= 0) || (val[2] >= 1)) {
-	stop("Initial value of nugget ratio must be in (0,1)")
+	stop("Initial value of nugget ratio must be between 0 and 1")
       }
     } else {				# only range parameter
       if (length(val) != 1) {
Nur in nlme.ReduceTranslation/R: corStruct.R~.

diff -ru nlme/R/gnls.R nlme.TypoFixes/R/gnls.R
--- nlme/R/gnls.R	2008-02-18 09:47:09.000000000 +0100
+++ nlme.TypoFixes/R/gnls.R	2009-05-11 10:15:04.000000000 +0200
@@ -271,7 +271,7 @@
   }
   pLen <- length(pn)
   if (length(start) != pLen)
-    stop ("starting values for parameters are not the correct length")
+    stop ("starting values for parameters are not of the correct length")
   spar <- start
   names(spar) <- pn
   NReal <- sum(naPat)
Nur in nlme.TypoFixes/R: gnls.R~.
diff -ru nlme/R/nlme.R nlme.TypoFixes/R/nlme.R
--- nlme/R/nlme.R	2007-07-27 11:26:37.000000000 +0200
+++ nlme.TypoFixes/R/nlme.R	2009-05-11 10:12:49.000000000 +0200
@@ -523,7 +523,7 @@
   }
   fLen <- length(fn)
   if (length(sfix) != fLen)
-    stop ("starting values for the fixed component are not the correct length")
+    stop ("starting values for the fixed component are not of the correct length")
   names(sfix) <- fn
   ##
   ## Random effects names
Nur in nlme.TypoFixes/R: nlme.R~.