{"action":"created","issue":{"url":"https://api.github.com/repos/mpiforumbot/mpi-forum-historic/issues/125","repository_url":"https://api.github.com/repos/mpiforumbot/mpi-forum-historic","labels_url":"https://api.github.com/repos/mpiforumbot/mpi-forum-historic/issues/125/labels{/name}","comments_url":"https://api.github.com/repos/mpiforumbot/mpi-forum-historic/issues/125/comments","events_url":"https://api.github.com/repos/mpiforumbot/mpi-forum-historic/issues/125/events","html_url":"https://github.com/mpiforumbot/mpi-forum-historic/issues/125","id":167224832,"number":125,"title":"Use of [] in output arrays","user":{"login":"mpiforumbot","id":20617676,"avatar_url":"https://avatars.githubusercontent.com/u/20617676?v=3","gravatar_id":"","url":"https://api.github.com/users/mpiforumbot","html_url":"https://github.com/mpiforumbot","followers_url":"https://api.github.com/users/mpiforumbot/followers","following_url":"https://api.github.com/users/mpiforumbot/following{/other_user}","gists_url":"https://api.github.com/users/mpiforumbot/gists{/gist_id}","starred_url":"https://api.github.com/users/mpiforumbot/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mpiforumbot/subscriptions","organizations_url":"https://api.github.com/users/mpiforumbot/orgs","repos_url":"https://api.github.com/users/mpiforumbot/repos","events_url":"https://api.github.com/users/mpiforumbot/events{/privacy}","received_events_url":"https://api.github.com/users/mpiforumbot/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/mpiforumbot/mpi-forum-historic/labels/MPI%203%5C.0","name":"MPI 3\\.0","color":"ededed"},{"url":"https://api.github.com/repos/mpiforumbot/mpi-forum-historic/labels/Passed","name":"Passed","color":"b60205"},{"url":"https://api.github.com/repos/mpiforumbot/mpi-forum-historic/labels/Text%20committed","name":"Text committed","color":"d93f0b"},{"url":"https://api.github.com/repos/mpiforumbot/mpi-forum-historic/labels/Trivial%20text%20changes","name":"Trivial text changes","color":"bfdadc"}],"state":"open","locked":false,"assignee":{"login":"mpiforumbot","id":20617676,"avatar_url":"https://avatars.githubusercontent.com/u/20617676?v=3","gravatar_id":"","url":"https://api.github.com/users/mpiforumbot","html_url":"https://github.com/mpiforumbot","followers_url":"https://api.github.com/users/mpiforumbot/followers","following_url":"https://api.github.com/users/mpiforumbot/following{/other_user}","gists_url":"https://api.github.com/users/mpiforumbot/gists{/gist_id}","starred_url":"https://api.github.com/users/mpiforumbot/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mpiforumbot/subscriptions","organizations_url":"https://api.github.com/users/mpiforumbot/orgs","repos_url":"https://api.github.com/users/mpiforumbot/repos","events_url":"https://api.github.com/users/mpiforumbot/events{/privacy}","received_events_url":"https://api.github.com/users/mpiforumbot/received_events","type":"User","site_admin":false},"assignees":[{"login":"mpiforumbot","id":20617676,"avatar_url":"https://avatars.githubusercontent.com/u/20617676?v=3","gravatar_id":"","url":"https://api.github.com/users/mpiforumbot","html_url":"https://github.com/mpiforumbot","followers_url":"https://api.github.com/users/mpiforumbot/followers","following_url":"https://api.github.com/users/mpiforumbot/following{/other_user}","gists_url":"https://api.github.com/users/mpiforumbot/gists{/gist_id}","starred_url":"https://api.github.com/users/mpiforumbot/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mpiforumbot/subscriptions","organizations_url":"https://api.github.com/users/mpiforumbot/orgs","repos_url":"https://api.github.com/users/mpiforumbot/repos","events_url":"https://api.github.com/users/mpiforumbot/events{/privacy}","received_events_url":"https://api.github.com/users/mpiforumbot/received_events","type":"User","site_admin":false}],"milestone":{"url":"https://api.github.com/repos/mpiforumbot/mpi-forum-historic/milestones/25","html_url":"https://github.com/mpiforumbot/mpi-forum-historic/milestones/2012/03/05%20Chicago,%20USA","labels_url":"https://api.github.com/repos/mpiforumbot/mpi-forum-historic/milestones/25/labels","id":1900486,"number":25,"title":"2012/03/05 Chicago, USA","description":"","creator":{"login":"mpiforumbot","id":20617676,"avatar_url":"https://avatars.githubusercontent.com/u/20617676?v=3","gravatar_id":"","url":"https://api.github.com/users/mpiforumbot","html_url":"https://github.com/mpiforumbot","followers_url":"https://api.github.com/users/mpiforumbot/followers","following_url":"https://api.github.com/users/mpiforumbot/following{/other_user}","gists_url":"https://api.github.com/users/mpiforumbot/gists{/gist_id}","starred_url":"https://api.github.com/users/mpiforumbot/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mpiforumbot/subscriptions","organizations_url":"https://api.github.com/users/mpiforumbot/orgs","repos_url":"https://api.github.com/users/mpiforumbot/repos","events_url":"https://api.github.com/users/mpiforumbot/events{/privacy}","received_events_url":"https://api.github.com/users/mpiforumbot/received_events","type":"User","site_admin":false},"open_issues":1,"closed_issues":0,"state":"closed","created_at":"2016-07-23T22:09:12Z","updated_at":"2016-07-24T09:16:20Z","due_on":"2012-03-07T20:00:00Z","closed_at":"2016-07-23T22:09:12Z"},"comments":13,"created_at":"2016-07-24T09:16:02Z","updated_at":"2016-07-24T09:16:23Z","closed_at":null,"body":"**Originally by _gropp_ on 2009-02-10 18:49:52 -0600**\n\n___\n## Description\nIt was noticed that the bindings do not consistently use [] or * for input arrays (many in the Forum expressed a preference for the  [] form for input arrays; e.g., const int recvcounts[] to make clear that the input is an array that is not changed; the const is included because this is better style for in-only arguments such as this).  The question was raised about *output* arrays and the use of [] in output arguments.  This ticket is a place holder for the chapter authors to inspect their chapters for such usage (e.g., output arrays) and then to propose any changes for consistency.  Chapter authors should edit this description to indicate that their chapter has been checked.\n\nTicket #126 covers input arrays\n\n## Proposed Solution\n### Chapter 1 (Introduction)\n### Chapter 2 (Terms and Conventions)\n### Chapter 3 (Pt2pt)\n### Chapter 4 (Datatypes)\n### Chapter 5 (Collective Communication)\n\nNo issues\n\n### Chapter 6 (Groups, Contexts, Communicators, and Caching)\n### Chapter 7 (Process Topologies)\n\np. 248, line 37, C++ binding should be changed from\n\n`void MPI::Graphcomm::Get_dims(comm, int nnodes[], int nnedges[]) const`\n\nto\n\n`void MPI::Graphcomm::Get_dims(comm, int &nnodes, int &nnedges) const`\n\np. 249, line 13-14: Replace\n\n```\n#!c\nint MPI_Graph_get(MPI_Comm comm, int maxindex, int maxedges, int *index,\n                  int *edges)\n```\n\nby\n\n```\n#!c\nint MPI_Graph_get(MPI_Comm comm, int maxindex, int maxedges, \n                  int index[], int edges[])\n```\n\np. 250, line 14-15: Replace\n\n```\n#!c\nint MPI_Cart_get(MPI_Comm comm, int maxdims, int *dims, int *periods,\n                 int *coords)\n```\n\nby\n\n```\n#!c\nint MPI_Cart_get(MPI_Comm comm, int maxdims, \n                 int dims[], int periods[], int coords[])\n```\n\np. 251, line 15: Replace\n\n```\n#!c\nint MPI_Cart_coords(MPI_Comm comm, int rank, int maxdims, int *coords)\n```\n\nby\n\n```\n#!c\nint MPI_Cart_coords(MPI_Comm comm, int rank, int maxdims, int coords[])\n```\n\np. 251, line 46-47: Replace\n\n```\n#!c\nint MPI_Graph_neighbors(MPI_Comm comm, int rank, int maxneighbors,\n                        int *neighbors)\n```\n\nby\n\n```\n#!c\nint MPI_Graph_neighbors(MPI_Comm comm, int rank, int maxneighbors,\n                        int neighbors[])\n```\n\n### Chapter 8 (Environmental Management)\n### Chapter 9 (Info)\n\nNo issues\n### Chapter 10 (Process Creation and Management)\nWhat about character arrays?\n\n```\n#!c\nint MPI_Open_port(MPI_Info info, char *port_name)\n```\nto \n```\n#!c\nint MPI_Open_port(MPI_Info info, char port_name[])\n```\nand\n```\n#!c\nvoid MPI::Open_port(const MPI::Info& info, char* port_name)\n```\nto \n```\n#!c\nvoid MPI::Open_port(const MPI::Info& info, char port_name[])\n```\n\n\n```\n#!c\nint MPI_Lookup_name(char *service_name, MPI_Info info, char *port_name)\n```\nto \n```\n#!c\nint MPI_Lookup_name(char *service_name, MPI_Info info, char port_name[])\n```\n\nand \n\n```\n#!c\nvoid MPI::Lookup_name(const char* service_name, const MPI::Info& info, char* port_name)\n```\nto \n```\n#!c\nvoid MPI::Lookup_name(const char* service_name, const MPI::Info& info, char port_name[])\n```\n\n\n### Chapter 11 (One-sided)\n\nNo issues\n\n### Chapter 12 (External Interfaces)\n\nNo issues\n\n### Chapter 13 (I/O)\n### Chapter 14 (Profiling Interface)\n\nNo issues\n\n### Chapter 15 (Deprecated Functions)\n### Chapter 16 (Language Bindings)\n\nNo issues.\n\n## Impact on Implementations\n\n## Impact on Applications / Users\n\n## Alternative Solutions\n\n## Entry for the Change Log\n\n"},"comment":{"url":"https://api.github.com/repos/mpiforumbot/mpi-forum-historic/issues/comments/234766644","html_url":"https://github.com/mpiforumbot/mpi-forum-historic/issues/125#issuecomment-234766644","issue_url":"https://api.github.com/repos/mpiforumbot/mpi-forum-historic/issues/125","id":234766644,"user":{"login":"mpiforumbot","id":20617676,"avatar_url":"https://avatars.githubusercontent.com/u/20617676?v=3","gravatar_id":"","url":"https://api.github.com/users/mpiforumbot","html_url":"https://github.com/mpiforumbot","followers_url":"https://api.github.com/users/mpiforumbot/followers","following_url":"https://api.github.com/users/mpiforumbot/following{/other_user}","gists_url":"https://api.github.com/users/mpiforumbot/gists{/gist_id}","starred_url":"https://api.github.com/users/mpiforumbot/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mpiforumbot/subscriptions","organizations_url":"https://api.github.com/users/mpiforumbot/orgs","repos_url":"https://api.github.com/users/mpiforumbot/repos","events_url":"https://api.github.com/users/mpiforumbot/events{/privacy}","received_events_url":"https://api.github.com/users/mpiforumbot/received_events","type":"User","site_admin":false},"created_at":"2016-07-24T09:16:22Z","updated_at":"2016-07-24T09:16:22Z","body":"**Originally by _htor_ on 2012-07-03 22:33:40 -0500**\n\n___\nImplemented in approved/topol.\n\nTorsten"}}
{"actor":{"display_login":"mpiforumbot"}}