search

davidmoten / rxjava-file

RxJava observables for files including NIO events
Apache License 2.0
82 stars 17 forks source link

StringObservable required? #9

Closed ayedo closed 8 years ago

ayedo commented 8 years ago

Hi,

first: thanks for your great work!

In the documentation you note:

[...] if you want the Observable to be emitting line by line then wrap it with a call like StringObservable.split(observable, "\n"). [...]

Is this still required? I saw in the source code that tailText calls a function "toLines". Does this call do something different? Why is StringObservable required?

davidmoten commented 8 years ago

Thanks for that. No not required now because of the dependency on rxjava-extras that has a backpressure supporting version of split. I'll update the doco.