[py] [force_reannounce] how to pass flags from python?

[laur89]

With force_reannounce, how would one pass a ignore_min_interval flag via reannounce_flags_t param in python bindings in order not to honor min-interval, and announce tracker immediately?
Would torrenthandle.force_reannounce(0, -1, 1) -- ie passing int(1) -- do the trick?

For completeness, here's the libtorrent method signature:

void force_reannounce(int seconds = 0, int idx = -1, reannounce_flags_t = {}) const;

[arvidn]

you can see the binding definition here: https://github.com/arvidn/libtorrent/blob/RC_2_0/bindings/python/src/torrent_handle.cpp#L536
the parameters are named, so I believe you should be able to get away with just specifying flags=

The flag value is defined here: https://github.com/arvidn/libtorrent/blob/RC_2_0/bindings/python/src/torrent_handle.cpp#L651

and should be accessible to python as lt.reannounce_flags_t.ignore_min_interval

[laur89]

Ah those are defaults in python as well, so

torrenthandle.force_reannounce(flags=lt.reannounce_flags_t.ignore_min_interval)

should be it. Thanks.

Out of curiosity, how would one provide multiple flags, given reannounce() accepted them? Would it be still single parameter passed via flags, but value gets merged together via bitwise operation?

[arvidn]

you combine flags with bitwise OR. in python (and most languages) this is done with the |-operator

[laur89]

Bit unrelated, but wouldn't the correct signature in the docs be

void force_reannounce(int seconds = 0, int idx = -1, reannounce_flags_t flags = {}) const;

vs current

void force_reannounce(int seconds = 0, int idx = -1, reannounce_flags_t = {}) const;

(note the missing flags in current version)

[arvidn]

in C++, those signatures are identical. the parameter name is not a necessary part of a signature. Even the argument name is not necessary in the definition, if it's unused.

[laur89]

It's not technically wrong, but it's not consistent with some other signatures in the docs is all. Not a major issue, just wanted to point out in case it's something that got missed.