summaryrefslogtreecommitdiffstats
path: root/third_party/libwebrtc/api/README.md
diff options
context:
space:
mode:
Diffstat (limited to 'third_party/libwebrtc/api/README.md')
-rw-r--r--third_party/libwebrtc/api/README.md37
1 files changed, 37 insertions, 0 deletions
diff --git a/third_party/libwebrtc/api/README.md b/third_party/libwebrtc/api/README.md
new file mode 100644
index 0000000000..cf6d73a855
--- /dev/null
+++ b/third_party/libwebrtc/api/README.md
@@ -0,0 +1,37 @@
+<!-- go/cmark -->
+<!--* freshness: {owner: 'hta' reviewed: '2021-01-01'} *-->
+
+# How to write code in the `api/` directory
+
+Mostly, just follow the regular [style guide](/g3doc/style-guide.md), but:
+
+* Note that `api/` code is not exempt from the “`.h` and `.cc` files come in
+ pairs” rule, so if you declare something in `api/path/to/foo.h`, it should be
+ defined in `api/path/to/foo.cc`.
+* Headers in `api/` should, if possible, not `#include` headers outside `api/`.
+ It’s not always possible to avoid this, but be aware that it adds to a small
+ mountain of technical debt that we’re trying to shrink.
+* `.cc` files in `api/`, on the other hand, are free to `#include` headers
+ outside `api/`.
+* Avoid structs in api, prefer classes.
+
+The preferred way for `api/` code to access non-`api/` code is to call
+it from a `.cc` file, so that users of our API headers won’t transitively
+`#include` non-public headers.
+
+For headers in `api/` that need to refer to non-public types, forward
+declarations are often a lesser evil than including non-public header files. The
+usual [rules](/g3doc/style-guide.md#forward-declarations) still apply, though.
+
+`.cc` files in `api/` should preferably be kept reasonably small. If a
+substantial implementation is needed, consider putting it with our non-public
+code, and just call it from the `api/` `.cc` file.
+
+Avoid defining api with structs as it makes harder for the api to evolve.
+Your struct may gain invariant, or change how it represents data.
+Evolving struct from the api is particular challenging as it is designed to be
+used in other code bases and thus needs to be updated independetly from its usage.
+Class with accessors and setters makes such migration safer.
+See [Google C++ style guide](https://google.github.io/styleguide/cppguide.html#Structs_vs._Classes) for more.
+
+If you need to evolve existent struct in api, prefer first to convert it into a class.