{"action":"created","issue":{"url":"https://api.github.com/repos/stan-dev/stan/issues/1509","labels_url":"https://api.github.com/repos/stan-dev/stan/issues/1509/labels{/name}","comments_url":"https://api.github.com/repos/stan-dev/stan/issues/1509/comments","events_url":"https://api.github.com/repos/stan-dev/stan/issues/1509/events","html_url":"https://github.com/stan-dev/stan/pull/1509","id":90038492,"number":1509,"title":"Bugfix/issue 1506 intel 14 parser","user":{"login":"bob-carpenter","id":3383807,"avatar_url":"https://avatars.githubusercontent.com/u/3383807?v=3","gravatar_id":"","url":"https://api.github.com/users/bob-carpenter","html_url":"https://github.com/bob-carpenter","followers_url":"https://api.github.com/users/bob-carpenter/followers","following_url":"https://api.github.com/users/bob-carpenter/following{/other_user}","gists_url":"https://api.github.com/users/bob-carpenter/gists{/gist_id}","starred_url":"https://api.github.com/users/bob-carpenter/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/bob-carpenter/subscriptions","organizations_url":"https://api.github.com/users/bob-carpenter/orgs","repos_url":"https://api.github.com/users/bob-carpenter/repos","events_url":"https://api.github.com/users/bob-carpenter/events{/privacy}","received_events_url":"https://api.github.com/users/bob-carpenter/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/stan-dev/stan/labels/code%20review%20complete","name":"code review complete","color":"009800"}],"state":"open","locked":false,"assignee":null,"milestone":{"url":"https://api.github.com/repos/stan-dev/stan/milestones/16","html_url":"https://github.com/stan-dev/stan/milestones/v2.7.0","labels_url":"https://api.github.com/repos/stan-dev/stan/milestones/16/labels","id":1026323,"number":16,"title":"v2.7.0","description":"Next version after v2.6.3.","creator":{"login":"syclik","id":425751,"avatar_url":"https://avatars.githubusercontent.com/u/425751?v=3","gravatar_id":"","url":"https://api.github.com/users/syclik","html_url":"https://github.com/syclik","followers_url":"https://api.github.com/users/syclik/followers","following_url":"https://api.github.com/users/syclik/following{/other_user}","gists_url":"https://api.github.com/users/syclik/gists{/gist_id}","starred_url":"https://api.github.com/users/syclik/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/syclik/subscriptions","organizations_url":"https://api.github.com/users/syclik/orgs","repos_url":"https://api.github.com/users/syclik/repos","events_url":"https://api.github.com/users/syclik/events{/privacy}","received_events_url":"https://api.github.com/users/syclik/received_events","type":"User","site_admin":false},"open_issues":1,"closed_issues":116,"state":"open","created_at":"2015-03-17T22:51:26Z","updated_at":"2015-06-22T08:05:10Z","due_on":null,"closed_at":null},"comments":7,"created_at":"2015-06-22T08:05:10Z","updated_at":"2015-06-22T21:19:08Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/stan-dev/stan/pulls/1509","html_url":"https://github.com/stan-dev/stan/pull/1509","diff_url":"https://github.com/stan-dev/stan/pull/1509.diff","patch_url":"https://github.com/stan-dev/stan/pull/1509.patch"},"body":"#### Summary:\r\n\r\n* Remove all uses of lazy evaluation of assignment (operator=) from semantic actions in the grammar\r\n\r\n* ifdef out the offending bit of code in the Boost lib for the Intel compiler (yes, this involved changing Boost);  see  https://software.intel.com/en-us/articles/boost-compiler-error-class-has-no-member-type\r\n\r\n* remove all the remaining lint from the grammars\r\n\r\n#### Intended Effect:\r\n\r\nAllow Stan to be compiled with Intel 14 compiler.\r\n\r\n#### How to Verify:\r\n\r\nVerifying the intended effect requires an Intel compiler, but it should be enough to verify that it passes unit tests.  No behavior should have changed --- this is purely refactoring.\r\n\r\n#### Side Effects:\r\n\r\nThat definition of `operator=` is now invisible for clients using our copy of Boost.  This is very unlikely to be a problem.\r\n\r\n#### Documentation:\r\n\r\nNothing user facing.\r\n\r\n#### Reviewer Suggestions:\r\n\r\nAnyone."},"comment":{"url":"https://api.github.com/repos/stan-dev/stan/issues/comments/114264288","html_url":"https://github.com/stan-dev/stan/pull/1509#issuecomment-114264288","issue_url":"https://api.github.com/repos/stan-dev/stan/issues/1509","id":114264288,"user":{"login":"bob-carpenter","id":3383807,"avatar_url":"https://avatars.githubusercontent.com/u/3383807?v=3","gravatar_id":"","url":"https://api.github.com/users/bob-carpenter","html_url":"https://github.com/bob-carpenter","followers_url":"https://api.github.com/users/bob-carpenter/followers","following_url":"https://api.github.com/users/bob-carpenter/following{/other_user}","gists_url":"https://api.github.com/users/bob-carpenter/gists{/gist_id}","starred_url":"https://api.github.com/users/bob-carpenter/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/bob-carpenter/subscriptions","organizations_url":"https://api.github.com/users/bob-carpenter/orgs","repos_url":"https://api.github.com/users/bob-carpenter/repos","events_url":"https://api.github.com/users/bob-carpenter/events{/privacy}","received_events_url":"https://api.github.com/users/bob-carpenter/received_events","type":"User","site_admin":false},"created_at":"2015-06-22T21:19:08Z","updated_at":"2015-06-22T21:19:08Z","body":"Correct.  \n\nBen's going to patch RStan so that it uses the same Boost\nheader.  As long as we make sure to search ours first, the header\nguard will silence the one from Boost.  At least I think that's\nBen's plan.\n\n- Bob\n\n> On Jun 22, 2015, at 5:12 PM, Daniel Lee <notifications@github.com> wrote:\n> \n> @bob-carpenter, everything looks good, but the change made to the boost library won't help Intel compile users with RStan, right? That will still rely on the BH package, which hasn't been patched.\n> \n> —\n> Reply to this email directly or view it on GitHub.\n> \n\n"}}