lmi-commits
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[lmi-commits] [lmi] master ec73905 058/156: Add extra pair of braces to


From: Greg Chicares
Subject: [lmi-commits] [lmi] master ec73905 058/156: Add extra pair of braces to std::array<> initializer for clang
Date: Tue, 30 Jan 2018 17:22:09 -0500 (EST)

branch: master
commit ec73905984b08d068e8d5a6aaf2f805a4534d85c
Author: Vadim Zeitlin <address@hidden>
Commit: Vadim Zeitlin <address@hidden>

    Add extra pair of braces to std::array<> initializer for clang
    
    It is not totally clear whether clang is correct or not in complaining
    about this code without the extra braces, but the fact is that using two
    pairs of braces is definitely correct and clang gives a -Wmissing-braces
    warning without them.
---
 ledger_pdf_generator_wx.cpp | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/ledger_pdf_generator_wx.cpp b/ledger_pdf_generator_wx.cpp
index 5dbb642..738d4e8 100644
--- a/ledger_pdf_generator_wx.cpp
+++ b/ledger_pdf_generator_wx.cpp
@@ -473,7 +473,8 @@ class pdf_illustration : protected html_interpolator
     std::vector<std::unique_ptr<page>> pages_;
 };
 
-std::array<int, 7> const pdf_illustration::html_font_sizes =
+std::array<int, 7> const pdf_illustration::html_font_sizes
+    {
     { 8
     , 9
     ,10
@@ -481,6 +482,7 @@ std::array<int, 7> const pdf_illustration::html_font_sizes =
     ,14
     ,18
     ,20
+    }
     };
 
 class cover_page : public page



reply via email to

[Prev in Thread] Current Thread [Next in Thread]