Merge pull request #281 from sergiud/msvc-error-define

Fixed undeclared identifier error

logging_unittest: fixed undeclared identifier error (fixes #267)

Merge pull request #278 from shinh/contrib

Update AUTHORS and CONTRIBUTORS for PR#232

Update AUTHORS and CONTRIBUTORS for PR#232

Merge pull request #232 from qzmfranklin/bazel

Build with Bazel.

Remove the use of bind() from WORKSPACE.

Remove an empty textual_hdrs field.

Hide certain header files from the cc_library().

This commit addresses a few issues:

    1.  No longer leak config.h in a way similar to
            https://github.com/gflags/gflags/issues/233
        The solution of prefixing the path by 'glog_internal' is modified from
            https://github.com/gflags/gflags/issues/234

    2.  No longer expose internal headers.

    3.  Replace PACKAGE_NAME with native.package_name()

    4.  Uers can choose namespaces via the newly added 'namespace' keyword.

    5.  Replace glob with explicitly listing of files.

    6.  Make the genrules more compact using pythonic list construction.

Can build with Bazel now

Build with Bazel.

Merge pull request #261 from pcc/fix3

Compute base addresses from program headers while reading /proc/self/maps.

Compute base addresses from program headers while reading /proc/self/maps.

We previously had logic to compute the base address from program
headers as part of symbolization. The problem is that we need a correct
base address earlier in order to adjust a PC into the image's address
space, as these addresses can appear in unsymbolized output.

There was previously an assumption that only the mapping that
was lowest in the address space did not need to be adjusted. This
assumption is not guaranteed (for example, the kernel may choose to
map an ET_DYN lowest) and in fact turned out to be wrong in binaries
linked with lld because the first mapping is read-only.

The solution is to move the program header reading logic into the
code that reads /proc/self/maps.

There is a change in semantics for clients that install a callback
using the InstallSymbolizeOpenObjectFileCallback function. Any such
clients will need to return a correct base address from the callback
by reading program headers using code similar to that in the function
OpenObjectFileContainingPcAndGetStartAddress.

Merge pull request #106 from dimhotepus/master

Cache strlen outside of cycles (PVS-Studio)

Merge pull request #260 from ukai/shell_escape

Shell escape arguments to /bin/mail.

Shell escape arguments to /bin/mail.

Style fixes for consistency

Merge pull request #245 from DariuszOstolski/issue211

Fix username lookup in case of missing USER environment variable

Merge pull request #254 from DariuszOstolski/issue253

#253: Use MS preprocessor idiom to disable warning

#253: Use MS preprocessor idiom to disable warning

Merge pull request #226 from sergiud/zero-allocation-log-failure

Zero allocation fix

fixed gcc logging failure

Added HAVE_PWD_H to cmake build

Fix username lookup in case of missing USER environment variable

Merge pull request #228 from sergiud/dll-export-fix

Fix for missing exports (fixes #227)

added missing exports (fixes #227)

Merge pull request #225 from jray272/remove-log-every-n-assert

Fix LOG_EVERY_N with clang -Wunused-local-typedef

Remove GOOGLE_GLOG_COMPILE_ASSERT

This compile time assert is no longer used anywhere in glog. Remove
it.

Merge pull request #224 from jray272/run-windows-script

Run src/windows/preprocess.sh to pick up latest logging.h.in changes

Fix LOG_EVERY_N with clang -Wunused-local-typedef

Glog uses a pre-C++11 compile time assert to verify the validity of
the severity parameter for LOG_EVERY_N. Unfortunately, some compilers
will complain about the usage of LOG_EVERY_N with
"-Wunused-local-typedef" due to the way the compile time assert is
constructed. This makes it impossible to use LOG_EVERY_N with this
warning treated as an error.

The fix simply removes the assert entirely. This is safe to do since
you can't put anything invalid into the severity parameters without
generating a compile error elsewhere. This has been safe to do ever
since the GLOG_ prefixes were added as part of 6febec361e.

Fixes #223

Update Windows logging.h based on 2df0ca34aa

Commit changes to src/windows/glog/logging.h that were missed in
2df0ca34aa. Because a change to src/glog/logging.h.in was made,
src/windows/preprocess.sh needed to be run.

Add Jim Ray to CONTRIBUTORS

Merge pull request #218 from andschwa/remove-sln

Update Windows docs

Update Windows readme for CMake

Update gitignore for CMake and Visual Studio

Delete ancient Visual Studio Solution file.

Merge pull request #216 from google/revert-120-master

Revert "configure.ac: Allow user to disable gflags"

Revert "configure.ac: Allow user to disable gflags"

Merge pull request #120 from ribalda/master

configure.ac: Allow user to disable gflags

Merge pull request #209 from sergiud/cmake-icc-fix

cmake: fixed gflags namespace detection (fixes #193)

cmake: fixed gflags namespace detection (fixes #193)

This commit fixes gflags namespace detection using Intel compiler and ensures
the generated try_compile command line does not produce malformed paths.

Merge pull request #153 from sergiud/cmake-gnu-install-dirs

cmake: do not hardcode relative install directories

Merge pull request #152 from sergiud/cmake-project-name

cmake: changed project name from google-glog to glog

Merge pull request #157 from sergiud/cygwin-support

Cygwin support

Merge pull request #158 from sergiud/zero-allocation

[RFC] reduce heap memory allocations to zero

Merge pull request #145 from pixelb/rate-limit-posix-fadvise

rate limit calls to posix_fadvise()

reduce memory allocations to zero

Merge pull request #199 from v1bri/brian-fix-gflags-link

Use gflags ALIAS instead of ${gflags_XXX} vars

Merge pull request #168 from andschwa/windows-stacktrace

Port stack tracing to Windows

Support signal handler on Windows

Link to DbgHelp using pre-processor directive

This method ensure that all users of glog get automatically linked to
the DbgHelp library without needing to set compiler flags.

Export GetStackTrace

Necessary when building with BUILD_SHARED_LIBS=1.

Support symbolizer and demangler on Windows

Support stacktrace on Windows

Copy stacktrace_generic-inl.h for Windows

Add Andrew Schwartzmeyer to authors and contributors

Specify gflags version in CMakeLists.txt

Use gflags ALIAS instead of ${gflags_XXX} vars

The gflags project updated their CMake config last year with a
`gflags` ALIAS target. This can be used instead of the legacy
`${gflags_LIBRARIES}` and `${gflags_INCLUDE_DIRS}` variables. It also
looks cleaner.

Fixes #198

Merge pull request #188 from shinh/remove-configure

Remove files generated by autotools

closes #165

Remove files generated by autotools

for #165. I think not many projects have them in their repo.

Update the version in CMakeLists.txt to 0.3.5

Merge pull request #186 from shinh/backport

Backport internal changes

Add a hack for naive include scanners

Define OS_LINUX only if it's not defined yet

symbolize: Allow 4kB stack consumption on PPC64

x86 stacktrace: Use __builtin_frame_address if possible

Relax test for symbolize

Don't rely on an internal-linkage extern "C" function having an
unmangled name. This isn't required by the ABI, and in fact is not
valid for a conforming compiler(!). Instead, allow symbolization to
produce either a mangled or an unmangled name here.

Use LOG(FATAL) instead of CHECK(false)

Run src/windows/preprocess.sh

glog: release 0.3.5

Merge pull request #179 from renning22/master

CHECK_NOTNULL works with smart pointers when compiled in C++11.

CHECK_NOTNULL works with smart pointers when compiled in C++11.

Merge pull request #169 from mayah/windows-no-return

Add __declspec(noreturn) on Win

Add __declspec(noreturn) on Win

Currently cl.exe doesn't know LOG(FATAL) exits the program. Set
__declspec(noreturn).

Merge pull request #164 from yoshisatoyanagisawa/dcheck_always_on

DCHECK_ALWAYS_ON to make D* enabled under NDEBUG

DCHECK_ALWAYS_ON to make D* enabled under NDEBUG

The macro NDEBUG could be automatically defined for release build on
some build environments (e.g. MSVC).  If we use NDEBUG as a key to
distinguish using DCHECK as CHECK (I call this DCHECK is enabled) or
not, we cannot make DCHECK enabled for release build on such
environments.

Considering people use a program with glog for presubmit testing or
dogfooding, they should need to do release build with DCHECK enabled.

fixed cygwin compilation errors

cmake: do not hardcode relative install directories

cmake: changed project name from google-glog to glog

This fixes the default install location on Windows. Specifically,
find_package works now out-of-the box with the default
CMAKE_INSTALL_PREFIX.

rate limit calls to posix_fadvise()

There can be a large kernel overhead involved in POSIX_FADV_DONTNEED.
There is no point in calling this per item logged, so rate limit
to at most once per 2MiB written.

With a simple test program that logs 100K items at WARNING level:

Before:

  $ time strace -c -e fadvise64 log.test \
    -log_dir=/dev/shm -logtofiles=true -logtostderr=false
  % time     seconds  usecs/call     calls    errors syscall
  ------ ----------- ----------- --------- --------- ----------------
  100.00   12.522509         125     99957           fadvise64
  ------ ----------- ----------- --------- --------- ----------------
  real    0m52.671s
  user    0m2.194s
  sys     0m44.022s

After:

  $ time strace -c -e fadvise64 log.test \
    -log_dir=/dev/shm -logtofiles=true -logtostderr=false

  % time     seconds  usecs/call     calls    errors syscall
  ------ ----------- ----------- --------- --------- ----------------
  100.00    0.000759         152         5           fadvise64
  ------ ----------- ----------- --------- --------- ----------------
  real    0m4.206s
  user    0m1.436s
  sys     0m3.153s

Fixes issue #84

configure.ac: Allow user to disable gflags

Under some circumstances like cross-compilation, the user might not want
to enable support for gflags.

This patch allows support for --without-gflags

Signed-off-by: Ricardo Ribalda Delgado <ricardo.ribalda@gmail.com>

Update github.com/sergiud in AUTHORS/CONTRIBUTORS

Merge pull request #132 from sergiud/cmake-disable-unittests

cmake: allow to disable building unit tests

cmake: allow to disable building unit tests

This commit introduces a CMake `BUILD_TESTING` boolean option that
allows to disable unit tests. This is especially useful in superbuilds
where typically only the main library is built. By default, this option
is enabled.

Merge pull request #127 from MinGW-caffe/master

try to avoid the error "conflicting declaration 'typedef DWORD pthread_t'" etc. in MinGW

Add MiniLight to AUTHORS/CONTRIBUTORS

Merge pull request #123 from yying/master

Set sinks_ to NULL after deletion in LogDestination::DeleteLogDestinations

Add Andy Ying to AUTHORS/CONTRIBUTORS

Merge pull request #116 from pcc/fix2

Fix autotools build.

Merge pull request #98 from willyd/relocatable

Added $<BUILD_INTERFACE> to gflags include directory

Add Guillaume Dumont to AUTHORS/CONTRIBUTORS

Merge pull request #129 from sergiud/cmake-PIC

cmake: enable position independent code (PIC)

Merge pull request #130 from sergiud/cmake-warning-fix

cmake: fixed clang warning propagation to gcc

cmake: fixed clang warning propagation to gcc

cmake: enable position independent code (PIC)

try to avoid the error "conflicting declaration 'typedef DWORD
pthread_t'" etc.

NULL sinks_ after deletion to prevent dangling pointer

Add Peter Collingbourne <pcc@google.com> to CONTRIBUTORS

Merge pull request #115 from pcc/fix1

symbolize: Calculate a module's zero VA using program headers.

Fix autotools build.

It looks like commit 3c49b93 modified the auto-generated file src/config.h.in
to add a definition of macro GOOGLE_GLOG_DLL_DECL. One of the autotools
reverts this change upon running "make", causing the build to fail when a
source file includes demangle.h.

To fix the problem, revert the change to src/config.h.in and include
glog/logging.h from demangle.h which provides a definition of that macro.

symbolize: Calculate a module's zero VA using program headers.

Previously we were using a module's "start address", i.e. the
address at which the module's executable region was mapped, as the
zero virtual address, i.e. the address from which the DSO's virtual
addresses are calculated. This works fine for DSOs created by the
bfd and gold linkers, which will emit a PT_LOAD directive into the
program header which loads the executable region at virtual address
(p_vaddr) and file offset (p_offset) 0.

However, the lld linker may place a read-only region before the
executable region, meaning that both p_vaddr and p_offset for the
executable region are non-zero. This means that any symbols resolved
by the symbolizer are resolved to an incorrect virtual address. To
correctly calculate the address corresponding to virtual address zero,
we need to take into account p_vaddr and p_offset.

Specifically, the calculation starts with the "base address", i.e. the
start address minus the file offset. To get from the base address to
virtual address zero, we first add p_offset. This gives us the mapped
address of the start of the segment, or in other words the mapped
address corresponding to the virtual address of the segment. (Note
that this is distinct from the start address, as p_offset is not
guaranteed to be page aligned.) We then subtract p_vaddr, which takes
us to virtual address zero.