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

TypedIOService interface #396

Merged
merged 2 commits into from
Aug 13, 2020
Merged

TypedIOService interface #396

merged 2 commits into from
Aug 13, 2020

Conversation

frauzufall
Copy link
Member

@frauzufall frauzufall commented Aug 12, 2020

This PR is based on the #395 branch.

When trying to make use of the Location API in scijava-table I realized that there is no common IO service interface for typed IO services, so I wrote TypedIOService. I think it simplifies things a lot, have a look at TextIOService and DefaultTextIOService.

@frauzufall frauzufall changed the base branch from master to use-location August 12, 2020 15:43
@frauzufall frauzufall changed the title Typedioservice TypedIOService interface Aug 12, 2020
@frauzufall frauzufall marked this pull request as draft August 12, 2020 22:34
@frauzufall frauzufall marked this pull request as ready for review August 13, 2020 09:08
Base automatically changed from use-location to master August 13, 2020 16:35
This is an interface which can be used to write IOServices opening and
saving a specific type. The TypedIOServiceTest class demonstrates how to
do that with an exemplary TextIOService.

So far, IO services like DatasetIOService or TableIOServcie don't share
a common IOService interface.
* .. which was originally implemented in the TypedIOServiceTest for
testing the TypedIOService architecture
@ctrueden ctrueden merged commit d387414 into master Aug 13, 2020
@ctrueden ctrueden deleted the typedioservice branch August 13, 2020 16:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants