{"action":"created","issue":{"url":"https://api.github.com/repos/SuperHouse/esp-open-rtos/issues/195","repository_url":"https://api.github.com/repos/SuperHouse/esp-open-rtos","labels_url":"https://api.github.com/repos/SuperHouse/esp-open-rtos/issues/195/labels{/name}","comments_url":"https://api.github.com/repos/SuperHouse/esp-open-rtos/issues/195/comments","events_url":"https://api.github.com/repos/SuperHouse/esp-open-rtos/issues/195/events","html_url":"https://github.com/SuperHouse/esp-open-rtos/pull/195","id":172055127,"number":195,"title":"sysparam: rework.","user":{"login":"ourairquality","id":20396158,"avatar_url":"https://avatars.githubusercontent.com/u/20396158?v=3","gravatar_id":"","url":"https://api.github.com/users/ourairquality","html_url":"https://github.com/ourairquality","followers_url":"https://api.github.com/users/ourairquality/followers","following_url":"https://api.github.com/users/ourairquality/following{/other_user}","gists_url":"https://api.github.com/users/ourairquality/gists{/gist_id}","starred_url":"https://api.github.com/users/ourairquality/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ourairquality/subscriptions","organizations_url":"https://api.github.com/users/ourairquality/orgs","repos_url":"https://api.github.com/users/ourairquality/repos","events_url":"https://api.github.com/users/ourairquality/events{/privacy}","received_events_url":"https://api.github.com/users/ourairquality/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":1,"created_at":"2016-08-19T05:04:28Z","updated_at":"2016-08-21T06:23:50Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/SuperHouse/esp-open-rtos/pulls/195","html_url":"https://github.com/SuperHouse/esp-open-rtos/pull/195","diff_url":"https://github.com/SuperHouse/esp-open-rtos/pull/195.diff","patch_url":"https://github.com/SuperHouse/esp-open-rtos/pull/195.patch"},"body":"Here's my wip changes to the sysparam code fwiw:\r\n\r\nAdds a semaphore used by readers are writers.\r\n\r\nFixes writing to the flash from constant data stored in the flash, using a bounce buffer.\r\n\r\nRemoved memory allocation from most read and write paths. Only read paths that return a blob of data allocate memory now, and the iterator.\r\n\r\nStore small integers as binary values, avoiding parsing and formatting in these paths."},"comment":{"url":"https://api.github.com/repos/SuperHouse/esp-open-rtos/issues/comments/241241214","html_url":"https://github.com/SuperHouse/esp-open-rtos/pull/195#issuecomment-241241214","issue_url":"https://api.github.com/repos/SuperHouse/esp-open-rtos/issues/195","id":241241214,"user":{"login":"ourairquality","id":20396158,"avatar_url":"https://avatars.githubusercontent.com/u/20396158?v=3","gravatar_id":"","url":"https://api.github.com/users/ourairquality","html_url":"https://github.com/ourairquality","followers_url":"https://api.github.com/users/ourairquality/followers","following_url":"https://api.github.com/users/ourairquality/following{/other_user}","gists_url":"https://api.github.com/users/ourairquality/gists{/gist_id}","starred_url":"https://api.github.com/users/ourairquality/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ourairquality/subscriptions","organizations_url":"https://api.github.com/users/ourairquality/orgs","repos_url":"https://api.github.com/users/ourairquality/repos","events_url":"https://api.github.com/users/ourairquality/events{/privacy}","received_events_url":"https://api.github.com/users/ourairquality/received_events","type":"User","site_admin":false},"created_at":"2016-08-21T06:23:50Z","updated_at":"2016-08-21T06:23:50Z","body":"There are dependent overlapping changes here, that is why I gave up on smaller patches. I doubt I have time to split it all up and incrementally get this from A to B, it's just my working version fwiw.\r\n\r\nThis set does design for small size and safety, probably at the expense of performance, for example it uses small stack allocated bounce buffers which allows it to avoid dynamic memory allocation in most paths, and this in turn allowed other simplifications."}}
{"actor":{"display_login":"ourairquality"}}