D24921: [KDevelop/PHP] Add support for "array of type"

Heinz Wiesinger <noreply-Buy57tU0AsCGxO/ty/[email protected]>
Newsgroups gmane.comp.kde.devel.kdevelop
Message-ID <[email protected]>
pprkut added a comment.


  Conceptually, this looks fine. However, like this we end up with two different descriptions of "Array of" types in the navigation popups.
  While this shows integer arrays as `int[]`, for variadics it will show `array of (int)`. I think at the very least both should be displayed the same.
  
  In the end I think the simplest solution here might just be moving variadics from IndexedContainer to ArrayType (I didn't know about ArrayType back when I added support for variadics), as IndexedContainer seems a bit overkill for what PHP supports.
  Could you perhaps have a look at that?

REPOSITORY
  R52 KDevelop: PHP Support

REVISION DETAIL
  https://phabricator.kde.org/D24921

To: hmitonneau
Cc: pprkut, kdevelop-devel, hmitonneau, christiant, glebaccon, domson, antismap, iodelay, alexeymin, geetamc, Pilzschaf, akshaydeo, surgenight, arrowd
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.