search

moreover22 / tp1-taller

Trabajo práctico 1 - Protocolo DBUS | Taller 7542
Apache License 2.0
0 stars 0 forks source link

Si dbus_process_file_and_process falla, no destruís el cliente (y tampoco al dbus) #9

Open fedemgp opened 4 years ago

fedemgp commented 4 years ago

https://github.com/moreover22/tp1-taller/blob/121553dc3372044c2ef3a1b6334713e27180039b/client.c#L40-L44

Acordate siempre de liberar todos los recursos.

Algo que no mencioné en la corrección del tp1 y que creo pertinente mencionar es el caso de dbus. Por mas que el constructor siempre sea exitoso, y que el destructor no haga nada, es una buena práctica chequear errores y siempre llamar al destructor si ocurre algún error (por ejemplo, en el bloque mencionado en este issue), si el día de mañana le querés agregar algún feature al código que implique reservar memoria en este TDA, podrías llegar a tener leaks, haciendo que tu codigo no escale bien. No lo voy a tener en cuenta en la nota final porque es algo que se me escapó de corregir en la primera entrega, pero atento a esas cosas en los tps siguientes.

fedemgp commented 4 years ago

Lo del cliente si va a ser tomado a cuenta ya que fue una corrección hecha en la primera entrega y que se te escapó un bug.