]> git.cworth.org Git - apitrace/blobdiff - DEVELOPMENT.markdown
Document the need to keep tracing robust.
[apitrace] / DEVELOPMENT.markdown
index a777b539b5bec24b7e521c013ccef45182783c4b..4a494c2dd9b5241f86b588b65275b1da8b0799bb 100644 (file)
@@ -1,12 +1,13 @@
 Overview
 =========
 
-Although focus has and still is on graphical APIs, apitrace has an
+Although focus was and still is on graphical APIs, apitrace has an
 infrastructure to trace generic APIs:
 
- * The APIs types and calls are specified in Python files in spec
+ * the APIs types and calls are specified in Python files in specs
+   sub-directory;
 
-   * there is a type hierarchy in specs\stdapi.py , capable of representing
+   * there is a type hierarchy in specs/stdapi.py, capable of representing
      most types in C language, and additional semantic metadata
 
  * Python scripts generate C code to trace and serialize calls to disk, and
@@ -125,12 +126,24 @@ Commit policy
 Feature development:
 
 * Existing features in master branch should not degrade at any time, for any
-  platform.  (Unless it is not widely used and there is agreement.)
+  platform.  (Unless they are seldom used or redundant and there is agreement.)
 
-* It's fine to add new features for only some platforms.
+  * In particular, new features / changes must not introduce any sort of
+    instability when tracing.
 
-* Non-trivial changes should be staged in a branch, to enable peer-review and
-  regression testing.  Branch should be deleted once code has been merged.
+    While application developers and driver developers may be able to
+    workaround quirks in apitrace, we want to be able to obtain traces from
+    non-technical end-users with minimal intervention.
+
+    This implies that tracing should not make any non-standard assumptions, and
+    care must be taken to ensure the tracing code is robust against invalid
+    parameters, multiple threads, etc.
+
+* It's fine to add new features for only some platforms or APIs.
+
+* Non-trivial changes should be staged in a branch, to allow review and
+  regression testing.  Feature branches should be deleted once they have been
+  merged.
 
 * Releases are tagged commits from master.  There are no stable branches.