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.

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

SetBuilder in Snapshots #5

Closed
Taluu opened this issue Nov 26, 2013 · 1 comment
Closed

SetBuilder in Snapshots #5

Taluu opened this issue Nov 26, 2013 · 1 comment

Comments

@Taluu
Copy link
Contributor

Taluu commented Nov 26, 2013

Allow to use a different Set (an extension, or something like that) in the built-in snapshots rather than only the built-in Set.

Currently, if you want to use something else, you have to reimplement the diff method of AbstractSnapshot... Even if all it does is just changing the class used.

@ghost ghost assigned Taluu Nov 26, 2013
@Taluu Taluu removed the ready label May 31, 2014
@Taluu
Copy link
Contributor Author

Taluu commented May 31, 2014

After some thoughts, I don't think SetBuilder would fit in here (it would be rather overkill).

I'm thinking on making a SetInterface, publicize compute method and putting it in the interface.

So, instead of using the constructor to initialize the object, we would do it from the compute method.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant