From cbfb417dce790dac833fcfcff7c4de05df547b65 Mon Sep 17 00:00:00 2001
From: Ashwin <ashwinr2k2@gmail.com>
Date: Mon, 17 Oct 2022 20:55:55 +1100
Subject: [PATCH 1/8] option to turn off struts

---
 include/components/types.hpp |  1 +
 src/components/bar.cpp       | 12 ++++++++++++
 2 files changed, 13 insertions(+)

diff --git a/include/components/types.hpp b/include/components/types.hpp
index 1d453e1c..b22990c0 100644
--- a/include/components/types.hpp
+++ b/include/components/types.hpp
@@ -183,6 +183,7 @@ struct bar_settings {
   position offset{0, 0};
   side_values padding{ZERO_SPACE, ZERO_SPACE};
   side_values module_margin{ZERO_SPACE, ZERO_SPACE};
+  bool struts{true};
   struct {
     int top;
     int bottom;
diff --git a/src/components/bar.cpp b/src/components/bar.cpp
index 9186d31d..ceed29da 100644
--- a/src/components/bar.cpp
+++ b/src/components/bar.cpp
@@ -218,6 +218,8 @@ bar::bar(connection& conn, signal_emitter& emitter, const config& config, const
 
   m_opts.double_click_interval = m_conf.get(bs, "double-click-interval", m_opts.double_click_interval);
 
+  m_opts.struts = m_conf.get(bs, "enable-struts", m_opts.struts);
+
   if (only_initialize_values) {
     return;
   }
@@ -546,6 +548,16 @@ void bar::reconfigure_pos() {
  * Reconfigure window strut values
  */
 void bar::reconfigure_struts() {
+  if (!m_opts.struts) {
+    if (m_conf.has("global/wm", "margin-bottom")) {
+      m_log.warn("Struts are disabled, ignoring margin-bottom");
+    }
+    if (m_conf.has("global/wm", "margin-top")) {
+      m_log.warn("Struts are disabled, ignoring margin-top");
+    }
+    return;
+  }
+
   window win{m_connection, m_opts.window};
   if (m_visible) {
     auto geom = m_connection.get_geometry(m_screen->root());

From cbf5439e80a1bd25d460c236fa93bf585314d135 Mon Sep 17 00:00:00 2001
From: Ashwin <ashwinr2k2@gmail.com>
Date: Mon, 17 Oct 2022 21:22:22 +1100
Subject: [PATCH 2/8] changelog

---
 CHANGELOG.md | 1 +
 1 file changed, 1 insertion(+)

diff --git a/CHANGELOG.md b/CHANGELOG.md
index 98aea186..62b054ca 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -28,6 +28,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 - `internal/backlight`: `scroll-interval` option ([`#2696`](https://github.com/polybar/polybar/issues/2696), [`#2700`](https://github.com/polybar/polybar/pull/2700))
 - `internal/temperature`: Added `zone-type` setting ([`#2572`](https://github.com/polybar/polybar/issues/2572), [`#2752`](https://github.com/polybar/polybar/pull/2752)) by [@xphoniex](https://github.com/xphoniex)
 - `internal/xwindow`: `%class%` and `%instance%` tokens, which show the contents of the `WM_CLASS` property of the active window ([`#2830`](https://github.com/polybar/polybar/pull/2830))
+- Added `enable-struts` option in bar section to enable/disable struts ([`#2769`](https://github.com/polybar/polybar/issues/2769), [`#2844`](https://github.com/polybar/polybar/pull/2844)) by [@VanillaViking](https://github.com/VanillaViking).
 
 ### Changed
 - `internal/fs`: Use `/` as a fallback if no mountpoints are specified ([`#2572`](https://github.com/polybar/polybar/issues/2572), [`#2705`](https://github.com/polybar/polybar/pull/2705))

From 1e582accb87240525c6087d882e7d84bf66a8f8d Mon Sep 17 00:00:00 2001
From: Ashwin <ashwinr2k2@gmail.com>
Date: Tue, 18 Oct 2022 08:49:24 +1100
Subject: [PATCH 3/8] moved warnings near config options

---
 src/components/bar.cpp | 15 +++++++++------
 1 file changed, 9 insertions(+), 6 deletions(-)

diff --git a/src/components/bar.cpp b/src/components/bar.cpp
index ceed29da..e35f9399 100644
--- a/src/components/bar.cpp
+++ b/src/components/bar.cpp
@@ -225,6 +225,15 @@ bar::bar(connection& conn, signal_emitter& emitter, const config& config, const
   }
 
   // Load values used to adjust the struts atom
+
+  if (!m_opts.struts) {
+    if (m_conf.has("global/wm", "margin-bottom")) {
+      m_log.warn("Struts are disabled, ignoring margin-bottom");
+    }
+    if (m_conf.has("global/wm", "margin-top")) {
+      m_log.warn("Struts are disabled, ignoring margin-top");
+    }
+  }
   auto margin_top = m_conf.get("global/wm", "margin-top", percentage_with_offset{});
   auto margin_bottom = m_conf.get("global/wm", "margin-bottom", percentage_with_offset{});
   m_opts.strut.top = units_utils::percentage_with_offset_to_pixel(margin_top, m_opts.monitor->h, m_opts.dpi_y);
@@ -549,12 +558,6 @@ void bar::reconfigure_pos() {
  */
 void bar::reconfigure_struts() {
   if (!m_opts.struts) {
-    if (m_conf.has("global/wm", "margin-bottom")) {
-      m_log.warn("Struts are disabled, ignoring margin-bottom");
-    }
-    if (m_conf.has("global/wm", "margin-top")) {
-      m_log.warn("Struts are disabled, ignoring margin-top");
-    }
     return;
   }
 

From 30e1cc259565ec61795de9b8adeb47476e242132 Mon Sep 17 00:00:00 2001
From: Madhav Prabhu C M <59736487+madhavpcm@users.noreply.github.com>
Date: Sun, 23 Oct 2022 14:17:22 +0530
Subject: [PATCH 4/8] feat(ipc): <label> support (#2841)

* Label = %output% working

* build fix

* Output as deprecated, Label as default

* Corrections

 * Corrections Simplified

* Changelog updated
---
 CHANGELOG.md            |  1 +
 include/modules/ipc.hpp |  9 +++++++--
 src/modules/ipc.cpp     | 29 ++++++++++++++++++++++-------
 3 files changed, 30 insertions(+), 9 deletions(-)

diff --git a/CHANGELOG.md b/CHANGELOG.md
index 62b054ca..8845bc23 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -19,6 +19,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 - `custom/text`: The `content` setting and all its properties are deprecated in favor of `format` with the same functionality. ([`#2676`](https://github.com/polybar/polybar/pull/2676))
 
 ### Added
+-  Added support for TAG_LABEL (`<label>`) in ipc module  ([`#2841`](https://github.com/polybar/polybar/pull/2841))  by [@madhavpcm](https://github.com/madhavpcm).
 -  Added support for format-i for each hook-i defined in ipc module ([`#2775`](https://github.com/polybar/polybar/issues/2775),  [`#2810`](https://github.com/polybar/polybar/pull/2810))  by [@madhavpcm](https://github.com/madhavpcm).
 - `internal/temperature`: `%temperature-k%` token displays the temperature in kelvin ([`#2774`](https://github.com/polybar/polybar/discussions/2774), [`#2784`](https://github.com/polybar/polybar/pull/2784))
 - `internal/pulseaudio`: `reverse-scroll` option ([`#2664`](https://github.com/polybar/polybar/pull/2664))
diff --git a/include/modules/ipc.hpp b/include/modules/ipc.hpp
index 11fdcff4..403280d3 100644
--- a/include/modules/ipc.hpp
+++ b/include/modules/ipc.hpp
@@ -54,12 +54,17 @@ namespace modules {
     bool has_hook() const;
 
     void set_hook(int h);
-
+    void update_output() ;
    private:
-    static constexpr const char* TAG_OUTPUT{"<output>"};
+    static constexpr auto TAG_OUTPUT = "<output>";
+    static constexpr auto TAG_LABEL = "<label>";
+
+    label_t m_label;
+
     vector<unique_ptr<hook>> m_hooks;
     map<mousebtn, string> m_actions;
     string m_output;
+
     int m_initial{-1};
     int m_current_hook{-1};
     void exec_hook();
diff --git a/src/modules/ipc.cpp b/src/modules/ipc.cpp
index 41ee3fea..aaaac0ff 100644
--- a/src/modules/ipc.cpp
+++ b/src/modules/ipc.cpp
@@ -2,6 +2,7 @@
 
 #include <unistd.h>
 
+#include "drawtypes/label.hpp"
 #include "modules/meta/base.inl"
 
 POLYBAR_NS
@@ -59,11 +60,14 @@ namespace modules {
     for (auto& hook : m_hooks) {
       hook->command = pid_token(hook->command);
     }
-    m_formatter->add(DEFAULT_FORMAT, TAG_OUTPUT, {TAG_OUTPUT});
+
+    m_formatter->add(DEFAULT_FORMAT, TAG_LABEL, {TAG_LABEL, TAG_OUTPUT});
+
+    m_label = load_optional_label(m_conf, name(), TAG_LABEL, "%output%");
 
     for (size_t i = 0; i < m_hooks.size(); i++) {
       string format_i = "format-" + to_string(i);
-      m_formatter->add_optional(format_i, {TAG_OUTPUT});
+      m_formatter->add_optional(format_i, {TAG_LABEL});
     }
   }
 
@@ -122,12 +126,14 @@ namespace modules {
    * Output content retrieved from hook commands
    */
   bool ipc_module::build(builder* builder, const string& tag) const {
-    if (tag == TAG_OUTPUT) {
+    if (tag == TAG_LABEL) {
+      builder->node(m_label);
+      return true;
+    } else if (tag == TAG_OUTPUT) {
       builder->node(m_output);
       return true;
-    } else {
-      return false;
     }
+    return false;
   }
 
   /**
@@ -150,7 +156,7 @@ namespace modules {
 
   void ipc_module::action_send(const string& data) {
     m_output = data;
-    broadcast();
+    update_output();
   }
 
   void ipc_module::action_hook(const string& data) {
@@ -185,7 +191,8 @@ namespace modules {
     } else {
       m_current_hook = -1;
       m_output.clear();
-      broadcast();
+
+      update_output();
     }
   }
 
@@ -238,6 +245,14 @@ namespace modules {
       m_output.clear();
     }
 
+    update_output();
+  }
+
+  void ipc_module::update_output() {
+    if (m_label) {
+      m_label->reset_tokens();
+      m_label->replace_token("%output%", m_output);
+    }
     broadcast();
   }
 } // namespace modules

From 291a4844bcfb613949618bddadcf21927e0765c1 Mon Sep 17 00:00:00 2001
From: patrick96 <p.ziegler96@gmail.com>
Date: Sun, 23 Oct 2022 14:11:31 +0200
Subject: [PATCH 5/8] fix(ipc): Crash when sending to multiple instances

In polybar-msg, the connection handle was captured as a reference to a
shared_ptr, but the shared_ptr went out of scope right after.

This probably always caused UB, but was only noticeable for two or more
instances.
---
 include/ipc/ipc.hpp |  4 ++--
 src/ipc/ipc.cpp     | 16 ++++++++--------
 src/polybar-msg.cpp |  6 +++---
 3 files changed, 13 insertions(+), 13 deletions(-)

diff --git a/include/ipc/ipc.hpp b/include/ipc/ipc.hpp
index fb642ac8..37e5812c 100644
--- a/include/ipc/ipc.hpp
+++ b/include/ipc/ipc.hpp
@@ -42,7 +42,7 @@ namespace ipc {
     const logger& m_log;
     eventloop::loop& m_loop;
 
-    eventloop::pipe_handle_t socket;
+    eventloop::pipe_handle_t m_socket;
 
     class connection : public non_copyable_mixin, public non_movable_mixin {
      public:
@@ -89,7 +89,7 @@ namespace ipc {
      * Buffer for the currently received IPC message over the named pipe
      */
     string m_pipe_buffer{};
-    void receive_data(string buf);
+    void receive_data(const string& buf);
     void receive_eof();
   };
 } // namespace ipc
diff --git a/src/ipc/ipc.cpp b/src/ipc/ipc.cpp
index df75490c..7aa2e6de 100644
--- a/src/ipc/ipc.cpp
+++ b/src/ipc/ipc.cpp
@@ -40,7 +40,7 @@ namespace ipc {
    * Construct ipc handler
    */
   ipc::ipc(signal_emitter& emitter, const logger& logger, loop& loop)
-      : m_sig(emitter), m_log(logger), m_loop(loop), socket(loop.handle<PipeHandle>()) {
+      : m_sig(emitter), m_log(logger), m_loop(loop), m_socket(loop.handle<PipeHandle>()) {
     m_pipe_path = string_util::replace(PATH_MESSAGING_FIFO, "%pid%", to_string(getpid()));
 
     if (file_util::exists(m_pipe_path) && unlink(m_pipe_path.c_str()) == -1) {
@@ -57,12 +57,12 @@ namespace ipc {
 
     m_log.info("Opening ipc socket at '%s'", sock_path);
     m_log.notice("Listening for IPC messages (PID: %d)", getpid());
-    socket->bind(sock_path);
-    socket->listen(
+    m_socket->bind(sock_path);
+    m_socket->listen(
         4, [this]() { on_connection(); },
         [this](const auto& e) {
           m_log.err("libuv error while listening to IPC socket: %s", uv_strerror(e.status));
-          socket->close();
+          m_socket->close();
         });
   }
 
@@ -133,12 +133,12 @@ namespace ipc {
         });
 
     auto& c = *connection;
-    socket->accept(*c.client_pipe);
+    m_socket->accept(*c.client_pipe);
 
     c.client_pipe->read_start(
         [this, &c](const auto& e) {
           try {
-            c.dec.on_read((const uint8_t*)e.data, e.len);
+            c.dec.on_read(reinterpret_cast<const uint8_t*>(e.data), e.len);
           } catch (const decoder::error& e) {
             m_log.err("ipc: Failed to decode IPC message (reason: %s)", e.what());
 
@@ -175,7 +175,7 @@ namespace ipc {
   }
 
   ipc::fifo::fifo(loop& loop, ipc& ipc, const string& path) : pipe_handle(loop.handle<PipeHandle>()) {
-    int fd;
+    int fd{};
     if ((fd = open(path.c_str(), O_RDONLY | O_NONBLOCK)) == -1) {
       throw system_error("Failed to open pipe '" + path + "'");
     }
@@ -196,7 +196,7 @@ namespace ipc {
   /**
    * Receive parts of an IPC message
    */
-  void ipc::receive_data(string buf) {
+  void ipc::receive_data(const string& buf) {
     m_pipe_buffer += buf;
 
     m_log.warn("Using the named pipe at '%s' for ipc is deprecated, always use 'polybar-msg'", m_pipe_path);
diff --git a/src/polybar-msg.cpp b/src/polybar-msg.cpp
index 7cc5d81a..1d856ec1 100644
--- a/src/polybar-msg.cpp
+++ b/src/polybar-msg.cpp
@@ -260,13 +260,13 @@ int run(int argc, char** argv) {
     /*
      * Index to decoder is captured because reference can be invalidated due to the vector being modified.
      */
-    int idx = decoders.size() - 1;
+    auto idx = decoders.size() - 1;
 
     auto conn = loop.handle<PipeHandle>();
     conn->connect(
         channel,
-        [&conn, &decoders, pid, type, payload, channel, idx]() {
-          on_connection(*conn, decoders[idx], pid, type, payload);
+        [&handle = *conn, &decoders, pid, type, payload, channel, idx]() {
+          on_connection(handle, decoders[idx], pid, type, payload);
         },
         [&](const auto& e) {
           fprintf(stderr, "%s: Failed to connect to '%s' (err: '%s')\n", exec, channel.c_str(), uv_strerror(e.status));

From 87750c064edcfd7198b00624d84b0f994f9e0b03 Mon Sep 17 00:00:00 2001
From: patrick96 <p.ziegler96@gmail.com>
Date: Sun, 23 Oct 2022 14:52:59 +0200
Subject: [PATCH 6/8] build: Bump C++ to C++17

Now requires gcc8 or clang7
---
 .clang-format   | 2 +-
 CHANGELOG.md    | 1 +
 cmake/cxx.cmake | 3 +--
 3 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/.clang-format b/.clang-format
index 4d561f89..2f6dfaf4 100644
--- a/.clang-format
+++ b/.clang-format
@@ -1,6 +1,6 @@
 ---
 Language: Cpp
-Standard: Cpp11
+Standard: c++17
 BasedOnStyle: Google
 ColumnLimit: 120
 NamespaceIndentation: All
diff --git a/CHANGELOG.md b/CHANGELOG.md
index 8845bc23..a901db9c 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -14,6 +14,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ### Build
 - Respect `CMAKE_INSTALL_PREFIX` when installing default config ([`#2770`](https://github.com/polybar/polybar/pull/2770))
+- Bump C++ version to C++17 ([`#2847`](https://github.com/polybar/polybar/pull/2847))
 
 ### Deprecated
 - `custom/text`: The `content` setting and all its properties are deprecated in favor of `format` with the same functionality. ([`#2676`](https://github.com/polybar/polybar/pull/2676))
diff --git a/cmake/cxx.cmake b/cmake/cxx.cmake
index f9b3db45..7caa5cda 100644
--- a/cmake/cxx.cmake
+++ b/cmake/cxx.cmake
@@ -15,13 +15,12 @@ endif()
 option(CXXLIB_CLANG "Link against libc++" OFF)
 option(CXXLIB_GCC "Link against stdlibc++" OFF)
 
-set(CMAKE_CXX_STANDARD 14)
+set(CMAKE_CXX_STANDARD 17)
 set(CMAKE_CXX_STANDARD_REQUIRED ON)
 set(CMAKE_CXX_EXTENSIONS OFF)
 
 set(THREADS_PREFER_PTHREAD_FLAG ON)
 
-
 set(POLYBAR_FLAGS "" CACHE STRING "C++ compiler flags used for compiling polybar")
 
 list(APPEND cxx_base -Wall -Wextra -Wpedantic -Wdeprecated-copy-dtor)

From 10cdec3b9f743e6fe9deaa63d2db50b2cd48e8c1 Mon Sep 17 00:00:00 2001
From: patrick96 <p.ziegler96@gmail.com>
Date: Sun, 23 Oct 2022 15:13:51 +0200
Subject: [PATCH 7/8] build: Fix deprecation errors

---
 src/utils/string.cpp | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/utils/string.cpp b/src/utils/string.cpp
index 00c7b7e6..aea57634 100644
--- a/src/utils/string.cpp
+++ b/src/utils/string.cpp
@@ -128,7 +128,7 @@ namespace string_util {
    * Trims all characters that match pred from the left
    */
   string ltrim(string value, function<bool(char)> pred) {
-    value.erase(value.begin(), find_if(value.begin(), value.end(), not1(pred)));
+    value.erase(value.begin(), find_if(value.begin(), value.end(), std::not_fn(pred)));
     return value;
   }
 
@@ -136,7 +136,7 @@ namespace string_util {
    * Trims all characters that match pred from the right
    */
   string rtrim(string value, function<bool(char)> pred) {
-    value.erase(find_if(value.rbegin(), value.rend(), not1(pred)).base(), value.end());
+    value.erase(find_if(value.rbegin(), value.rend(), std::not_fn(pred)).base(), value.end());
     return value;
   }
 

From ed6a0e90fb8c4af711bab9bd0bf5709ccb855da5 Mon Sep 17 00:00:00 2001
From: tnixeu <4436784+tnixeu@users.noreply.github.com>
Date: Wed, 26 Oct 2022 23:15:14 +0200
Subject: [PATCH 8/8] fix code in order to support cmake unity builds

---
 include/modules/meta/base.inl |  1 +
 src/ipc/encoder.cpp           | 10 +++++-----
 2 files changed, 6 insertions(+), 5 deletions(-)

diff --git a/include/modules/meta/base.inl b/include/modules/meta/base.inl
index 9b6899ad..ffbe9250 100644
--- a/include/modules/meta/base.inl
+++ b/include/modules/meta/base.inl
@@ -1,3 +1,4 @@
+#pragma once
 #include <cassert>
 
 #include "components/builder.hpp"
diff --git a/src/ipc/encoder.cpp b/src/ipc/encoder.cpp
index 905be07e..87cbd915 100644
--- a/src/ipc/encoder.cpp
+++ b/src/ipc/encoder.cpp
@@ -11,11 +11,11 @@ namespace ipc {
     size_t total_size = HEADER_SIZE + payload.size();
     std::vector<uint8_t> data(total_size);
 
-    auto* header = reinterpret_cast<ipc::header*>(data.data());
-    std::copy(ipc::MAGIC.begin(), ipc::MAGIC.end(), header->s.magic);
-    header->s.version = ipc::VERSION;
-    header->s.size = payload.size();
-    header->s.type = type;
+    auto* msg_header = reinterpret_cast<header*>(data.data());
+    std::copy(MAGIC.begin(), MAGIC.end(), msg_header->s.magic);
+    msg_header->s.version = VERSION;
+    msg_header->s.size = payload.size();
+    msg_header->s.type = type;
 
     std::copy(payload.begin(), payload.end(), data.begin() + HEADER_SIZE);
     return data;