DReichLab / AdmixTools

Tools test whether admixture occurred and more
183 stars 64 forks source link

fatalx :: avoid coredump #40

Closed EricDeveaud closed 6 years ago

EricDeveaud commented 6 years ago

Would you mind using exit(X) instead of abort() in nicksrc/strsubs.c

with X obviously different from 0

this way error will be raised and no core dump generated. if core is important for debug purpose, some #define will allow to toggle fatalx beetween exit() and abort()

regards

Eric

bumblenick commented 6 years ago

The way I do it is convenient for me. Feel free to change your local version.

Nick

On Fri, Jun 29, 2018 at 8:30 AM, Eric Deveaud notifications@github.com wrote:

Would you mind using exit(X) instead of abort() in nicksrc/strsubs.c

with X obviously different from 0

this way error will be raised and no core dump generated. if core is important for debug purpose, some #define will allow to toggle fatalx beetween exit() and abort()

regards

Eric

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/DReichLab/AdmixTools/issues/40, or mute the thread https://github.com/notifications/unsubscribe-auth/AQn_hxlXSgzaE1nAfD6CLVd3ngjL07jhks5uBh3cgaJpZM4U88Xw .

EricDeveaud commented 6 years ago

point taken

Eric