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

Handling of SYNTHETIC and BRIDGE modifiers added #243

Merged
merged 2 commits into from
Oct 20, 2019
Merged

Handling of SYNTHETIC and BRIDGE modifiers added #243

merged 2 commits into from
Oct 20, 2019

Conversation

alexfedorenchik
Copy link
Contributor

Issue: #212
Issue: #87

Signed-off-by: Alexander Fedorenchik alexander.fedorenchik@gmail.com

@ghost
Copy link

ghost commented Oct 4, 2019 •
edited by ghost
Loading

DeepCode Report (#da5b6f)

DeepCode analyzed this pull request.
There are no new issues.

alexfedorenchik and others added 2 commits October 21, 2019 00:46
@alexfedorenchik @codecholeric
Handling of SYNTHETIC and BRIDGE modifiers was added.
571a26d 
Issue: TNG#212
Issue: TNG#87

Signed-off-by: Alexander Fedorenchik <alexander.fedorenchik@gmail.com>
@codecholeric
Review: Simplify test and add some comments as to why synthetic / bri…
da5b6f4 
…dge members are created in the example class

Signed-off-by: Peter Gafert <peter.gafert@tngtech.com>
@codecholeric
Copy link
Collaborator

Thanks a lot 😃
I've simplified the test a little bit and added some comments to the example class as to why synthetic members are actually created (just for the next time someone needs to understand the test 😉)

@codecholeric codecholeric merged commit 4248da4 into TNG:master Oct 20, 2019
@codecholeric codecholeric mentioned this pull request Nov 3, 2019
Closed
codecholeric added a commit that referenced this pull request Feb 21, 2021
@codecholeric
Merge pull request #243 from alexfedorenchik/synthetics
12e3984 
Handling of SYNTHETIC and BRIDGE modifiers added
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@alexfedorenchik @codecholeric