handles, not handlers

remove scheduler->internal code from driver loop

Remove busy waiting checks

Busy waiting should never happen (at least the shutdown pipe is always
there for the driver to wait for).
When busy waiting happens, i.e. GNUNET_SCHEDULER_run_from_driver is
called without any task ready, it is a programming error (at least I
don't know any valid use case for busy waiting).
Hence, remove the busy waiting checks and let
GNUNET_SCHEDULER_run_from_driver return GNUNET_SYSERR instead in this
case.

Don't consider restart when the test action already finished

In the old scheduler implementation tasks with different priorities were
run in the same event loop iteration, in the new implementation this
cannot be guaranteed anymore, as it's driver-dependent.
For the FS tests this means that when adding both an 'abort' task and a
'restart' task (the latter with higher priority) the 'restart' task
didn't take effect with the old implementation but does with the new
implementation (and leads to an assertion failure).
Not scheduling the 'restart' task when the action (download/publish) has
already finished successfully solves this issue.

clarify documentation about adding / deleting tasks

Simplify driver callback del; fix shutdown logic

The driver callback for deleting a task has been simplified: Now it is
only possible to delete a task, not single FdInfos.

A logic bug in GNUNET_SCHEDULER_cancel has been fixed (FD-related tasks
need to be deleted from the driver, when they are already in the ready
queue).

fix behaviour of GNUNET_SCHEDULER_add_select on empty fdsets

if GNUNET_SCHEDULER_add_select is called with empty fdsets, the
    resulting task is now added to the pending_timeout queue instead of
    the pending queue. This way the driver will not know about the task.

NULL-initialize array pointers

cleanup

GNUNET_SCHEDULER_check_lifeness not needed anymore

tasks in the pending queue must be be checked for reached timeouts, too; allow multiple event types per FdInfo

improve documentation about the GNUNET_array_append and GNUNET_array_grow macros

only allocate network / file handles and store them in the task when GNUNET_SCHEDULER_add_select is used

implement bookkeeping about which FDs related to a tasks have been marked ready (required API change in GNUNET_SCHEDULER_task_ready)

increasing length variable is already done by GNUNET_xgrow_ (called by the GNUNET_array_append macro)

add buggy state of refactored GNUNET_SCHEDULER_add_select

reimplement init_fd_info as preparation for refactoring GNUNET_SCHEDULER_add_select

fix off-by-one error that prevented reading from stdin

init_fd_info now completely initializes fd information; Allow calling add/cancel functions only when the scheduler is running

set fds_len in all cases; call set_wakeup only in two places

Revert "set fds_len in all cases; call set_wakeup only in two places"

This reverts commit 7d42c453fbb032ac8fb43c8e56bb84354ecedf2d.

set fds_len in all cases; call set_wakeup only in two places

remove unnecessary set_wakup call

fix compilation of debug code enabled by DEBUG_FDS; fix indentation

cosmetic fixes

test_scheduler passing; new API function GNUNET_SCHEDULER_check_lifeness

lifeness checks; shutdown now working; checkSignal not passing yet

fix initFdInfo; rename to init_fd_info

fix indentation; remove unneeded set_wakeup call

Merge branch 'refactoring-scheduler' of ssh://gnunet.org/gnunet

separate the select driver's fd sets from the driver-internal fdsets

Smaller changes. Changed order when set_wakeup is called after some task is removed from pending_timeout

fix segfault and pending_timeout logic

add missing DLL insert

fix warnings

make input of getters const (needed for new scheduler API)

GNUNET_SCHEDULER_run is calling GNUNET_SCHEDULER_run_with_driver with the default select polling backend.

Call to set_wakeup added after tasks added/removed to/from pending_timeout

Nearly finished. Call to set_wakeup missing in case of tasks added to pending_timeout

adapt add_without_sets to scheduler driver API; reimplement initFdInfo

small fixes

started further refactoring of scheduler. ATTENTION, actual Status does not compile!

-very unimportant typo fixed

implement GNUNET_h2s2 for printing two hashes in one log output line

fix hostlist server option

multicast: remove unnecessary header file in Makefile.am

deactivate hostlist server option

Merge branch 'master' of ssh://gnunet.org/gnunet

deaktivate hostlist server and use topology option

do not invoke callback after set operation destruction, should fix #5082

revert change, tests must not rely on Internet

make test work on gnunet with public hostlist server

fix FTBFS issues if --enable-logging=no is set

fix nat service port collision during test

fix configs, inlines are now relative to source dir of inlining file

fix test failure introduced by latest configuration parser fix, avoid unnecessary copying around of the configuration file

[transport] Fix EBADF in select()

This fixes issue #3691.

fix configuration parsing issue with inlines from configuration not in cwd

allow NULL fields for varsize

start to fix some test issues with postgres psycstore

Fix debug message.

Fix quota compliance tests.

Ensure peer started callback recvs PeerContext

bitch more if NULL rows cause PQ result extraction to fail

fix crash if block plugin not found

dht: install cli clients

[topology] Fix blacklist condition

This fixes issue #5086.

if (1) holds here, remove redundant if

start VPN helper also if only services are running

fix indexing issue found by jah and reported on gnunet-developers on 6-8/7/2017

fix misc namestore crashes from pq migration

namestore: postgres: fix iterating over zone

Stop iterating on GNUNET_DB_STATUS_SUCCESS_NO_RESULTS.

kill short zone remenants

Merge branch 'master' of ssh://gnunet.org/gnunet

Update multicast test

Remove valgrind from test config

fix indentation

set: disconnect, don't destroy yet

set: remove workaround to test if a set bug and not cadet was at fault

set: adjust log type

consensus: logging text

consensus/set: saner log levels

CONSENSUS/SET:  even uglier CADET workarounds

SET: add another CADET bug workaround

This time it's something that occurs pretty rarely, and only when
sending the full set is triggered.

[transport] copy-paste error, nothing to see here

So embarassing!

[transport] Don't exit with suspended connections

[transport] suspend connection to avoid busy-wait

configure.ac: don't make guesses which uid 'make install' will have

gnunet-social: removed duplicate getopt

set: add 'over' message, since cadet does not reliably detect channel termination

clarify enum values

rename GNUNET_PQ_QueryStatus to GNUNET_DB_QueryStatus

handle UNIQUE violation nicely

[transport] http(s) are stable

Don't valgrind

Update Makefile for deleted files

Update configure.ac POTFILES.in for deleted files

mark dv/http/https/wlan/bt as experimental

removing dead libgnunetpostgres

Fixes.

I left the ?? comments because someone should probably change
how return values are handled