PhysicalPrinterDialog: Avoid ambiguous call of overloaded fn set_values #1791

Closed
opened 2026-04-05 19:57:13 +02:00 by MrUnknownDE · 0 comments
Owner

Originally created by @frmdstryr on 10/28/2025

Description

When compiling with gcc 13 I get the following compile error:

/home/usr/projects/OrcaSlicer/src/slic3r/GUI/PhysicalPrinterDialog.cpp: In member function ‘void Slic3r::GUI::PhysicalPrinterDialog::build_printhost_settings(Slic3r::GUI::ConfigOptionsGroup*)’:
/home/usr/projects/OrcaSlicer/src/slic3r/GUI/PhysicalPrinterDialog.cpp:377: error: call of overloaded ‘set_values(<brace-enclosed initializer list>)’ is ambiguous
  377 |         choice->set_values({ m_config->opt_string("printhost_port") });
      | 
In file included from /home/usr/projects/OrcaSlicer/src/slic3r/GUI/ConfigManipulation.hpp:12,
                 from /home/usr/projects/OrcaSlicer/src/slic3r/GUI/Tab.hpp:35,
                 from /home/usr/projects/OrcaSlicer/src/slic3r/GUI/PhysicalPrinterDialog.cpp:28:
/home/usr/projects/OrcaSlicer/src/slic3r/GUI/Field.hpp:451: note: candidate: ‘void Slic3r::GUI::Choice::set_values(const std::vector<std::__cxx11::basic_string<char> >&)’
  451 |         void                    set_values(const std::vector<std::string> &values);
      | 
/home/usr/projects/OrcaSlicer/src/slic3r/GUI/Field.hpp:452: note: candidate: ‘void Slic3r::GUI::Choice::set_values(const wxArrayString&)’
  452 |         void                    set_values(const wxArrayString &values);
      | 

This uses wxArrayString to avoid the ambiguity.

Screenshots/Recordings/Graphs

Tests

Compiling.

*Originally created by @frmdstryr on 10/28/2025* # Description When compiling with gcc 13 I get the following compile error: ```gdb /home/usr/projects/OrcaSlicer/src/slic3r/GUI/PhysicalPrinterDialog.cpp: In member function ‘void Slic3r::GUI::PhysicalPrinterDialog::build_printhost_settings(Slic3r::GUI::ConfigOptionsGroup*)’: /home/usr/projects/OrcaSlicer/src/slic3r/GUI/PhysicalPrinterDialog.cpp:377: error: call of overloaded ‘set_values(<brace-enclosed initializer list>)’ is ambiguous 377 | choice->set_values({ m_config->opt_string("printhost_port") }); | In file included from /home/usr/projects/OrcaSlicer/src/slic3r/GUI/ConfigManipulation.hpp:12, from /home/usr/projects/OrcaSlicer/src/slic3r/GUI/Tab.hpp:35, from /home/usr/projects/OrcaSlicer/src/slic3r/GUI/PhysicalPrinterDialog.cpp:28: /home/usr/projects/OrcaSlicer/src/slic3r/GUI/Field.hpp:451: note: candidate: ‘void Slic3r::GUI::Choice::set_values(const std::vector<std::__cxx11::basic_string<char> >&)’ 451 | void set_values(const std::vector<std::string> &values); | /home/usr/projects/OrcaSlicer/src/slic3r/GUI/Field.hpp:452: note: candidate: ‘void Slic3r::GUI::Choice::set_values(const wxArrayString&)’ 452 | void set_values(const wxArrayString &values); | ``` This uses wxArrayString to avoid the ambiguity. # Screenshots/Recordings/Graphs <!-- > Please attach relevant screenshots to showcase the UI changes. > Please attach images that can help explain the changes. --> ## Tests Compiling.
Sign in to join this conversation.
No Label
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github/OrcaSlicer#1791