SynCurl: some minor improvements

zed · 2019-10-22 11:39:26

1. Add some missing/renamed constants to the TCurlOption: https://github.com/synopse/mORMot/pull/243
Main issue of this constants is to be able to use modern progress callback function (CURLOPT_XFERINFOFUNCTION) as recommended in documentation https://curl.haxx.se/libcurl/c/CURLOPT_ … CTION.html

2. Let pass libcurl dll name as a parameter: https://github.com/synopse/mORMot/pull/245
I would prefer to use the same library name for Win32 and Win64 versions with my application.

Last edited by zed (2019-10-22 11:42:22)

ab · 2019-10-23 08:14:11

I merged both pull requests.

Thanks!

mpv · 2019-10-26 13:43:27

And another one finalization moved into SynCurl

ab · 2019-10-26 20:26:27

Merged.

zed · 2019-10-27 08:15:25

And now, Delphi (2007, 10.3.2) can't compile this unit because of missing initialization section.

Fix:

initialization
  // empty

finalization
  if PtrInt(curl.Module)>0 then begin
    curl.global_cleanup;
    FreeLibrary(curl.Module);
  end;

Last edited by zed (2019-10-27 08:29:29)

ab · 2019-10-27 19:11:02

Delphi 2007 and 10.3 didn't complain on my computer...

Because the unit was disabled on Windows!

Should be fixed now by https://synopse.info/fossil/info/0eaf4c1cc3

mORMot Open Source

#1 2019-10-22 11:39:26

SynCurl: some minor improvements

#2 2019-10-23 08:14:11

Re: SynCurl: some minor improvements

#3 2019-10-26 13:43:27

Re: SynCurl: some minor improvements

#4 2019-10-26 20:26:27

Re: SynCurl: some minor improvements

#5 2019-10-27 08:15:25

Re: SynCurl: some minor improvements

#6 2019-10-27 19:11:02

Re: SynCurl: some minor improvements

Board footer