{"comment":{"url":"https://api.github.com/repos/datacratic/rtbkit/comments/11893846","html_url":"https://github.com/datacratic/rtbkit/commit/1c3811610a0260f1df3c87291b2a82985161fc07#commitcomment-11893846","id":11893846,"user":{"login":"oktal","id":859699,"avatar_url":"https://avatars.githubusercontent.com/u/859699?v=3","gravatar_id":"","url":"https://api.github.com/users/oktal","html_url":"https://github.com/oktal","followers_url":"https://api.github.com/users/oktal/followers","following_url":"https://api.github.com/users/oktal/following{/other_user}","gists_url":"https://api.github.com/users/oktal/gists{/gist_id}","starred_url":"https://api.github.com/users/oktal/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/oktal/subscriptions","organizations_url":"https://api.github.com/users/oktal/orgs","repos_url":"https://api.github.com/users/oktal/repos","events_url":"https://api.github.com/users/oktal/events{/privacy}","received_events_url":"https://api.github.com/users/oktal/received_events","type":"User","site_admin":false},"position":2,"line":49,"path":"rtbkit/core/router/filters/creative_filters.h","commit_id":"1c3811610a0260f1df3c87291b2a82985161fc07","created_at":"2015-06-27T01:56:05Z","updated_at":"2015-06-27T01:56:05Z","body":"Small detail but in the fast-path you don't even need to declare and allocate a CreativeMatrix, you could simply early return:\r\n\r\n```cpp\r\nif (imp.formats.empty()) return;\r\n\r\nCreativeMatrix creatives = get(Format(0, 0));\r\n...\r\n```"}}