{"action":"created","issue":{"url":"https://api.github.com/repos/RobotLocomotion/drake/issues/2399","repository_url":"https://api.github.com/repos/RobotLocomotion/drake","labels_url":"https://api.github.com/repos/RobotLocomotion/drake/issues/2399/labels{/name}","comments_url":"https://api.github.com/repos/RobotLocomotion/drake/issues/2399/comments","events_url":"https://api.github.com/repos/RobotLocomotion/drake/issues/2399/events","html_url":"https://github.com/RobotLocomotion/drake/pull/2399","id":155726362,"number":2399,"title":"Fixed some more signed/unsigned compare warnings.","user":{"login":"liangfok","id":1388098,"avatar_url":"https://avatars.githubusercontent.com/u/1388098?v=3","gravatar_id":"","url":"https://api.github.com/users/liangfok","html_url":"https://github.com/liangfok","followers_url":"https://api.github.com/users/liangfok/followers","following_url":"https://api.github.com/users/liangfok/following{/other_user}","gists_url":"https://api.github.com/users/liangfok/gists{/gist_id}","starred_url":"https://api.github.com/users/liangfok/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/liangfok/subscriptions","organizations_url":"https://api.github.com/users/liangfok/orgs","repos_url":"https://api.github.com/users/liangfok/repos","events_url":"https://api.github.com/users/liangfok/events{/privacy}","received_events_url":"https://api.github.com/users/liangfok/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/RobotLocomotion/drake/labels/priority:%20low","name":"priority: low","color":"e99695"},{"url":"https://api.github.com/repos/RobotLocomotion/drake/labels/team:%20software%20core","name":"team: software core","color":"0052cc"},{"url":"https://api.github.com/repos/RobotLocomotion/drake/labels/type:%20cleanup","name":"type: cleanup","color":"009800"}],"state":"open","locked":false,"assignee":{"login":"jwnimmer-tri","id":17596505,"avatar_url":"https://avatars.githubusercontent.com/u/17596505?v=3","gravatar_id":"","url":"https://api.github.com/users/jwnimmer-tri","html_url":"https://github.com/jwnimmer-tri","followers_url":"https://api.github.com/users/jwnimmer-tri/followers","following_url":"https://api.github.com/users/jwnimmer-tri/following{/other_user}","gists_url":"https://api.github.com/users/jwnimmer-tri/gists{/gist_id}","starred_url":"https://api.github.com/users/jwnimmer-tri/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jwnimmer-tri/subscriptions","organizations_url":"https://api.github.com/users/jwnimmer-tri/orgs","repos_url":"https://api.github.com/users/jwnimmer-tri/repos","events_url":"https://api.github.com/users/jwnimmer-tri/events{/privacy}","received_events_url":"https://api.github.com/users/jwnimmer-tri/received_events","type":"User","site_admin":false},"milestone":null,"comments":4,"created_at":"2016-05-19T12:37:05Z","updated_at":"2016-05-19T15:10:11Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/RobotLocomotion/drake/pulls/2399","html_url":"https://github.com/RobotLocomotion/drake/pull/2399","diff_url":"https://github.com/RobotLocomotion/drake/pull/2399.diff","patch_url":"https://github.com/RobotLocomotion/drake/pull/2399.patch"},"body":"\n\n<!-- Reviewable:start -->\nThis change is [<img src=\"https://reviewable.io/review_button.svg\" height=\"35\" align=\"absmiddle\" alt=\"Reviewable\"/>](https://reviewable.io/reviews/robotlocomotion/drake/2399)\n<!-- Reviewable:end -->\n"},"comment":{"url":"https://api.github.com/repos/RobotLocomotion/drake/issues/comments/220354533","html_url":"https://github.com/RobotLocomotion/drake/pull/2399#issuecomment-220354533","issue_url":"https://api.github.com/repos/RobotLocomotion/drake/issues/2399","id":220354533,"user":{"login":"liangfok","id":1388098,"avatar_url":"https://avatars.githubusercontent.com/u/1388098?v=3","gravatar_id":"","url":"https://api.github.com/users/liangfok","html_url":"https://github.com/liangfok","followers_url":"https://api.github.com/users/liangfok/followers","following_url":"https://api.github.com/users/liangfok/following{/other_user}","gists_url":"https://api.github.com/users/liangfok/gists{/gist_id}","starred_url":"https://api.github.com/users/liangfok/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/liangfok/subscriptions","organizations_url":"https://api.github.com/users/liangfok/orgs","repos_url":"https://api.github.com/users/liangfok/repos","events_url":"https://api.github.com/users/liangfok/events{/privacy}","received_events_url":"https://api.github.com/users/liangfok/received_events","type":"User","site_admin":false},"created_at":"2016-05-19T15:10:11Z","updated_at":"2016-05-19T15:10:11Z","body":"Thanks. PTAL. Thanks!\n\n<details><summary><i>Previously, jwnimmer-tri (Jeremy Nimmer) wrote…</i></summary>\n> Yes.\n</details>\n\n\n---\n\nReview status: all files reviewed at latest revision, 1 unresolved discussion, some commit checks failed.\n\n---\n\n*[drake/systems/plants/RigidBodyTree.cpp, line 1731 \\[r1\\]](https://reviewable.io:443/reviews/robotlocomotion/drake/2399#-KI80T0ddNVlSr3WZQ5S:-KI8Z5FQPczYP_KRT4oF:-2132873309) ([raw file](https://github.com/robotlocomotion/drake/blob/bbf8a8034303ead0a1e9ac0bcfd9af9bd17a38ea/drake/systems/plants/RigidBodyTree.cpp#L1731)):*\n<details><summary><i>Previously, jwnimmer-tri (Jeremy Nimmer) wrote…</i></summary>\n> I see no reason why an extra boolean is needed here.  It seems identical to num_match, such that only one or the other should exist.\n</details>\nOnce I switched `num_match` to be a `size_t`, it cannot be negative. So I added a boolean variable to capture the situation where not match was found.\n\n---\n\n\n*Comments from [Reviewable](https://reviewable.io:443/reviews/robotlocomotion/drake/2399#-:-KI8ZEPZAExFRQnZK64B:-1142876745)*\n<!-- Sent from Reviewable.io -->\n"}}