From 5fc7b6e796b9c6264e538a699ec3f50235a85449 Mon Sep 17 00:00:00 2001
From: dan sinclair <dj2@everburning.com>
Date: Thu, 28 Mar 2019 10:32:11 -0400
Subject: Hide pipeline buffer formats (#420)

The default colour formats are only used in the pipeline class so this
CL moves them into the anonymous namespace.
---
 samples/amber.cc | 8 +++++---
 1 file changed, 5 insertions(+), 3 deletions(-)

(limited to 'samples/amber.cc')

diff --git a/samples/amber.cc b/samples/amber.cc
index 3b9e263..542a2e0 100644
--- a/samples/amber.cc
+++ b/samples/amber.cc
@@ -32,6 +32,8 @@
 
 namespace {
 
+const char* kGeneratedColorBuffer = "framebuffer";
+
 struct Options {
   std::vector<std::string> input_filenames;
 
@@ -376,7 +378,7 @@ int main(int argc, const char** argv) {
 
   if (!options.image_filename.empty()) {
     amber::BufferInfo buffer_info;
-    buffer_info.buffer_name = "framebuffer";
+    buffer_info.buffer_name = kGeneratedColorBuffer;
     amber_options.extractions.push_back(buffer_info);
   }
 
@@ -400,7 +402,7 @@ int main(int argc, const char** argv) {
       bool usePNG = pos != std::string::npos &&
                     options.image_filename.substr(pos + 1) == "png";
       for (const amber::BufferInfo& buffer_info : amber_options.extractions) {
-        if (buffer_info.buffer_name == "framebuffer") {
+        if (buffer_info.buffer_name == kGeneratedColorBuffer) {
           if (usePNG) {
             result = png::ConvertToPNG(buffer_info.width, buffer_info.height,
                                        buffer_info.values, &out_buf);
@@ -436,7 +438,7 @@ int main(int argc, const char** argv) {
         std::cerr << options.buffer_filename << std::endl;
       } else {
         for (const amber::BufferInfo& buffer_info : amber_options.extractions) {
-          if (buffer_info.buffer_name == "framebuffer")
+          if (buffer_info.buffer_name == kGeneratedColorBuffer)
             continue;
 
           buffer_file << buffer_info.buffer_name << std::endl;
-- 
cgit v1.2.3