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

Projects
None yet
1 participant
@pavel-pimenov
Copy link
Owner

pavel-pimenov commented Aug 21, 2015

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

This comment has been minimized.

Copy link
Owner Author

pavel-pimenov commented Aug 21, 2015

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

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

@pavel-pimenov

This comment has been minimized.

Copy link
Owner Author

pavel-pimenov commented Aug 21, 2015

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

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

@pavel-pimenov

This comment has been minimized.

Copy link
Owner Author

pavel-pimenov commented Aug 21, 2015

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

This comment has been minimized.

Copy link
Owner Author

pavel-pimenov commented Aug 21, 2015

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

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

@pavel-pimenov

This comment has been minimized.

Copy link
Owner Author

pavel-pimenov commented Aug 21, 2015

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