-
Notifications
You must be signed in to change notification settings - Fork 62
program for calculator #60
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: master
Are you sure you want to change the base?
Conversation
| scanf("%lf",&num2); | ||
| switch(op){ | ||
| case 1: | ||
|
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Unnecessary Spacing.
| printf("=%lf\n", num1*num2); | ||
| break; | ||
| case 4: | ||
| if(num2==0){ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No need of curly braces
|
|
||
| printf("Enter second number\n"); | ||
| scanf("%lf",&num2); | ||
| switch(op){ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Use proper indentation
|
@1abhi23 good work with the PR, but we have to make sure that the code we merge is proper, so please make the necessary changes and push again. Also claim the issue that you are working on in the issues section. So that no one else works in it till the time you do. |
divyam3897
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why have you added two files? Review comments are still not addressed.
No description provided.