From 91343f5cfc53e18d5a59eecef3a0c3726fb0c04e Mon Sep 17 00:00:00 2001 From: =?utf8?q?Jos=C3=A9=20Fonseca?= Date: Fri, 1 Apr 2011 08:37:06 +0100 Subject: [PATCH] Handle correctly GL apps which use hardcoded (non generated) names. For example, tests/vparray mesa demo. --- retrace.hpp | 78 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ retrace.py | 5 ++-- 2 files changed, 81 insertions(+), 2 deletions(-) create mode 100644 retrace.hpp diff --git a/retrace.hpp b/retrace.hpp new file mode 100644 index 0000000..c87fc94 --- /dev/null +++ b/retrace.hpp @@ -0,0 +1,78 @@ +/************************************************************************** + * + * Copyright 2011 Jose Fonseca + * All Rights Reserved. + * + * Permission is hereby granted, free of charge, to any person obtaining a copy + * of this software and associated documentation files (the "Software"), to deal + * in the Software without restriction, including without limitation the rights + * to use, copy, modify, merge, publish, distribute, sublicense, and/or sell + * copies of the Software, and to permit persons to whom the Software is + * furnished to do so, subject to the following conditions: + * + * The above copyright notice and this permission notice shall be included in + * all copies or substantial portions of the Software. + * + * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR + * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY, + * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE + * AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER + * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, + * OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN + * THE SOFTWARE. + * + **************************************************************************/ + +#ifndef _RETRACE_HPP_ +#define _RETRACE_HPP_ + +#include + +namespace retrace { + + +/** + * Handle map. + * + * It is just like a regular std::map container, but lookups of missing + * keys return the key instead of default constructor. + * + * This is necessary for several GL named objects, where one can either request + * the implementation to generate an unique name, or pick a value never used + * before. + * + * XXX: In some cases, instead of returning the key, it would make more sense + * to return an unused data value (e.g., container count). + */ +template +class map +{ +private: + typedef std::map base_type; + base_type base; + +public: + + T & operator[] (const T &key) { + typename base_type::iterator it; + it = base.find(key); + if (it == base.end()) { + return (base[key] = key); + } + return it->second; + } + + const T & operator[] (const T &key) const { + typename base_type::const_iterator it; + it = base.find(key); + if (it == base.end()) { + return (base[key] = key); + } + return it->second; + } +}; + + +} /* namespace retrace */ + +#endif /* _RETRACE_HPP_ */ diff --git a/retrace.py b/retrace.py index ad8c844..1cdb10a 100644 --- a/retrace.py +++ b/retrace.py @@ -266,6 +266,7 @@ class Retracer: def retrace_api(self, api): print '#include "trace_parser.hpp"' + print '#include "retrace.hpp"' print types = api.all_types() @@ -274,10 +275,10 @@ class Retracer: for handle in handles: if handle.name not in handle_names: if handle.key is None: - print 'static std::map<%s, %s> __%s_map;' % (handle.type, handle.type, handle.name) + print 'static retrace::map<%s> __%s_map;' % (handle.type, handle.name) else: key_name, key_type = handle.key - print 'static std::map<%s, std::map<%s, %s> > __%s_map;' % (key_type, handle.type, handle.type, handle.name) + print 'static std::map<%s, retrace::map<%s> > __%s_map;' % (key_type, handle.type, handle.name) handle_names.add(handle.name) print -- 2.7.4