{"action":"created","issue":{"url":"https://api.github.com/repos/chapel-lang/chapel/issues/2741","labels_url":"https://api.github.com/repos/chapel-lang/chapel/issues/2741/labels{/name}","comments_url":"https://api.github.com/repos/chapel-lang/chapel/issues/2741/comments","events_url":"https://api.github.com/repos/chapel-lang/chapel/issues/2741/events","html_url":"https://github.com/chapel-lang/chapel/pull/2741","id":110053143,"number":2741,"title":"Fix fielderator bug","user":{"login":"mppf","id":3653132,"avatar_url":"https://avatars.githubusercontent.com/u/3653132?v=3","gravatar_id":"","url":"https://api.github.com/users/mppf","html_url":"https://github.com/mppf","followers_url":"https://api.github.com/users/mppf/followers","following_url":"https://api.github.com/users/mppf/following{/other_user}","gists_url":"https://api.github.com/users/mppf/gists{/gist_id}","starred_url":"https://api.github.com/users/mppf/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mppf/subscriptions","organizations_url":"https://api.github.com/users/mppf/orgs","repos_url":"https://api.github.com/users/mppf/repos","events_url":"https://api.github.com/users/mppf/events{/privacy}","received_events_url":"https://api.github.com/users/mppf/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":2,"created_at":"2015-10-06T16:57:12Z","updated_at":"2015-10-06T17:21:12Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/chapel-lang/chapel/pulls/2741","html_url":"https://github.com/chapel-lang/chapel/pull/2741","diff_url":"https://github.com/chapel-lang/chapel/pull/2741.diff","patch_url":"https://github.com/chapel-lang/chapel/pull/2741.patch"},"body":"The fielderator was not ignoring param fields, as a\r\nresult, I was running into some strange \"not an lvalue\"\r\nerrors when trying to use it on string-as-rec.\r\n\r\nThe isNormalField call just needed to check for FLAG_PARAM.\r\nAlso updates fielderator.chpl to check that this works correctly.\r\n\r\n- [ ] local testing"},"comment":{"url":"https://api.github.com/repos/chapel-lang/chapel/issues/comments/145932997","html_url":"https://github.com/chapel-lang/chapel/pull/2741#issuecomment-145932997","issue_url":"https://api.github.com/repos/chapel-lang/chapel/issues/2741","id":145932997,"user":{"login":"mppf","id":3653132,"avatar_url":"https://avatars.githubusercontent.com/u/3653132?v=3","gravatar_id":"","url":"https://api.github.com/users/mppf","html_url":"https://github.com/mppf","followers_url":"https://api.github.com/users/mppf/followers","following_url":"https://api.github.com/users/mppf/following{/other_user}","gists_url":"https://api.github.com/users/mppf/gists{/gist_id}","starred_url":"https://api.github.com/users/mppf/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mppf/subscriptions","organizations_url":"https://api.github.com/users/mppf/orgs","repos_url":"https://api.github.com/users/mppf/repos","events_url":"https://api.github.com/users/mppf/events{/privacy}","received_events_url":"https://api.github.com/users/mppf/received_events","type":"User","site_admin":false},"created_at":"2015-10-06T17:21:12Z","updated_at":"2015-10-06T17:21:12Z","body":"@bradcray, could you have a look at this patch? Besides the tiny code change, I'm wondering if we want param fields to print out by default.\r\n\r\nrecord R { param p = 1; }\r\n\r\nwriteln(new R(1));\r\n\r\ndoes it print out () or (p = 1) ?\r\n\r\nRight now it prints out p = 1, but I'm not sure that makes sense. (It does not\r\nprint out a type name, for example).\r\n\r\nIf we want to keep it that way, I'll need a way to know if a field is a param field\r\nfrom module code. (in order to resolve the issue I was running into on string-as-rec)."}}