Skip to content

Primitive List feature(Part6: OsList)#5208

Merged
zaki50 merged 9 commits intofeature/primitive_listfrom
my/primitive_list_6_OsList
Sep 11, 2017
Merged

Primitive List feature(Part6: OsList)#5208
zaki50 merged 9 commits intofeature/primitive_listfrom
my/primitive_list_6_OsList

Conversation

@zaki50
Copy link
Copy Markdown
Contributor

@zaki50 zaki50 commented Sep 5, 2017

No description provided.

@zaki50 zaki50 force-pushed the my/primitive_list_6_OsList branch from 0519058 to 6ec1b48 Compare September 5, 2017 18:30
@zaki50 zaki50 added this to the 4.0 (RMP 2.0) milestone Sep 5, 2017
Copy link
Copy Markdown
Contributor

@cmelchior cmelchior left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If CI is happy 👍

Long val = null;
if (reader.peek() != JsonToken.NULL) {
val = reader.nextLong()
val = reader.nextLong();
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

How did this compile before? 🤔

Copy link
Copy Markdown
Contributor Author

@zaki50 zaki50 Sep 5, 2017

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I guess Google's compile-testing library does not see semi-colons when comparing generated code.

@zaki50
Copy link
Copy Markdown
Contributor Author

zaki50 commented Sep 5, 2017

@cmelchior I've asked @beeender to implement empty methods in this PR.
Could you review this again after that?

@zaki50 zaki50 mentioned this pull request Sep 6, 2017
21 tasks
@zaki50
Copy link
Copy Markdown
Contributor Author

zaki50 commented Sep 9, 2017

Implementation of OSList is added in separate PR (Part 10, #5216)

Could you review this? @realm/java

Copy link
Copy Markdown
Contributor

@beeender beeender left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If CI is happy

@zaki50 zaki50 merged commit 0fb5ed3 into feature/primitive_list Sep 11, 2017
@zaki50 zaki50 deleted the my/primitive_list_6_OsList branch September 15, 2017 05:44
@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Mar 15, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants