Move assert(ev) to before we use ev in EV_CLOSURE_EVENT_FINALIZE case

Based on a patch from Harlan Stenn.

Merge remote-tracking branch 'origin/pr/110'

Merge remote-tracking branch 'origin/patches-2.0'

Add -Qunused-arguments for clang on macos

The clang compiler provided with macosx emits warnings like:

  CC       bufferevent.lo
clang: warning: argument unused during compilation: '-I .'
clang: warning: argument unused during compilation: '-I ./compat'
clang: warning: argument unused during compilation: '-I ./include'
clang: warning: argument unused during compilation: '-I ./include'

for each file being compiled. This generates a lot of noise during
compilation making it hard to see "real" errors. This patch mute
those warnings.

Add option to build shared library

Do not offer EV_FEATURE_EARLY_CLOSE if we have no EPOLLRDHUP

Merge pull request #97 from JoakimSoderberg/openwrt_fixes

BUGFIX: Fix compilation on systems with EPOLLRDHUP undefined.

Stop checking for inet_aton; we don't use it.

We had cmake and autoconf tests for the inet_aton function... but we
never actually use it any more.

(The autoconf tests still use the callability of inet_aton to decide
whether we need to link against -lresolv)

Reported by Harlan Stenn.

Rename event_extras to event_extra

The library event_extras is named event_extra when building with
GNU autotools

BUGFIX: Fix compilation on systems with EPOLLRDHUP undefined.

Since epolltable-internal.h uses this define, it must be defined before that is included.

Merge pull request #108 from trondn/cmake-build-fix

CMake build-related fixes

Add cmake-related files to .gitignore

Added -Qunused-arguments for clang on macosx

The CLang provided through Xcode on Mac OSX emits warnings for
all unused include paths making the compilation extremely verbose.

Merge remote-tracking branch 'origin/patches-2.0'

bufferevent_pair: don't call downcast(NULL)

