Skip to content

Conversation

@PabloAsensio
Copy link

Fallo de segmentación cuando se elimina el último avión solucionado.

@fmrico
Copy link
Owner

fmrico commented Jan 7, 2019

Hola @pasensio97 ,

Necesito que no modifiques el CMakeLists.txt, ni añadas los ficheros que se autogeneran (src/ICE/cpp/ATCDisplay.cpp, src/ICE/cpp/ATCDisplay.h). Haz cambios solamente en el código que sea relevante para tu PR, que son 3 líneas realmente.

Por lo demás, creo que tu solución es correcta :)

Saludos

@@ -0,0 +1,1025 @@
// **********************************************************************
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No añadas este fichero

@@ -0,0 +1,2594 @@
// **********************************************************************
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No este otro

Airport::NextFocus()
{
if(flights.empty()) return;
//if(flights.empty()) return;
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seguro que esto no tiene que estar?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No sé por qué no lo descomenté.

@fmrico
Copy link
Owner

fmrico commented Jan 8, 2019

Comprueba #69

@PabloAsensio
Copy link
Author

Creo que ya está todo actualizado.
Aunque con la última actualización... se generan muchos fallos de segmentación. :c

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants