search

iuliusfdp / mixare

Automatically exported from code.google.com/p/mixare
0 stars 0 forks source link

I propose to divide MixContext #115

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago 
I propose to divide MixContext into 4 different class:

LocationManager - manage Location of device.
DownloadManager - manage process of downloading.
DataSourceManager - manage e provide DataSources.
WebPageManager - manage a opening of webpage.
HttpUtils - static function to manage stream.
MixContext - Only provider of managers

Other info at link 
https://groups.google.com/d/topic/mixare-development/Aw3PUwsX8tw/discussion

Original issue reported on code.google.com by alessand...@gmail.com on 11 May 2012 at 12:35

GoogleCodeExporter commented 8 years ago 
LocationManager was already implemented by abduegal.

I agree on the changes.
and this should help alot
HttpUtils - static function to manage stream.
Since there is leak - usually one IOStream is not closed.

Original comment by DevBinn...@gmail.com on 11 May 2012 at 12:56

GoogleCodeExporter commented 8 years ago 
Pull request at https://github.com/mixare/mixare/pull/29

Original comment by trs...@gmail.com on 15 May 2012 at 1:19

GoogleCodeExporter commented 8 years ago

Original comment by daniele.gobbetti on 16 May 2012 at 12:07

GoogleCodeExporter commented 8 years ago 
Your pull request has been merged, thanks a lot for the contribution!

Original comment by daniele.gobbetti on 16 May 2012 at 12:07