Skip to content

Remove unused mixed type#3041

Merged
kneth merged 3 commits intomasterfrom
mc/cleanup-mixedtype
Jun 24, 2016
Merged

Remove unused mixed type#3041
kneth merged 3 commits intomasterfrom
mc/cleanup-mixedtype

Conversation

@beeender
Copy link
Copy Markdown
Contributor

The mixed type is not used by now and it is very unlikely going to be
used in the future.

The mixed type is not used by now and it is very unlikely going to be
used in the future.
@cmelchior
Copy link
Copy Markdown
Contributor

Some RealmProcessor unit tests seems to fail, but apart from that 👍

#include <realm.hpp>

template <class T>
jbyteArray tbl_GetByteArray(JNIEnv* env, jlong nativeTablePtr, jlong columnIndex, jlong rowIndex)
Copy link
Copy Markdown
Contributor

@stk1m1 stk1m1 Jun 22, 2016

Choose a reason for hiding this comment

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

is it possible to inline this as this isn't used in many places?

Sorry. Nevermind.

@stk1m1
Copy link
Copy Markdown
Contributor

stk1m1 commented Jun 22, 2016

👍

2 similar comments
@cmelchior
Copy link
Copy Markdown
Contributor

👍

@kneth
Copy link
Copy Markdown
Contributor

kneth commented Jun 24, 2016

👍

@kneth kneth merged commit 96b4a5f into master Jun 24, 2016
@kneth kneth deleted the mc/cleanup-mixedtype branch June 24, 2016 13:31
@kneth kneth removed the S:Review label Jun 24, 2016
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Mar 16, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants