{"action":"created","issue":{"url":"https://api.github.com/repos/zeux/pugixml/issues/53","labels_url":"https://api.github.com/repos/zeux/pugixml/issues/53/labels{/name}","comments_url":"https://api.github.com/repos/zeux/pugixml/issues/53/comments","events_url":"https://api.github.com/repos/zeux/pugixml/issues/53/events","html_url":"https://github.com/zeux/pugixml/issues/53","id":104343563,"number":53,"title":"Enable long long by default (needed by filezilla)","user":{"login":"kwizart","id":1677957,"avatar_url":"https://avatars.githubusercontent.com/u/1677957?v=3","gravatar_id":"","url":"https://api.github.com/users/kwizart","html_url":"https://github.com/kwizart","followers_url":"https://api.github.com/users/kwizart/followers","following_url":"https://api.github.com/users/kwizart/following{/other_user}","gists_url":"https://api.github.com/users/kwizart/gists{/gist_id}","starred_url":"https://api.github.com/users/kwizart/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/kwizart/subscriptions","organizations_url":"https://api.github.com/users/kwizart/orgs","repos_url":"https://api.github.com/users/kwizart/repos","events_url":"https://api.github.com/users/kwizart/events{/privacy}","received_events_url":"https://api.github.com/users/kwizart/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":7,"created_at":"2015-09-01T20:04:13Z","updated_at":"2015-09-07T18:27:40Z","closed_at":null,"body":"Currently the pugixml.hpp doesn't enable long long by default, as a result the fedora package has it disabled. For this reason the filezilla package (which requires the long long feature) cannot use the system pugixml.\r\n\r\nAs a packager, we are often are reluctant to tweak upstream default options, so it would be easier if you can reword the condition to enable it by default, so pugixml will be available for in distributions as fully featured."},"comment":{"url":"https://api.github.com/repos/zeux/pugixml/issues/comments/138353643","html_url":"https://github.com/zeux/pugixml/issues/53#issuecomment-138353643","issue_url":"https://api.github.com/repos/zeux/pugixml/issues/53","id":138353643,"user":{"login":"zeux","id":1106629,"avatar_url":"https://avatars.githubusercontent.com/u/1106629?v=3","gravatar_id":"","url":"https://api.github.com/users/zeux","html_url":"https://github.com/zeux","followers_url":"https://api.github.com/users/zeux/followers","following_url":"https://api.github.com/users/zeux/following{/other_user}","gists_url":"https://api.github.com/users/zeux/gists{/gist_id}","starred_url":"https://api.github.com/users/zeux/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/zeux/subscriptions","organizations_url":"https://api.github.com/users/zeux/orgs","repos_url":"https://api.github.com/users/zeux/repos","events_url":"https://api.github.com/users/zeux/events{/privacy}","received_events_url":"https://api.github.com/users/zeux/received_events","type":"User","site_admin":false},"created_at":"2015-09-07T18:27:40Z","updated_at":"2015-09-07T18:27:40Z","body":"@jwakely C++11 is checked as a first thing; if it is supported I just assume long long is present and fully functional, including library functions.\r\n\r\nI know the standard guarantees but they do not necessarily hold. I've seen platforms with wchar_t type being present but no standard (C++03!) wchar_t functions being available. I would not be surprised to see a platform that advertises C99 support but %lld support is accidentally compiled out of printf code or long long codegen generates an ICE. Regardless, it's impossible to check for C99 support from C++ code... which is why originally I enabled long long only for C++11 or MSVC 2008+ where I trust the CRT more."}}