{"payload":{"feedbackUrl":"https://github.com/orgs/community/discussions/53140","repo":{"id":605339559,"defaultBranch":"trunk","name":"carbon-lang","ownerLogin":"nnesetto","currentUserCanPush":false,"isFork":true,"isEmpty":false,"createdAt":"2023-02-23T00:21:23.000Z","ownerAvatar":"https://avatars.githubusercontent.com/u/125788694?v=4","public":true,"private":false,"isOrgOwned":false},"refInfo":{"name":"","listCacheKey":"v0:1677111689.6371992","currentOid":""},"activityList":{"items":[{"before":"8ad08e34e2b4da15c778ff5db60023123577b085","after":"4ae0fa6f8621e3f114edd2df8d738580f2ba44cc","ref":"refs/heads/trunk","pushedAt":"2023-08-19T22:22:58.000Z","pushType":"push","commitsCount":219,"pusher":{"login":"nnesetto","name":"ittercho","path":"/nnesetto","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/125788694?s=80&v=4"},"commit":{"message":"Adjust handling of cases where conditions are missing. (#3119)\n\nIn #3064, code was changed to look at a future token. This is an issue\nbecause the parser is set up to enforce that tokens aren't used without\nbeing consumed. That's part of #3118; related validation fails. Also,\nsince it's not necessarily the open paren that was consumed, it could be\na different opening symbol, which the closing symbol handling doesn't\ncheck.\n\nUnder this approach, it's tracked whether an open paren was consumed,\nand the open paren is associated with the state. That's more aligned\nwith how the parser expects to be fed information.\n\nIn paren condition handling for if and while, I'm also adding some\nspecial casing for `if {` in particular to not assume the `{` is a\nstruct. I just think that this will come up somewhat often and the\nresulting output is better this way (an error either way). I'm not doing\nsimilar with `for` because there's already some `var` handling there,\nand I'd need a little more time to think about structure -- whereas\nright now I'm just trying to fix the crashes (`if {}`, `if []`, etc).\n\nFixes #3118","shortMessageHtmlLink":"Adjust handling of cases where conditions are missing. (carbon-langua…"}},{"before":"681dcfdcefc1292ced16323ba74c50592e8f5c23","after":"8ad08e34e2b4da15c778ff5db60023123577b085","ref":"refs/heads/trunk","pushedAt":"2023-05-22T09:00:12.640Z","pushType":"push","commitsCount":18,"pusher":{"login":"nnesetto","name":"ittercho","path":"/nnesetto","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/125788694?s=80&v=4"},"commit":{"message":"Refactor diagnostics out of parser_context.h (#2834)\n\nThis is addressing an issue left behind by the context switch, removing a few diagnostics that had been in the header rather than figuring out proper homes. I'm splitting one for semis a little further, sharing one, and then the other two are actually able to be moved into more specific homes as-is (one is only used in one place, clearly an oversight that it wasn't there already).","shortMessageHtmlLink":"Refactor diagnostics out of parser_context.h (carbon-language#2834)"}},{"before":"81a7adc0e5115b3399fb4803572c7617ada07ee2","after":"681dcfdcefc1292ced16323ba74c50592e8f5c23","ref":"refs/heads/trunk","pushedAt":"2023-05-09T07:01:02.531Z","pushType":"push","commitsCount":34,"pusher":{"login":"nnesetto","name":"ittercho","path":"/nnesetto","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/125788694?s=80&v=4"},"commit":{"message":"Fix local and global uninitialized array access (#2816)\n\n* Fix uninitialized array access/initialization\r\n\r\n* Handle printing uninitialized value\r\n\r\n* Fix error message directory name","shortMessageHtmlLink":"Fix local and global uninitialized array access (carbon-language#2816)"}},{"before":"77d38a8350d3b7270805c9a7460ee951e34790c6","after":"81a7adc0e5115b3399fb4803572c7617ada07ee2","ref":"refs/heads/trunk","pushedAt":"2023-04-11T05:50:03.533Z","pushType":"push","commitsCount":57,"pusher":{"login":"nnesetto","name":"ittercho","path":"/nnesetto","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/125788694?s=80&v=4"},"commit":{"message":"Fix handling of invalid struct entries (#2755)\n\nThis was incorrectly setting the finish state back to unknown, which resulted in a check. Instead, the finish state should be maintained.","shortMessageHtmlLink":"Fix handling of invalid struct entries (carbon-language#2755)"}},{"before":"c8141b59d97d4454c16b84d4a0ec1007035b15fb","after":"77d38a8350d3b7270805c9a7460ee951e34790c6","ref":"refs/heads/trunk","pushedAt":"2023-03-09T00:37:06.769Z","pushType":"push","commitsCount":10,"pusher":{"login":"nnesetto","name":"ittercho","path":"/nnesetto","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/125788694?s=80&v=4"},"commit":{"message":"Replace the map of lowered nodes with an array. (#2661)\n\nThis is something we'd discussed. I added a TODO that we may want to eventually make this a map, but I remain uncertain and think it's not something that's going to really cost us if we end up switching back. In the meantime, I think this approach does offer simplicity. As discussed too, the performance overhead of a map may ultimately not be worthwhile here versus the relative memory costs.","shortMessageHtmlLink":"Replace the map of lowered nodes with an array. (carbon-language#2661)"}}],"hasNextPage":false,"hasPreviousPage":false,"activityType":"all","actor":null,"timePeriod":"all","sort":"DESC","perPage":30,"cursor":"djE6ks8AAAADbkE9XgA","startCursor":null,"endCursor":null}},"title":"Activity · nnesetto/carbon-lang"}