moved the call back function for the NoC display button from draw.cpp to draw_toggle_functions.cpp. This was done to match the previous refactor changes.

Author: Srivat97

vpr/src/draw/draw.cpp

@@ -536,27 +536,6 @@ static void toggle_window_mode(GtkWidget* /*widget*/,
     window_mode = true;
 }
 
-void toggle_noc_display(GtkWidget* /*widget*/, gint /*response_id*/, gpointer /*data*/)
-{
-    /* this is the callback function for runtime created toggle_noc_display button
-     * which is written in button.cpp                                         */
-    t_draw_state* draw_state = get_draw_state_vars();
-    std::string button_name = "toggle_noc_display";
-    auto toggle_crit_path = find_button(button_name.c_str());
-
-    gchar* combo_box_content = gtk_combo_box_text_get_active_text(
-        GTK_COMBO_BOX_TEXT(toggle_crit_path));
-    if (strcmp(combo_box_content, "None") == 0) {
-        draw_state->draw_noc = DRAW_NO_NOC;
-    } else if (strcmp(combo_box_content, "NoC Links") == 0)
-        draw_state->draw_noc = DRAW_NOC_LINKS;
-    else
-        draw_state->draw_noc = DRAW_NOC_LINK_USAGE;
-
-    g_free(combo_box_content);
-    application.refresh_drawing();
-}
-
 #endif // NO_GRAPHICS
 
 void alloc_draw_structs(const t_arch* arch) {

vpr/src/draw/draw.h

@@ -78,9 +78,6 @@ bool trace_routed_connection_rr_nodes_recurr(const t_rt_node* rt_node,
                                              int sink_rr_node,
                                              std::vector<int>& rr_nodes_on_path);
 
-// toggle functions
-void toggle_noc_display(GtkWidget* widget, gint /*response_id*/, gpointer /*data*/);
-
 /* This routine highlights the blocks affected in the latest move      *
  * It highlights the old and new locations of the moved blocks         *
  * It also highlights the moved block input and output terminals       *

vpr/src/draw/draw_toggle_functions.cpp

@@ -378,6 +378,27 @@ void toggle_router_expansion_costs(GtkWidget* /*widget*/, gint /*response_id*/,
     application.refresh_drawing();
 }
 
+void toggle_noc_display(GtkWidget* /*widget*/, gint /*response_id*/, gpointer /*data*/)
+{
+    /* this is the callback function for runtime created toggle_noc_display button
+     * which is written in button.cpp                                         */
+    t_draw_state* draw_state = get_draw_state_vars();
+    std::string button_name = "toggle_noc_display";
+    auto toggle_crit_path = find_button(button_name.c_str());
+
+    gchar* combo_box_content = gtk_combo_box_text_get_active_text(
+        GTK_COMBO_BOX_TEXT(toggle_crit_path));
+    if (strcmp(combo_box_content, "None") == 0) {
+        draw_state->draw_noc = DRAW_NO_NOC;
+    } else if (strcmp(combo_box_content, "NoC Links") == 0)
+        draw_state->draw_noc = DRAW_NOC_LINKS;
+    else
+        draw_state->draw_noc = DRAW_NOC_LINK_USAGE;
+
+    g_free(combo_box_content);
+    application.refresh_drawing();
+}
+
 // Callback function for NetMax Fanout checkbox
 void net_max_fanout(GtkWidget* /*widget*/, gint /*response_id*/, gpointer /*data*/) {
     /* this is the callback function for runtime created net_max_fanout widget

vpr/src/draw/draw_toggle_functions.h

@@ -88,6 +88,12 @@ void toggle_router_expansion_costs(GtkWidget* /*widget*/, gint /*response_id*/,
  * Controls if placement macros should be visualized. Changes value of draw_state->show_placement_macros. */
 void toggle_placement_macros(GtkWidget* /*widget*/, gint /*response_id*/, gpointer /*data*/);
 
+/* Callback function for runtime created toggle_noc_display 
+ * in button.cpp.
+ * Controls if the NoC on chip should be visualized and whether the link usage
+ * in the NoC should be visualized. Changes value of draw_state->draw_noc */
+void toggle_noc_display(GtkWidget* widget, gint /*response_id*/, gpointer /*data*/);
+
 /* Callback function for runtime created netMaxFanout widget in button.cpp.
  * Sets draw_state->draw_net_max_fanout to its corresponding value in the UI. */
 void net_max_fanout(GtkWidget* /*widget*/, gint /*response_id*/, gpointer /*data*/);