X-Git-Url: https://git.cworth.org/git?a=blobdiff_plain;f=cli%2Fcli_trim.cpp;h=f7bc92bacaee63d9fb7fea32e3e8a4bb1e03f54b;hb=38839b7ee68a63cae3d782368988daabb808bfd1;hp=3302dffc98325fcb64876db0192d16367b6e60e0;hpb=6338d4c26c8ba7afad886404921bf1988ced3822;p=apitrace

diff --git a/cli/cli_trim.cpp b/cli/cli_trim.cpp
index 3302dff..f7bc92b 100644
--- a/cli/cli_trim.cpp
+++ b/cli/cli_trim.cpp
@@ -456,6 +456,20 @@ class TraceAnalyzer {
             unlinkAll(ss_target.str());
             link(ss_target.str(), ss_texture.str());
 
+            /* FIXME: This really shouldn't be necessary. The effect
+             * this provide() has is that all glBindTexture calls will
+             * be preserved in the output trace (never trimmed). Carl
+             * has a trace ("btr") where a glBindTexture call should
+             * not be necessary at all, (it's immediately followed
+             * with a glBindTexture to a different texture and no
+             * intervening texture-related calls), yet this 'provide'
+             * makes the difference between a trim_stress test failing
+             * and passing.
+             *
+             * More investigation is necessary, but for now, be
+             * conservative and don't trim. */
+            provide("state", call->no);
+
             return;
         }