Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Misc code cleanup #1114

Open
smedilol opened this issue Aug 23, 2018 · 0 comments
Open

Misc code cleanup #1114

smedilol opened this issue Aug 23, 2018 · 0 comments
Labels
Enhancement
Milestone
Intern and techni...

Comments

@smedilol
Copy link
Contributor

Misc code cleanup like :

  • use ?. instead to check for null null
  • Delete dead code
  • Reduce nesting of if
  • ...
@smedilol smedilol added this to the Intern and technical issues milestone Aug 23, 2018
smedilol pushed a commit that referenced this issue Aug 23, 2018
@cobarzanc @smedilol
WI #1114 Code cleanup
ccc9dc8
@mayanje mayanje mentioned this issue Sep 26, 2018
Merged
smedilol added a commit that referenced this issue Oct 11, 2018
@smedilol
WI #1114 Avoid multiple enumeration of variable found in return of Sy…
c86f059 
…mbolTable

Avoid double cast
osmedile added a commit to osmedile/TypeCobol that referenced this issue Mar 10, 2019
@osmedile
WI TypeCobolTeam#1114 Fix typo in SymbolTable
be517dd
osmedile added a commit to osmedile/TypeCobol that referenced this issue Mar 10, 2019
@osmedile
WI TypeCobolTeam#1114 Fix typo in SymbolTable
b73afd5
osmedile added a commit to osmedile/TypeCobol that referenced this issue Mar 10, 2019
@osmedile
WI TypeCobolTeam#1114 Fix typo, use pattern matching, replace null ch…
2a9a16f 
…eck with `?`
osmedile added a commit to osmedile/TypeCobol that referenced this issue Mar 10, 2019
@osmedile
WI TypeCobolTeam#1114 Fix typo in SymbolTable
db4f3fb
@osmedile osmedile mentioned this issue Mar 10, 2019
Merged
osmedile added a commit to osmedile/TypeCobol that referenced this issue Mar 11, 2019
@osmedile @smedilol
WI TypeCobolTeam#1114 Fix typo, use pattern matching, replace null ch…
2877aed 
…eck with `?`
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Enhancement
Projects
None yet
Milestone
Intern and technical issues
Development

No branches or pull requests
1 participant
@smedilol