{"action":"created","issue":{"url":"https://api.github.com/repos/SnabbCo/snabbswitch/issues/504","labels_url":"https://api.github.com/repos/SnabbCo/snabbswitch/issues/504/labels{/name}","comments_url":"https://api.github.com/repos/SnabbCo/snabbswitch/issues/504/comments","events_url":"https://api.github.com/repos/SnabbCo/snabbswitch/issues/504/events","html_url":"https://github.com/SnabbCo/snabbswitch/pull/504","id":80822052,"number":504,"title":"Vm sandbox","user":{"login":"javierguerragiraldez","id":147561,"avatar_url":"https://avatars.githubusercontent.com/u/147561?v=3","gravatar_id":"","url":"https://api.github.com/users/javierguerragiraldez","html_url":"https://github.com/javierguerragiraldez","followers_url":"https://api.github.com/users/javierguerragiraldez/followers","following_url":"https://api.github.com/users/javierguerragiraldez/following{/other_user}","gists_url":"https://api.github.com/users/javierguerragiraldez/gists{/gist_id}","starred_url":"https://api.github.com/users/javierguerragiraldez/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/javierguerragiraldez/subscriptions","organizations_url":"https://api.github.com/users/javierguerragiraldez/orgs","repos_url":"https://api.github.com/users/javierguerragiraldez/repos","events_url":"https://api.github.com/users/javierguerragiraldez/events{/privacy}","received_events_url":"https://api.github.com/users/javierguerragiraldez/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":2,"created_at":"2015-05-26T06:38:55Z","updated_at":"2015-05-26T14:44:43Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/SnabbCo/snabbswitch/pulls/504","html_url":"https://github.com/SnabbCo/snabbswitch/pull/504","diff_url":"https://github.com/SnabbCo/snabbswitch/pull/504.diff","patch_url":"https://github.com/SnabbCo/snabbswitch/pull/504.patch"},"body":"A cleaner and less disrupting version of what we've seen on previous 'style proposal' branches.\r\n\r\n- There's no 'general purpose' wrapping of the Lua C API, just enough methods to make `sandbox_loader.lua` source readable.\r\n\r\n- Doesn't depend on `app.lua` calling `app:relink()`. Instead, adds an explicit `:inject_links()` method.\r\n\r\n- Cleaner method to share the packet freelist.\r\n\r\n- The `engine.frees`, `.freebytes` and `.freebits` are moved to an `ipc.shmem.counters` file, allowing any sandbox to update it.  It could also be an FFI struct shared in a similar manner to `packet_fl`. In a multithreaded future it could be a per-thread struct, updating the central statistics after each `breath()` cycle."},"comment":{"url":"https://api.github.com/repos/SnabbCo/snabbswitch/issues/comments/105549300","html_url":"https://github.com/SnabbCo/snabbswitch/pull/504#issuecomment-105549300","issue_url":"https://api.github.com/repos/SnabbCo/snabbswitch/issues/504","id":105549300,"user":{"login":"javierguerragiraldez","id":147561,"avatar_url":"https://avatars.githubusercontent.com/u/147561?v=3","gravatar_id":"","url":"https://api.github.com/users/javierguerragiraldez","html_url":"https://github.com/javierguerragiraldez","followers_url":"https://api.github.com/users/javierguerragiraldez/followers","following_url":"https://api.github.com/users/javierguerragiraldez/following{/other_user}","gists_url":"https://api.github.com/users/javierguerragiraldez/gists{/gist_id}","starred_url":"https://api.github.com/users/javierguerragiraldez/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/javierguerragiraldez/subscriptions","organizations_url":"https://api.github.com/users/javierguerragiraldez/orgs","repos_url":"https://api.github.com/users/javierguerragiraldez/repos","events_url":"https://api.github.com/users/javierguerragiraldez/events{/privacy}","received_events_url":"https://api.github.com/users/javierguerragiraldez/received_events","type":"User","site_admin":false},"created_at":"2015-05-26T14:44:43Z","updated_at":"2015-05-26T14:44:43Z","body":"> Looks like the file core/stats.lua is missing at the moment.\r\n\r\n[D'oh!](http://www.oed.com/view/Entry/249869)\r\n\r\ni'll push it at lunchtime..."}}