Thread: function program, return value to remove warning message (math.h)

Code:

#include <stdio.h>
#include <stdlib.h>
#include <math.h>

double sqrt_function (double number);

int main (void)
{
	double val;
	double p = 0;
	printf("input number: ");
	scanf("%lf", &val);

	p = sqrt_function (val);	

	printf("\n\n\t%f\n\n", p);
	return 0;
}

double sqrt_function (double number)
{
	sqrt (number);

	return;
}

please, with -Wall and -lm on Debian linux...

Code:

jamie@box6:~/git/square_c$ cc -Wall -lm lost.c
lost.c: In function ‘sqrt_function’:
lost.c:24:2: warning: ‘return’ with no value, in function returning non-void
  return;
  ^~~~~~
lost.c:20:8: note: declared here
 double sqrt_function (double number)
        ^~~~~~~~~~~~~
jamie@box6:~/git/square_c$

code works, what would be a better way to write the sqrt_function?

Thanks!

Originally Posted by _jamie

Code:

jamie@box6:~/git/square_c$ cc -Wall -lm lost.c
lost.c: In function ‘sqrt_function’:
lost.c:24:2: warning: ‘return’ with no value, in function returning non-void
  return;
  ^~~~~~
lost.c:20:8: note: declared here
 double sqrt_function (double number)
        ^~~~~~~~~~~~~
jamie@box6:~/git/square_c$

code works, what would be a better way to write the sqrt_function?

Thanks!

Haha. Interdersting. Had to try this out. It actually works, yes.

It would have been great to test this out with clang but:

Code:

Testsqrt.c:24:5: error: non-void function 'sqrt_function' should return a value [-Wreturn-type]
    return;
    ^
1 error generated.

It reports an error instead of a warning ;-(

Originally Posted by Salem

> code works, what would be a better way to write the sqrt_function?
Only by dumb luck.

Code:

double sqrt_function (double number)
{
    double answer = sqrt (number);// you were just ignoring the result
 
    return answer; // this is a value to return
}

thanks, looks good, and works great.