stack_remove_top: no return value

Author: chatziko

include/ADTStack.h

@@ -3,7 +3,7 @@
 // ADT Stack
 //
 // Abstract στοίβα. Λειτουργία "First In Last Out", δηλαδή σε κάθε
-// remove επιστρέφεται το τελευταίο στοιχείο που έγινε insert.
+// remove αφαιρείται το τελευταίο στοιχείο που έγινε insert.
 //
 ///////////////////////////////////////////////////////////////////
 
@@ -38,9 +38,9 @@ Pointer stack_top(Stack stack);
 
 void stack_insert_top(Stack stack, Pointer value);
 
-// Αφαιρεί και επιστρέφει  την τιμή στην κορυφή της στοίβας (μη ορισμένο αποτέλεσμα αν η στοίβα είναι κενή)
+// Αφαιρεί την τιμή στην κορυφή της στοίβας (μη ορισμένο αποτέλεσμα αν η στοίβα είναι κενή)
 
-Pointer stack_remove_top(Stack stack);
+void stack_remove_top(Stack stack);
 
 // Αλλάζει τη συνάρτηση που καλείται σε κάθε αφαίρεση/αντικατάσταση στοιχείου σε
 // destroy_value. Επιστρέφει την προηγούμενη τιμή της συνάρτησης.

modules/UsingADTList/ADTStack.c

@@ -48,13 +48,9 @@ void stack_insert_top(Stack stack, Pointer value) {
 	list_insert_next(list, NULL, value);
 }
 
-Pointer stack_remove_top(Stack stack) {
+void stack_remove_top(Stack stack) {
 	List list = (List)stack;
-
-	Pointer result = stack_top(stack);	// Αποθήκευση πριν κάνουμε remove
 	list_remove_next(list, NULL);
-
-	return result;
 }
 
 DestroyFunc stack_set_destroy_value(Stack stack, DestroyFunc destroy_value) {

tests/ADTStack_test.c

@@ -44,7 +44,8 @@ void test_remove(void) {
 
 	// Διαδοχικά remove
 	for (int i = 999; i >= 0; i--) {
-		TEST_CHECK(stack_remove_top(stack) == &array[i]);
+		TEST_CHECK(stack_top(stack) == &array[i]);
+		stack_remove_top(stack);
 		TEST_CHECK(stack_size(stack) == i);
 	}