From 71d810f3f518b41b4d9f7d9b0c9bf00d02fdcb3f Mon Sep 17 00:00:00 2001 From: Carl Worth Date: Mon, 4 Nov 2013 14:22:03 -0800 Subject: [PATCH] metrics: Move the create/fini/destroy functions to the top of file. This is simple code movement, which no code changes whatsoever. --- metrics.c | 158 +++++++++++++++++++++++++++--------------------------- 1 file changed, 79 insertions(+), 79 deletions(-) diff --git a/metrics.c b/metrics.c index fdd794a..0a69e58 100644 --- a/metrics.c +++ b/metrics.c @@ -95,6 +95,85 @@ struct metrics op_metrics_t *op_metrics; }; +metrics_t * +metrics_create (void) +{ + metrics_t *metrics; + + metrics = xmalloc (sizeof (metrics_t)); + + metrics->op = 0; + + metrics->timer_begun_id = 0; + + metrics->timer_head = NULL; + metrics->timer_tail = NULL; + + metrics->monitor_begun_id = 0; + + metrics->monitor_head = NULL; + metrics->monitor_tail = NULL; + + metrics->monitors_in_flight = 0; + + metrics->num_op_metrics = 0; + metrics->op_metrics = NULL; + + return metrics; +} + +void +metrics_fini (metrics_t *metrics) +{ + timer_query_t *timer, *timer_next; + monitor_t *monitor, *monitor_next; + + /* Discard and cleanup any outstanding queries. */ + if (metrics->timer_begun_id) { + glEndQuery (GL_TIME_ELAPSED); + glDeleteQueries (1, &metrics->timer_begun_id); + metrics->timer_begun_id = 0; + } + + for (timer = metrics->timer_head; + timer; + timer = timer_next) + { + glDeleteQueries (1, &timer->id); + timer_next = timer->next; + free (timer); + } + metrics->timer_head = NULL; + metrics->timer_tail = NULL; + + if (metrics->monitor_begun_id) { + glEndPerfMonitorAMD (metrics->monitor_begun_id); + glDeletePerfMonitorsAMD (1, &metrics->monitor_begun_id); + metrics->monitor_begun_id = 0; + } + + for (monitor = metrics->monitor_head; + monitor; + monitor = monitor_next) + { + glDeletePerfMonitorsAMD (1, &monitor->id); + monitor_next = monitor->next; + free (monitor); + } + metrics->monitor_head = NULL; + metrics->monitor_tail = NULL; + + metrics->monitors_in_flight = 0; +} + +void +metrics_destroy (metrics_t *metrics) +{ + metrics_fini (metrics); + + free (metrics); +} + static const char * metrics_op_string (metrics_op_t op) { @@ -756,82 +835,3 @@ metrics_end_frame (void) print_program_metrics (); } } - -metrics_t * -metrics_create (void) -{ - metrics_t *metrics; - - metrics = xmalloc (sizeof (metrics_t)); - - metrics->op = 0; - - metrics->timer_begun_id = 0; - - metrics->timer_head = NULL; - metrics->timer_tail = NULL; - - metrics->monitor_begun_id = 0; - - metrics->monitor_head = NULL; - metrics->monitor_tail = NULL; - - metrics->monitors_in_flight = 0; - - metrics->num_op_metrics = 0; - metrics->op_metrics = NULL; - - return metrics; -} - -void -metrics_fini (metrics_t *metrics) -{ - timer_query_t *timer, *timer_next; - monitor_t *monitor, *monitor_next; - - /* Discard and cleanup any outstanding queries. */ - if (metrics->timer_begun_id) { - glEndQuery (GL_TIME_ELAPSED); - glDeleteQueries (1, &metrics->timer_begun_id); - metrics->timer_begun_id = 0; - } - - for (timer = metrics->timer_head; - timer; - timer = timer_next) - { - glDeleteQueries (1, &timer->id); - timer_next = timer->next; - free (timer); - } - metrics->timer_head = NULL; - metrics->timer_tail = NULL; - - if (metrics->monitor_begun_id) { - glEndPerfMonitorAMD (metrics->monitor_begun_id); - glDeletePerfMonitorsAMD (1, &metrics->monitor_begun_id); - metrics->monitor_begun_id = 0; - } - - for (monitor = metrics->monitor_head; - monitor; - monitor = monitor_next) - { - glDeletePerfMonitorsAMD (1, &monitor->id); - monitor_next = monitor->next; - free (monitor); - } - metrics->monitor_head = NULL; - metrics->monitor_tail = NULL; - - metrics->monitors_in_flight = 0; -} - -void -metrics_destroy (metrics_t *metrics) -{ - metrics_fini (metrics); - - free (metrics); -} -- 2.43.0