Tweaked callbacks to prevent race condition (https://github.com/libevent/libevent/issues/104)

Tweaked callbacks to prevent race condition (https://github.com/libevent/libevent/issues/104)

Added a Travis-CI configuration file.

Initial stab at a first Travis config file.

Merge remote-tracking branch 'joakimsoderberg/win32_fixes'

Merge remote-tracking branch 'origin/patches-2.0'

Check does arch have the epoll_create and __NR_epoll_wait syscalls.

Some architectures (like AArch64) do not have deprecated syscalls.

Signed-off-by: Marcin Juszkiewicz <mjuszkiewicz@redhat.com>

Check for OSX when checking for clang.

Guard against EVENT_NOWIN32 being set during testing.

On windows all tests will fail if EVENT_NOWIN32 is set, since then there
will be no backend available.

Question is if we should simply disable the environment variable check on
Windows, since there's only one backend available anyway?

Fix https-client compilation on Windows.

ssize_t is not defined. But using ssize_t for s in this context makes no
sense, since fread returns size_t.

CMake: Get rid of python not found warning when regress tests turned off.

It would warn that python wasn't found when it was, just because the regress tests where turned off... confusing.

Fix CMake compile when OpenSSL is disabled.

Split epoll lookup table into a separate header file

It accounted for more than half the length of epoll.c, and it's
machine-generated, so we might as well keep it separate.

Expand EV_CLOSED documentation a bit

Merge remote-tracking branch 'origin/pr/98'

Disclaimerize cmake a little in the README

Merge remote-tracking branch 'joakimsoderberg/new_cmake'

Conflicts:
sample/https-client.c

Fix broken autotools build.

The move of the static *config.h files to a sub directory for WIN32 broke the autotools build. Use the proper subpath for that as well...

Fix a c90 warning

Check if we're on OSX before disabling deprecation in le-proxy

Change all uses of WIN32 to _WIN32

Fix include bug.

On systems where a previous version of Libevent is installed we don't want the system version of the headers to be included before the ones in the build tree. This happened on my OSX system where I had an ancient version of Libevent installed. It would then load the incorrect event-config.h and fail because the system introspection macros weren't set properly.

Implemented EV_CLOSED event for epoll backend (EPOLLRDHUP).

- Added new EV_CLOSED event - detects premature connection close
  by clients without the necessity of reading all the pending
  data. Does not depend on EV_READ and/or EV_WRITE.

- Added new EV_FEATURE_EARLY_CLOSED feature for epoll.
  Must be supported for listening to EV_CLOSED event.

- Added new regression test: test-closed.c

- All regression tests passed (test/regress and test/test.sh)

- strace output of test-closed using EV_CLOSED:
    socketpair(PF_LOCAL, SOCK_STREAM, 0, [6, 7]) = 0
    sendto(6, "test string\0", 12, 0, NULL, 0) = 12
    shutdown(6, SHUT_WR)                    = 0
    epoll_ctl(3, EPOLL_CTL_ADD, 7, {EPOLLRDHUP, {u32=7, u64=7}}) = 0
    epoll_wait(3, {{EPOLLRDHUP, {u32=7, u64=7}}}, 32, 3000) = 1
    epoll_ctl(3, EPOLL_CTL_MOD, 7, {EPOLLRDHUP, {u32=7, u64=7}}) = 0
    fstat(1, {st_mode=S_IFCHR|0620, st_rdev=makedev(136, 4), ...})
    mmap(NULL, 4096, PROT_READ|PROT_WRITE, MAP_PRIVATE|MAP_ANONYM...
    write(1, "closed_cb: detected connection close "..., 45) = 45

Forgotten headers for old nmake project compatability.

Clean up the README some.

Update README with CMake build instructions.

Rename README to README.md and use markdown to format.

Fix so that old nmake project still builds.

Set USE_DEBUG=1 on EVENT__ENABLE_VERBOSE_DEBUG

Some work on making it possible to simply do add_subdirectory() on the project.

Fix typo

Add CMake config and install targets.

Make it easier for other projects to do find_package by creating config
files both in the build and install tree.

Fix even more coverity warnings.

Fix a couple of compilation warnings in regress_http.c

Remove spurious checks in evrpc.c error cases (coverity)

Coverity hates the case where you have a redundant NULL check after
using something.

Whoops; fix compilation in bench.c

Fix coverity warnings in benchmark tools.

Again, not harmful, but best to stay warning-free.

Fix a pile of coverity warnings in the unit tests

Most of these problems can only trigger when the tests fail, but it's
good hygiene to keep the code warning-free in any case.

Merge remote-tracking branch 'asweeny86/on-complete-cb'

Update unit test to make sure that the callback happens after the output data is written

evhttp_request_set_on_complete_cb to be more specific about what the function actually does and usage

Merge remote-tracking branch 'asweeny86/event-count-max'

Provide on request complete callback facility

This patch provides the ability to receive a callback on the completion of a
request.  The callback takes place immediately before the request's resources
are released.

Added unit test for max event counts

Fixed bug using wrong variable in max event compare

Add access to max event count stats

This commit provides an interface for accessing and resetting the maximum
number of events in a given period.  This information provides better insight
into event queue pressure.

Remove unneeded declaration in bufferevent-internal.h

Fix needless bufferevent includes in evdns.c

Fix a couple of "#ifdef WIN32" instances

Sample HTTPS Client: Set hostname for SNI extension (by f69m)

Patch from f69m on SourceForge

 fix for ServFail from RIPE Atlas release

Small tweaks to https-client.c

Merge remote-tracking branch 'resetius/https'

Merge remote-tracking branch 'rbalint/from-forked-daapd'

Minor optimizations on bufferevent_trigger options

By making BEV_TRIG_DEFER_CALLBACKS equal to BEV_OPT_DEFER_CALLBACKS,
and BEV_TRIG_IGNORE_WATERMARKS disjoint from BEV_OPT_*, we can save a
few operations in bufferevent_run_*, which is critical-path.

Make bufferevent_trigger_nolock_() inline

Since most of its callers are using constant EV_READ or EV_WRITE, and
using constant 0 as its argument, this should eliminate most of the
overhead for this function in the fast case.

Merge remote-tracking branch 'mistotebe/bufferevent_trigger'

Unit tests for active_by_fd; unsupport active_by_fd(TIMEOUT)

[It turns out that event_base_active_by_fd(TIMEOUT) didn't actually
work right. Feel free to add it back in as a patch.]

Add event_base_active_by_signal by analogy

Sanity-check arguments to event_base_active_by_fd()

Merge remote-tracking branch 'ghazel/event_base_active_by_fd'

Refactor evmap_{io,signal}_active_() to tolerate bad inputs

Merge remote-tracking branch 'origin/patches-2.0'

Merge remote-tracking branch 'sourceforge/patches-2.0' into patches-2.0

Conflicts:
include/event2/event.h

Typo fixes from Linus Nordberg

Typo fixes from Linus Nordberg

Added EVENT__ENABLE_GCC_WARNINGS, turns all warnings into errors.

Only look for ZLib when it is used (if tests are included).

Also make the project language C.

Fix the "make verify" target on NetBSD

Add a "make verify_coverage" target generation coverage info.

Use lcov/gcov to gather coverage info for the tests (Only works with gcc/clang and make).

cmake -DEVENT__COVERAGE=1 -DCMAKE_BUILD_TYPE=Debug ..
make
make verify_coverage

Current coverage (run on debian):
Line coverage 79.1 % 10231 / 12939
Function coverage 86.1 % 933 / 1083

Get rid of unknown pragma warnings.

Minimum required python version is 2.4.

Got rid of TODO... The version you specify to find_package is the minimum
version, so exactly what we want.

Change the BSD license from 4 to 3-clause.

This makes it GPL compatible. Changing the license from 4 to 3-clause BSD
license is OK according to a Berkley announcement on July 22, 1999 by
William Hoskins, the director of the fofice of technology licensing for
Berkley:

ftp://ftp.cs.berkeley.edu/pub/4bsd/README.Impt.License.Change

Added a test for testing if kqueue works with pipes.

Fix kqueue support.

Get rid of deprecation warnings for OpenSSL on OSX 10.7+

Fix the make "verify" target on Windows.

Windows does not have the "unset" command, but this doesn't matter since
the problem that requires us to use unset doesn't happen on Windows.

Also did some minor cosmetic changes, and dependcy changes.

Clarify event_base_loop exit conditions

Added a "make verify" target.

This is more than for cosmetic purposes to match how it's done with autoconf.

Due to the fact that we use environment variables to turn off certain backends during the tests, simply running "ctest" or "make test" can result in failed tests.

This is because if you do "EVENT_NOEPOLL=yes && export EVENT_NOEPOLL" and then run the tests, when running the epoll tests, the EPOLL backend will be turned off. There is no way of unsetting an environment variable for a test in CMake, you can only set them. And since libevent simply checks if the environment variable is set (it doesn't check the actual value of it), this won't work.

So to remedy this, we create the "make verify" target that first unsets all the EVENT_NO* environment variables, and then runs ctest.

Also bumped the required CMake version from 2.6 to 2.8, since the set_test_properties(bla PROPERTIES ENVIRONMENT "SOME_VAR") requires 2.8

Added some explicit dependencies for the test programs to libevent, so they don't just fail if you try to run the tests without first doing "make"

Only test the event backends available on the system.

Fix how the CMake project adds the tests using the different backends. At
first we tried to do it exactly as it's done in test/test.sh.

However, test.sh uses a special program test-init to decide if a given
backend is available or not before running the actual tests. Doing it this way
will not be possible using CMake. Since then we would have to have the
test-init executable compiled at the time we run CMake, to know what tests
we should add. (And since CMake generates the make/project files that
compiles the executables, there's a catch 22).

Instead of deciding what tests to run this way, we simply use the result
of the CMake system introspection (that figures out what backends are
available) to decide what backend tests to add.

Don't segfault on no found event backend.

Fix bench_cascade program on Windows.

Only include WIN32 getopt where it is used.

getopt is only used in the benchmark tests, don't include it in the core
lib...

Add copyright and licensing files for CMake modules.

Use evutil_closesocket instead.

Added some GCC specific options.

- Added sample applications.
- Fixed the https-client to work on Windows kind of (No cert validation).