Skip to content

Commit

Permalink
fix(window): remove useless operator= and add default copy cstr (#1729)
Browse files Browse the repository at this point in the history
Fixes compilation under GCC 9

The default copy constructor implicit generation is deprecated by C++ standard.

The window& operator=(const xcb_window_t win); operator seems to be useless.

Fixes #1728
Ref polybar/xpp#16
  • Loading branch information
Lomadriel authored and patrick96 committed Apr 7, 2019
1 parent 627c4ac commit fca4151
Show file tree
Hide file tree
Showing 3 changed files with 2 additions and 8 deletions.
3 changes: 1 addition & 2 deletions include/x11/window.hpp
Expand Up @@ -12,10 +12,9 @@ class connection;

class window : public xpp::window<connection&> {
public:
window(const window&) = default;
using xpp::window<class connection&>::window;

window& operator=(const xcb_window_t win);

window reconfigure_geom(unsigned short int w, unsigned short int h, short int x = 0, short int y = 0);
window reconfigure_pos(short int x, short int y);
window reconfigure_struts(unsigned short int w, unsigned short int h, short int x, bool bottom = false);
Expand Down
2 changes: 1 addition & 1 deletion lib/xpp
Submodule xpp updated from ab6247 to d2ff2a
5 changes: 0 additions & 5 deletions src/x11/window.cpp
Expand Up @@ -7,11 +7,6 @@

POLYBAR_NS

window& window::operator=(const xcb_window_t win) {
resource(connection(), win);
return *this;
}

/**
* Reconfigure the window geometry
*/
Expand Down

0 comments on commit fca4151

Please sign in to comment.