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

Отрефакторить класс HttpConnection заменив имеющийся там велосипед на функции ОС. #671

Open
pavel-pimenov opened this issue Aug 21, 2015 · 5 comments

Comments

@pavel-pimenov
Copy link
Owner

From a.rain...@gmail.com on December 17, 2011 16:12:19

Отрефакторить класс HttpConnection заменив имеющийся там велосипед на функции ОС.

Для Win32 реализации можно использовать Util::getDataFromInet, однако требуется его немного доработать убрав IDateReceiveReporter.

Original issue: http://code.google.com/p/flylinkdc/issues/detail?id=632

@pavel-pimenov
Copy link
Owner Author

From a.rain...@gmail.com on December 17, 2011 09:59:22

Owner: a.rain...@gmail.com

@pavel-pimenov
Copy link
Owner Author

From a.rain...@gmail.com on December 19, 2011 06:41:11

Дополнение: для сохранения совместимости необходимо HttpConnection сделать обёрткой над парой функций из будущего InternetConnectionManager (полностью базирующегося на API ОС).

@pavel-pimenov
Copy link
Owner Author

From a.rain...@gmail.com on December 19, 2011 15:47:08

Дополнение 2:
int64_t
Util::getDataFromInet(LPCWSTR agent, const DWORD frameBufferSize, string const& url, boost::shared_array& dataOut, LONG timeOut /=0/, IDateReceiveReporter* reporter /* = NULL */)

size_t
Util::getDataFromInet(LPCWSTR agent, const DWORD frameBufferSize, string const& url, string& data, LONG timeOut /=0/, IDateReceiveReporter* reporter /* = NULL */)

практичеки 100% бартья близнецы, зачем оно такое я не понимаю.

Owner: ---

@pavel-pimenov
Copy link
Owner Author

From Pavel.Pimenov@gmail.com on December 19, 2011 18:27:31

про братьев согласен - первая качает бинари а вторая текст?

@pavel-pimenov
Copy link
Owner Author

From a.rain...@gmail.com on December 21, 2011 05:12:15

фз, я не вникал пока.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant