GitHub PullRequestReviewCommentEvent at 2014-03-11 18:18:17
Repository: racc
Actor: zzak
Payload: {"comment":{"url":"https://api.github.com/repos/tenderlove/racc/pulls/comments/10487763","id":10487763,"diff_hunk":"@@ -175,39 +177,38 @@ preclow\n --\n \n The token written in the line closer to prechigh has the higher precedence.\n-You can also write this upside-down, that is, writing preclow above and prechigh\n-at the bottom.\n+You can also write this in reverse order, such as defining preclow before\n+prechigh at the bottom.\n \n Left, right and nonassoc represent associativity.\n-When a confliction occurs between the operators whose precedences are the same\n-their associativities are used to decide whether shift or reduce.\n+When a conflict occurs between operators whose precedence is equal, their\n+associativity is used to decide whether to shift or reduce.\n \n For instance, think of the following case:\n --\n a - b - c\n --\n \n-When - is left-associative, this is interpreted as follows:\n+When the \"-\" operator is left-associative this is interpreted as follows:\n --\n (a - b) - c\n --\n-Arithmetic operators are usually left-associative.\n+Note: arithmetic operators are usually left-associative.\n \n On the other hand,\n-if - is right-associative, it is interpreted as follows:\n+if the \"-\" operator is right-associative it is interpreted as follows:\n --\n a - (b - c)\n --\n \n When an operator is not allowed to be written in succession like this in the\n first place, it is nonassoc.\n-++ in C language and unary minus are nonassoc.\n+Note: in the C language \"++\", \"unary\" and \"minus\" are nonassoc.","path":"rdoc/en/grammar.en.rdoc","position":156,"original_position":156,"commit_id":"715621b2414e8aae1042492f761496830ec839d0","original_commit_id":"715621b2414e8aae1042492f761496830ec839d0","user":{"login":"zzak","id":277819,"avatar_url":"https://gravatar.com/avatar/7fe945668a4fc098e886e20dea71d2ee?d=https%3A%2F%2Fidenticons.github.com%2Ffa7c1d32672b3a269aaf69257901adcb.png&r=x","gravatar_id":"7fe945668a4fc098e886e20dea71d2ee","url":"https://api.github.com/users/zzak","html_url":"https://github.com/zzak","followers_url":"https://api.github.com/users/zzak/followers","following_url":"https://api.github.com/users/zzak/following{/other_user}","gists_url":"https://api.github.com/users/zzak/gists{/gist_id}","starred_url":"https://api.github.com/users/zzak/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/zzak/subscriptions","organizations_url":"https://api.github.com/users/zzak/orgs","repos_url":"https://api.github.com/users/zzak/repos","events_url":"https://api.github.com/users/zzak/events{/privacy}","received_events_url":"https://api.github.com/users/zzak/received_events","type":"User","site_admin":false},"body":"@ocha- Let me say, you’ve done a great job translating this documentation!\r\n\r\n“For instance,” sounds good.\r\n\r\nFeel free to submit any more patches and ping me when you’d like a review :)\r\n\r\n\r\nOn Mar 11, 2014, at 9:27 AM, ocha- <notifications@github.com> wrote:\r\n\r\n> In rdoc/en/grammar.en.rdoc:\r\n> \r\n> >  --\r\n> >  a - (b - c)\r\n> >  --\r\n> >  \r\n> >  When an operator is not allowed to be written in succession like this in the\r\n> >  first place, it is nonassoc.\r\n> > -++ in C language and unary minus are nonassoc.\r\n> > +Note: in the C language \"++\", \"unary\" and \"minus\" are nonassoc.\r\n> @zzak Then I'll take care of this. Probably I'll also edit the \"Options\" section to be more precise.\r\n> \r\n> I have one thing I'd like to ask your advice.\r\n> \r\n> \"Note:\" looks like different from my intention,\r\n> so if you don't mind I'd like to remove this.\r\n> \r\n> But if something is needed to be natural as English,\r\n> I'd like to use \"For instance,\" instead.\r\n> \r\n> How do you think?\r\n> \r\n> I'm sorry that I'm asking this just to improve my English skill.\r\n> \r\n> —\r\n> Reply to this email directly or view it on GitHub.\r\n>","created_at":"2014-03-11T18:18:17Z","updated_at":"2014-03-11T18:18:17Z","html_url":"https://github.com/tenderlove/racc/pull/47#discussion_r10487763","pull_request_url":"https://api.github.com/repos/tenderlove/racc/pulls/47","_links":{"self":{"href":"https://api.github.com/repos/tenderlove/racc/pulls/comments/10487763"},"html":{"href":"https://github.com/tenderlove/racc/pull/47#discussion_r10487763"},"pull_request":{"href":"https://api.github.com/repos/tenderlove/racc/pulls/47"}}}}