Menu

#890 Function Block types for array manipulation

1.6
closed-fixed
nobody
Arrays (2)
FORTE
New_Feature
5
2014-08-12
2014-08-05
No

New function block types for manipulating arrays:

  • GEN_ARRAY2ARRAY: Generic conversion FB for arrays. Array data type needs to be specified in typename as follows: ARRAY2ARRAY_5_INT ==> conversion of INT array with length 5. Make sure, that applied input array is consistent with output array.

  • GET_AT_INDEX: get array value at specified index. output data type and array data type need to be equal.

  • SET_AT_INDEX: set array value at given index. data types and array lengths need to be equal.

patch will be applied.

Related

Issues: #891

Discussion

  • Matthias Plasch

    Matthias Plasch - 2014-08-05

    patch file enclosed.
    One more thing: since a number of array manipulation FB types do now exist in the folder utils, does it make sense to create a new folder or even a module for array manipulation?
    Looking forward to your opinion.

     
  • Zoitl Alois

    Zoitl Alois - 2014-08-05

    I had a look on your patch in your CMakeLists.txt to futher FBs are mentioned: GEN_ARRAY2VALUES GEN_VALUES2ARRAY but not part of the patch. Are they missing or should they be removed from the CMakeLists.txt.

     
  • Matthias Plasch

    Matthias Plasch - 2014-08-06

    I provided these changes (entries of GEN_ARRAY2VALUES GEN_VALUES2ARRAY) earlier, see the meanwhile closed issue [issues:873]. There i orignially forgot to add the entries and uploaded a new patch. Please apply this patch at first.
    The file is called issues#873_2.patch or somehow similar.

     

    Last edit: Matthias Plasch 2014-08-06
  • Zoitl Alois

    Zoitl Alois - 2014-08-12
    • status: open --> closed-fixed
     
  • Zoitl Alois

    Zoitl Alois - 2014-08-12

    I'm very sorry but I somehow missed the patch you are referencing. I applied the referenced patch but still had some issues for applying the patch submitted here. I recreated it and pushed it under cset [forte:177d05]. I hope that is ok for you.

     

    Related

    Commit: [177d05]


Log in to post a comment.

MongoDB Logo MongoDB