Kirill Bobyrev
Kirill Bobyrev
Sounds like a good compliment to LLVM's ["Use early exits"](https://llvm.org/docs/CodingStandards.html#use-early-exits-and-continue-to-simplify-code) guideline. Probably the best way to do this is a Clang-Tidy check.
> Is there a clang-tidy equivilent to this? Rmoving unused includes (and enforcing their removal in CI) seems like something I would expect to see there, but I haven't been...
@jmooring why does that patch address this issue? IIUC author is asking for Google Analytics 4 support in `_async` version, not the regular one. Unless I am mistaken, linked PR...
@jmooring oh, actually, I think you are right, sorry, I think I didn't read the code carefully enough. The problem for me was that [PageSpeed](https://pagespeed.web.dev/) for mobile shows that the...
@sam-mccall great job, congrats to everyone on the team!
While it looks like a clearly useful feature, I think adding support for a single snippet creates a precedent that might lead to the argument for more in the future....
Also, VS Code [supports](https://code.visualstudio.com/docs/editor/userdefinedsnippets) snippets out of the box and it's really easy to add custom ones. There are multiple plugins providing a comprehensive set of snippets for selected language...
Also, I think `return 0;` should be redundant as it's always executed if control reaches the end of `main`.
Seems to be fixed, makes sense to close.
The new GitHub UI makes me miss the button every time 😆 Thanks, @HighCommander4!