[bugs/lgtm] code + doc fixes + lgtm corrections

[kvedala]

Description of Change

• Fixed misc/cantor_set.c for lgtm errors
• Fixed searching/binary_search.c for lgtm errors
• fixed cmake - removed /Za option for MSVC compiler
• added details section to project_euler/problem_1/sol1.c
• fix lgtm error for project_euler/problem_401/sol1.c

References

Issue: #550, lgtm alert link

Checklist

• Added description of change
• Added file name matches File name guidelines
• Added tests and example, test must pass
• Relevant documentation/comments is changed or added
• PR title follows semantic commit guidelines
• Search previous suggestions before making a new one, as yours may be a duplicate.
• I acknowledge that all my contributions will be made under the project's license.

Notes:

[lgtm-com]

This pull request fixes 2 alerts when merging cc45bea into 74c091b - view on LGTM.com

fixed alerts:

• 2 for Local variable hides global variable

[lgtm-com]

This pull request fixes 2 alerts when merging 3b0ff5f into 74c091b - view on LGTM.com

fixed alerts:

• 2 for Local variable hides global variable

[lgtm-com]

This pull request fixes 2 alerts when merging e75dfb8 into 74c091b - view on LGTM.com

fixed alerts:

• 2 for Local variable hides global variable

[lgtm-com]

This pull request fixes 3 alerts when merging 2a0b891 into 74c091b - view on LGTM.com

fixed alerts:

• 2 for Local variable hides global variable
• 1 for Comparison result is always the same

[kvedala]

@tjgurwara99
@ayaankhan98
can anyone review these changes

[tjgurwara99]

Also edit the detail of this PR by adding the fix: problem_401/sol1.c to keep track of these changes.