search

Konstantin8105 / c4go

Transpiling C code to Go code
MIT License
363 stars 37 forks source link

math.log1p #151

Closed Konstantin8105 closed 3 years ago

Konstantin8105 commented 5 years ago

Add test and implementation on Go for function math.log1p Description: https://en.cppreference.com/w/c/numeric/math/log1p File for test code add : c4go/tests/math.c

ChrsMark commented 5 years ago

@Konstantin8105 could I claim this one?

Konstantin8105 commented 5 years ago

@ChrsMark , Fell free

Konstantin8105 commented 5 years ago

Like you can use PR: https://github.com/Konstantin8105/c4go/pull/155

rjalfa commented 5 years ago

math.Log1p already exists. We can simply map it in function_definition

Konstantin8105 commented 5 years ago

@rjalfa , yes in Go we have that functon, but c4go haven't test, like in PR: https://github.com/Konstantin8105/c4go/pull/156/files

rjalfa commented 5 years ago

Okay I will add the declarations and the tests

ChrsMark commented 5 years ago

@Konstantin8105 seems that this is fixed on https://github.com/Konstantin8105/c4go/pull/158. It was supposed to be claimed by me, but apparently @rjalfa hopped on it....anyway.

Konstantin8105 commented 5 years ago

Yes, that issue is solved. It will be close. Dear @ChrsMark , please choose any another issue or run 'go test -v -run=TestCSTD' and choose any unsupported function. If my answer is not enought , then please clarify - how I can help.