Opened 6 years ago
Closed 6 years ago
#1135 closed task (fixed)
Move streaming impl to new package
Reported by: | zzz | Owned by: | dg |
---|---|---|---|
Priority: | minor | Milestone: | 0.9.10 |
Component: | streaming | Version: | 0.9.8.1 |
Keywords: | Cc: | Zlatin Balevsky, dg | |
Parent Tickets: | Sensitive: | no |
Description
Streaming is oddly split across ministreaming.jar (apps/ministreaming) and streaming.jar (apps/streaming) but they share the same package net.i2p.client.streaming.
This is a result of history as there originally was a very simple ministreaming library, and later came the "full" streaming library. The ministreaming implementation is long gone and everything in the "full" lib is essentially the implementation, all package private. What remains in ministreaming is the interfaces, public classes, factory, and SocketEepGet?.
As they share the same package it makes the javadocs a mess. The public classes and the implementation are all smashed together. So let's fix this by splitting up the packages and therefore the javadocs.
As everything in the implementation is package private, we should be able to move the entire implementation (i.e. what's in apps/streaming) to net.i2p.client.streaming.impl without breaking anything and retaining compatibility.
Do not change the way we build the jars. Still two jars, ministreaming and streaming. Still two apps/ dirs, ministreaming and streaming.
Subtickets
Change History (5)
comment:1 Changed 6 years ago by
Cc: | Zlatin Balevsky added |
---|
comment:2 follow-up: 3 Changed 6 years ago by
Cc: | dg added |
---|
comment:3 Changed 6 years ago by
Owner: | set to dg |
---|---|
Status: | new → accepted |
Replying to zzz:
I believe dg started this? If you did and plan to finish it, please assign it to yourself. If you abandoned the effort please let us know so somebody else can do it.
I began it, ran into some issues. I'll investigate after Xmas.
comment:4 Changed 6 years ago by
Milestone: | 0.9.12 → 0.9.10 |
---|---|
Status: | accepted → testing |
Fixed in 0.9.9-4 c1533191d292cff36b78951d0ff9c6a9abd1df7e.
Setting to 'testing'.
comment:5 Changed 6 years ago by
Resolution: | → fixed |
---|---|
Status: | testing → closed |
Closing. This is in 0.9.10.
I believe dg started this? If you did and plan to finish it, please assign it to yourself. If you abandoned the effort please let us know so somebody else can do it.