queue_remove_front: no return value

Author: chatziko

include/ADTQueue.h

@@ -3,7 +3,7 @@
 // ADT Queue
 //
 // Abstract ουρά. Λειτουργία "First In First Out", δηλαδή σε κάθε
-// remove επιστρέφεται το παλιότερο στοιχείο που έγινε insert.
+// remove αφαιρείται το παλιότερο στοιχείο που έγινε insert.
 //
 ///////////////////////////////////////////////////////////////////
 
@@ -42,9 +42,9 @@ Pointer queue_back(Queue queue);
 
 void queue_insert_back(Queue queue, Pointer value);
 
-// Αφαιρεί και επιστρέφει την τιμή στο μπροστά μέρος της ουράς (μη ορισμένο αποτέλεσμα αν η ουρά είναι κενή)
+// Αφαιρεί την τιμή στο μπροστά μέρος της ουράς (μη ορισμένο αποτέλεσμα αν η ουρά είναι κενή)
 
-Pointer queue_remove_front(Queue queue);
+void queue_remove_front(Queue queue);
 
 // Αλλάζει τη συνάρτηση που καλείται σε κάθε αφαίρεση/αντικατάσταση στοιχείου σε
 // destroy_value. Επιστρέφει την προηγούμενη τιμή της συνάρτησης.

modules/UsingADTList/ADTQueue.c

@@ -54,10 +54,8 @@ void queue_insert_back(Queue queue, Pointer value) {
 	list_insert_next(queue->list, list_last(queue->list), value);		// Προσθήκη στο _τέλος_
 }
 
-Pointer queue_remove_front(Queue queue) {
-	Pointer result = queue_front(queue);	// Αποθήκευση πριν κάνουμε remove
+void queue_remove_front(Queue queue) {
 	list_remove_next(queue->list, NULL);
-	return result;
 }
 
 DestroyFunc queue_set_destroy_value(Queue queue, DestroyFunc destroy_value) {

tests/ADTQueue_test.c

@@ -45,7 +45,8 @@ void test_remove(void) {
 
 	// Διαδοχικά remove, πρέπει να βγουν με την ίδια σειρά που είναι στο array
 	for (int i = 0; i < 1000; i++) {
-		TEST_CHECK(queue_remove_front(queue) == &array[i]);
+		TEST_CHECK(queue_front(queue) == &array[i]);
+		queue_remove_front(queue);
 		TEST_CHECK(queue_size(queue) == 999-i);
 	}