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

Introduce abstraction from java.io.File #2

Closed
nlmarco opened this issue Mar 17, 2017 · 1 comment
Closed

Introduce abstraction from java.io.File #2

nlmarco opened this issue Mar 17, 2017 · 1 comment
Assignees
@nlmarco
Labels
enhancement

Comments

@nlmarco
Copy link
Contributor

nlmarco commented Mar 17, 2017

As pointed out by bcgit in bcgit/bc-java#119 it is indeed not a good idea to directly use java.io.File. We should introduce an abstraction layer making it possible to use other implementations -- e.g. reading/writing from/to a database or simply memory.
@nlmarco nlmarco self-assigned this Mar 17, 2017
nlmarco added a commit that referenced this issue Mar 17, 2017
@nlmarco
https://github.com/subshare/bc-pgp-wot/issues/2
99d9aa5
nlmarco added a commit that referenced this issue Mar 17, 2017
@nlmarco
https://github.com/subshare/bc-pgp-wot/issues/2
ccb7fa3
nlmarco added a commit to subshare/subshare that referenced this issue Mar 17, 2017
@nlmarco
https://github.com/subshare/bc-pgp-wot/issues/2
fc520e2
@nlmarco
Copy link
Contributor Author

nlmarco commented Mar 17, 2017

There are now the interfaces PgpFile and PgpRandomAccessFile that can easily be implemented to write to sth. else, i.e. not directly into the file system.

The file-system, i.e. java.io.File-based stuff is implemented in the classes IoFile and IoRandomAccessFile.

@nlmarco nlmarco closed this as completed Mar 17, 2017
@nlmarco nlmarco mentioned this issue Mar 17, 2017
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@nlmarco nlmarco

Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@nlmarco