search

okbob / ncurses-st-menu

ncurses (pdcurses) based library for CUA look menu
BSD 2-Clause "Simplified" License
62 stars 9 forks source link

Remove unnecessary null pointer checks #2

Closed elfring closed 5 years ago

elfring commented 5 years ago

Extra null pointer checks are not needed in functions like the following.

okbob commented 5 years ago

I use this technique in conformance with PostgreSQL community. So I don't see this as issue.

elfring commented 5 years ago

I suggest to take the current POSIX standard into account for corresponding software adjustments. Will another small source code reduction be nicer here?

okbob commented 5 years ago

pá 13. 9. 2019 v 12:30 odesílatel Markus Elfring notifications@github.com napsal:

I suggest to take the current POSIX standard into account https://pubs.opengroup.org/onlinepubs/9699919799/functions/free.html for corresponding software adjustments. Will another small source code reduction be nicer here?

maybe on computers with 10kB memory. It's few bytes.

— You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub https://github.com/okbob/ncurses-st-menu/issues/2?email_source=notifications&email_token=AAEFO46SD3VSLSAZYFNGMNDQJNTTZA5CNFSM4IWOAED2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD6UUISI#issuecomment-531186761, or mute the thread https://github.com/notifications/unsubscribe-auth/AAEFO45RCAVCUXQGAW6WED3QJNTTZANCNFSM4IWOAEDQ .