{"comment":{"url":"https://api.github.com/repos/elmart/neovim/comments/10989986","html_url":"https://github.com/elmart/neovim/commit/6725754df7778719f149ceecaf1e852931c4b4b3#commitcomment-10989986","id":10989986,"user":{"login":"oni-link","id":2077655,"avatar_url":"https://avatars.githubusercontent.com/u/2077655?v=3","gravatar_id":"","url":"https://api.github.com/users/oni-link","html_url":"https://github.com/oni-link","followers_url":"https://api.github.com/users/oni-link/followers","following_url":"https://api.github.com/users/oni-link/following{/other_user}","gists_url":"https://api.github.com/users/oni-link/gists{/gist_id}","starred_url":"https://api.github.com/users/oni-link/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/oni-link/subscriptions","organizations_url":"https://api.github.com/users/oni-link/orgs","repos_url":"https://api.github.com/users/oni-link/repos","events_url":"https://api.github.com/users/oni-link/events{/privacy}","received_events_url":"https://api.github.com/users/oni-link/received_events","type":"User","site_admin":false},"position":4,"line":5803,"path":"src/nvim/ex_docmd.c","commit_id":"6725754df7778719f149ceecaf1e852931c4b4b3","created_at":"2015-04-30T17:50:40Z","updated_at":"2015-04-30T17:50:40Z","body":"Moving the `assert()` down would still catch the `NULL` problem with `goto_tabpage()`, but it could also catch an error with `tabpage_close_other()`, so I would move the test to the dereference."}}