{"action":"created","issue":{"url":"https://api.github.com/repos/servo/html5ever/issues/203","repository_url":"https://api.github.com/repos/servo/html5ever","labels_url":"https://api.github.com/repos/servo/html5ever/issues/203/labels{/name}","comments_url":"https://api.github.com/repos/servo/html5ever/issues/203/comments","events_url":"https://api.github.com/repos/servo/html5ever/issues/203/events","html_url":"https://github.com/servo/html5ever/issues/203","id":143239400,"number":203,"title":"C API references non-existing \"string_cache.h\"","user":{"login":"RReverser","id":557590,"avatar_url":"https://avatars.githubusercontent.com/u/557590?v=3","gravatar_id":"","url":"https://api.github.com/users/RReverser","html_url":"https://github.com/RReverser","followers_url":"https://api.github.com/users/RReverser/followers","following_url":"https://api.github.com/users/RReverser/following{/other_user}","gists_url":"https://api.github.com/users/RReverser/gists{/gist_id}","starred_url":"https://api.github.com/users/RReverser/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/RReverser/subscriptions","organizations_url":"https://api.github.com/users/RReverser/orgs","repos_url":"https://api.github.com/users/RReverser/repos","events_url":"https://api.github.com/users/RReverser/events{/privacy}","received_events_url":"https://api.github.com/users/RReverser/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2016-03-24T13:07:21Z","updated_at":"2016-03-24T13:57:24Z","closed_at":null,"body":"`html5ever.h` [references](https://github.com/servo/html5ever/blob/master/capi/include/html5ever.h#L15) `string_cache.h` although that definition file can't be found neither in this repo nor in https://github.com/servo/string-cache.\r\n\r\nAlso, https://github.com/servo/html5ever/blob/master/examples/capi/tokenize.c example uses `struct h5e_buf` in place of `scache_atom`s in `html5ever.h` and undefined function `h5e_buf_from_cstr`, so I suspect header is just outdated and needs to define new structure instead of including `string_cache.h`.\r\n\r\n**UPD**: Actually, if I understand https://github.com/servo/html5ever/blob/master/capi/src/tokenizer.rs#L121 correctly, those functions accept just Tendrils (instead of `scache_atom` / `h5e_buf`), so it should be possible to change definitions like following:\r\n\r\n```c\r\nvoid (*do_start_tag)(void *user, tendril name, int self_closing, size_t num_attrs);\r\nvoid (*do_tag_attr)(void *user, tendril name, tendril value);\r\nvoid (*do_end_tag)(void *user, tendril name);\r\n```\r\n\r\nand then code like this would work:\r\n\r\n```c\r\ntendril buf = TENDRIL_INIT;\r\ntendril_push_buffer(&buf, html, size);\r\nh5e_tokenizer_feed(tok, buf);\r\n```\r\n\r\n(although I might be wrong)"},"comment":{"url":"https://api.github.com/repos/servo/html5ever/issues/comments/200848017","html_url":"https://github.com/servo/html5ever/issues/203#issuecomment-200848017","issue_url":"https://api.github.com/repos/servo/html5ever/issues/203","id":200848017,"user":{"login":"Ygg01","id":1146204,"avatar_url":"https://avatars.githubusercontent.com/u/1146204?v=3","gravatar_id":"","url":"https://api.github.com/users/Ygg01","html_url":"https://github.com/Ygg01","followers_url":"https://api.github.com/users/Ygg01/followers","following_url":"https://api.github.com/users/Ygg01/following{/other_user}","gists_url":"https://api.github.com/users/Ygg01/gists{/gist_id}","starred_url":"https://api.github.com/users/Ygg01/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Ygg01/subscriptions","organizations_url":"https://api.github.com/users/Ygg01/orgs","repos_url":"https://api.github.com/users/Ygg01/repos","events_url":"https://api.github.com/users/Ygg01/events{/privacy}","received_events_url":"https://api.github.com/users/Ygg01/received_events","type":"User","site_admin":false},"created_at":"2016-03-24T13:57:24Z","updated_at":"2016-03-24T13:57:24Z","body":"kmcallister (kmc) worked on it, before they left. It was here at  https://github.com/servo/string-cache/pull/84 but was never merged with string-cache master. It has bitroted somewhat (May 2015). \r\n\r\nI'm willing to work on it, tomorrow, since it's one of the rare days I have time to commit to it.\r\n\r\n"}}