envoy icon indicating copy to clipboard operation
envoy copied to clipboard

presubmit-check the use of DECLARE_FACTORY for each DEFINE_FACTORY

Open alyssawilk opened this issue 3 years ago • 2 comments

E-M apparently depends on the former and nothing checks on it upstream :-(

alyssawilk avatar Sep 19 '22 19:09 alyssawilk

cc @Augustyniak @snowp @phlax

alyssawilk avatar Sep 19 '22 19:09 alyssawilk

To make things a tad more complicated, EM only relies on DECLARE_FACTORY for things that need to be "force initialized", which is basically just a linker trick to get iOS to include the TUs. For stuff that uses a factory pattern in common code but doesn't rely the extension build system this may not be necessary

That said, enforcing this everywhere probably wouldn't hurt

snowp avatar Sep 19 '22 19:09 snowp

This issue has been automatically marked as stale because it has not had activity in the last 30 days. It will be closed in the next 7 days unless it is tagged "help wanted" or "no stalebot" or other activity occurs. Thank you for your contributions.

github-actions[bot] avatar Oct 19 '22 20:10 github-actions[bot]