Comment #3 on issue 1647 by jameson....@gmail.com: one instance of </ Left"> instead of </script> in output.xml
http://code.google.com/p/robotframework/issues/detail?id=1647

Note: "t`.</doc>" matches the end of the <doc> section of Collections.Get From List:

<kw type="kw" name="${c} = Collections.Get From List">
<doc>Returns the value specified with an `index` from `list`.</doc>

" Left"" matches part of String.Fetch From Left:

<kw type="kw" name="${text} = String.Fetch From Left">
<doc>Returns contents of the `string` before the first occurrence of `marker`.</doc>


This is the keyword where the issue happened both times:

Convert RGBA Colour From Text To Array
[Documentation] Converts the text RGBA colour to an array of four numbers, which is what CanvasTestHelper expects
  [Arguments]  ${text}
  ${text}=  Fetch From Right  ${text}  (
  ${text}=  Fetch From Left  ${text}  )
  ${colour}=  Split String  ${text}  ,${SPACE}
  :FOR  ${n}  IN RANGE  4
  \   ${c}=  Get From List  ${colour}  ${n}
  \   ${c}=  Run Keyword If  ${n} == 3  Evaluate  ${c} * 255
  \          ...       ELSE             Evaluate  ${c}
  \   ${c}=  Convert To Integer  ${c}
  \   ${c}=  Set List Value  ${colour}  ${n}  ${c}
  [Return]  ${colour}

Another test developer wrote this keyword. Looks like he added a lot of spaces in order to line stuff up. Are all those spaces OK in the "ELSE" line? Are 3 spaces OK between the \ and ${c} in the other lines? Maybe this is a syntax error manifesting itself as a sort of data corruption parsing issue??

--
You received this message because this project is configured to send all issue notifications to this address.
You may adjust your notification preferences at:
https://code.google.com/hosting/settings

--

--- You received this message because you are subscribed to the Google Groups "robotframework-commit" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to robotframework-commit+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/groups/opt_out.

Reply via email to