{"action":"created","issue":{"url":"https://api.github.com/repos/varnishcache/varnish-cache/issues/2100","repository_url":"https://api.github.com/repos/varnishcache/varnish-cache","labels_url":"https://api.github.com/repos/varnishcache/varnish-cache/issues/2100/labels{/name}","comments_url":"https://api.github.com/repos/varnishcache/varnish-cache/issues/2100/comments","events_url":"https://api.github.com/repos/varnishcache/varnish-cache/issues/2100/events","html_url":"https://github.com/varnishcache/varnish-cache/pull/2100","id":178872830,"number":2100,"title":"RFC: add va_args interface to varargisch vrt api functions","user":{"login":"nigoroll","id":1528104,"avatar_url":"https://avatars.githubusercontent.com/u/1528104?v=3","gravatar_id":"","url":"https://api.github.com/users/nigoroll","html_url":"https://github.com/nigoroll","followers_url":"https://api.github.com/users/nigoroll/followers","following_url":"https://api.github.com/users/nigoroll/following{/other_user}","gists_url":"https://api.github.com/users/nigoroll/gists{/gist_id}","starred_url":"https://api.github.com/users/nigoroll/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/nigoroll/subscriptions","organizations_url":"https://api.github.com/users/nigoroll/orgs","repos_url":"https://api.github.com/users/nigoroll/repos","events_url":"https://api.github.com/users/nigoroll/events{/privacy}","received_events_url":"https://api.github.com/users/nigoroll/received_events","type":"User","site_admin":false},"labels":[{"id":268918104,"url":"https://api.github.com/repos/varnishcache/varnish-cache/labels/enhancement","name":"enhancement","color":"84b6eb","default":true},{"id":357752916,"url":"https://api.github.com/repos/varnishcache/varnish-cache/labels/master","name":"master","color":"0052cc","default":false},{"id":357753553,"url":"https://api.github.com/repos/varnishcache/varnish-cache/labels/vmod","name":"vmod","color":"d93f0b","default":false}],"state":"closed","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":2,"created_at":"2016-09-23T13:08:10Z","updated_at":"2016-10-31T12:48:08Z","closed_at":"2016-10-31T12:48:08Z","pull_request":{"url":"https://api.github.com/repos/varnishcache/varnish-cache/pulls/2100","html_url":"https://github.com/varnishcache/varnish-cache/pull/2100","diff_url":"https://github.com/varnishcache/varnish-cache/pull/2100.diff","patch_url":"https://github.com/varnishcache/varnish-cache/pull/2100.patch"},"body":"currently, vmods cannot use the varargish VRT functions for own varargs because we do not yet have the `va_list` variants.\n\nRFC, some thoughts about this:\n- this would be the first `#include` in `vrt.h`. I understand we want to avoid creating additional dependencies, but we require the C compiler, so can't we also require some basic system includes?\n  - if we wanted to avoid this, starting another `vrt_vmod.h` would be an option\n- we'd not need to wrap all the `VRT_l_*_*()` functions, we could also export `vrt_do_string`. For now I think that the first option may be POLA  though\n  comments?\n"},"comment":{"url":"https://api.github.com/repos/varnishcache/varnish-cache/issues/comments/257285004","html_url":"https://github.com/varnishcache/varnish-cache/pull/2100#issuecomment-257285004","issue_url":"https://api.github.com/repos/varnishcache/varnish-cache/issues/2100","id":257285004,"user":{"login":"nigoroll","id":1528104,"avatar_url":"https://avatars.githubusercontent.com/u/1528104?v=3","gravatar_id":"","url":"https://api.github.com/users/nigoroll","html_url":"https://github.com/nigoroll","followers_url":"https://api.github.com/users/nigoroll/followers","following_url":"https://api.github.com/users/nigoroll/following{/other_user}","gists_url":"https://api.github.com/users/nigoroll/gists{/gist_id}","starred_url":"https://api.github.com/users/nigoroll/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/nigoroll/subscriptions","organizations_url":"https://api.github.com/users/nigoroll/orgs","repos_url":"https://api.github.com/users/nigoroll/repos","events_url":"https://api.github.com/users/nigoroll/events{/privacy}","received_events_url":"https://api.github.com/users/nigoroll/received_events","type":"User","site_admin":false},"created_at":"2016-10-31T12:48:08Z","updated_at":"2016-10-31T12:48:08Z","body":"irc discussion: @bsdphk proposed a new VRT STRING_LIST type to replace both the varargish and va interfaces. To transition, the current vrt interface could wrap to the new interfaces and with vrt 6.0 we could change (break) the interface."}}
{"actor":{"display_login":"nigoroll"}}