Re: Error when diffing Excel files (new file has one additional worksheet)

On 23.10.2015 14:09, Niemann, Hartmut wrote:
> Hello!
>
> When diffing an excel file with its previous version, I got
>
> Windows Scripting Host
>
> Skript: ...Diff-Scripts\diff-xls.js
>
> Zeile/Line 145
>
> Fehler: Index außerhalb des gültigen Bereichs
>
> (Error: Index out of range)
>
> Code: 800A0009
>
> Quelle: Laufzeitfehler in Microsoft JScript
>
> (Source: Runtime error in Microsoft JScript)
>
> The relevant portion is
>
> // Mark differences in sNewDoc red
>
> var length = objNewWorkbook.Worksheets.Count;
>
> for (var i = 1; i <= length; i++)
>
> {
>
> var objBaseWorksheet = objBaseWorkbook.Worksheets(i); <<<< line 145
>
> var objNewWorksheet = objNewWorkbook.Worksheets(i);
>
> and in this case the new version has an additional worksheet which base
> does not have,
>
> so last i = length (from objNewWorkbook) is out of range for objBaseWorkbook
>
> I don’t know this language and excel well enough to suggest a fix - who
> can repair this?
>
> (I got this report from a coworker using 1.8.11, same here with 1.9.2)

Fixed in r26930.

Stefan

-- 
        ___
   oo  // \\      "De Chelonian Mobile"
  (_,\/ \_/ \     TortoiseSVN
    \ \_/_\_/>    The coolest interface to (Sub)version control
    /_/   \_\     http://tortoisesvn.net
------------------------------------------------------
http://tortoisesvn.tigris.org/ds/viewMessage.do?dsForumId=4061&dsMessageId=3144040
To unsubscribe from this discussion, e-mail: [users-unsubscribe_at_tortoisesvn.tigris.org].