Lose the empty doxygen markers
- nothing wrong with comments but empty comment placeholders
  are not exactly useful

Put some consistency to python object creation
- all type object creation goes through foo_Wrap() which handle OOM
  and all type specific initialization

Make all python object creation wrappers return PyObject pointers
- this way the only place where casts are needed are in the wrapper itself

Dont leak memory on rpm.ds iteration

Lose unnecessary next() methods
- python adds next() methods for objects supporting iterators

All rpm-python iterators are self-iterators, just use PyObject_SelfIter

Dont leak memory from rpm.dsSingle()

Simplify fiFromHeader()
- just call the fi object, no need to manually redo all this stuff

Simplify dsFromHeader()
- just call the ds object, no need to manually redo all this stuff

Sanitize python object -> tag number exception handling
- raise exception in tagNumFromPyObject(), not in 12 different callers
- check against RPMTAG_NOT_FOUND consistently instead of -1 and whatnot
- unknown tags are value, not key or type errors

Move allocations out of rpmps object init method
- tp_init can be called several times, allocating from there leaks memory
- tp_init gets called automatically on object creation, dont call manually
- nothing to initialize for rpmps...

Move allocations out of rpmds object init method
- tp_init can be called several times, allocating from there leaks memory
- tp_init gets called automatically on object creation, dont call manually

Move allocations out of rpmfi object init method
- tp_init can be called several times, allocating from there leaks memory
- tp_init gets called automatically on object creation, dont call manually

Remove useless rpmts python initialization

Realize the remaining bits of direct rpmdb interface are dead too
- the code hasn't been enabled in a long long time despite being built...

Use Py_RETURN_NONE macro for returning None everywhere

Eliminate unnecessary custom object allocation functions
- these are just calling python defaults, no point at all...

Use generic python get/set attribute functions directly where appropriate
- no point in wrapping all this stuff in our own functions

Python initialization spring clean
- split rpmtag enum additions to seprate function
- PyModule_AddIntConstant() does exactly what we need for the rest

Make the python object structures opaque

Lose the utterly broken rpmio FD_t wrappers

Lose the useless python debug stuff

We dont support ancient python versions...

Kick out mummified rpmdb python interface remnants

Missing <string.h> include

Clean up + fix bad recno erasure in python ts.addErase()
- now that we can loop over RPMDBI_PACKAGES matches, the code can
  be simplified quite a bit
- walk through the iterator always to get a reliable installed status,
  previously ts.addErase() with bad record number wouldn't raise exception

Remove no longer necessary RPMDBI_PACKAGES special casing

Fix endless loop on RPMDBI_PACKAGES iteration when offset specified

Unbreak --tid and --querybynumber
- broken ages ago somewhere between 4.4.x and 4.6.x by typing changes
  presumably
- C99 strtoul() can figure the base on its own, remove unnecessary work

DB_INCOMPLETE was removed in BDB 4.1 and we dont support older versions

Remove ancient, unused code to filter duplicates on rpmdb rebuild
- enabling this as-is would be murderous to multilib systems
- getting duplicates in the db would be a bug elsewhere

Rip out dependency caching
- it doesn't speed up things that much, is broken in some chroot
  scenarios and is ugly ugly hardwired BDB hackery where it doesn't belong

Move rpmdb home creation from backends to one level higher
- this is a common operation for any backends, avoid duplication
- only try to create it once instead of every index open, and if it
  doesn't exist and cant be created, error out instead of trying to continue
  in vain

Add an internal API for getting rpmdb home wrt chroot state, use it

Remove pointless per-index api version

DB config stuff not needed outside dbconfig.c, make them static
- additionally make the popt struct const since it can...

BDB is not optional at least currently, remove useless ifdefs

Remove useless no-op "teardown" and "usecursors" config bits

Remove BDB RPC "support"
- it's insecure, used by nobody (its been rather broken for a long
  time) and all, but BDB 4.8.x removed RPC support from upstream so we
  cant very well support in the future even if we wanted to

Remove XA "support" from the backend
- it's removed from BDB 4.8.x and this has hardly ever done anything anyway

Remove unused db/dbenv methods "documentary" from the source

Use db3Free() for on sqlite backend close too

Always populate dbi->dbi_file on db3New()
- Only the actual db->open() wants NULL for temporary databases, everywhere
  else we want the tag name. Handle the temporary case where it counts
  instead of complicating things everywhere for the one special case.

Mop up leftover root/home foobar

Per-dbi root and home dont make sense, always use rpmdb paths

Sigh.. when not using environment, absolute paths are required

Always open database files relative to database home
- This allows the db to move freely wherever (such as across chroots,
  RhBug:507309 and pile of related issues) .. and is even recommended
  by BDB documentation. Well duh.

EXIT_FAILURE being defined is already handled in system.h

Compare pointers against NULL, not 0

Have BDB use our allocation routines now that we can
- rfree() prototype doesn't match free() but it doesn't matter in this
  case as it's just a convencience wrapper with no extra functionality
  and setting NULL is ok according to BDB docs
- db->set_alloc() can't be used if environment is used, fix the condition

Add runtime settable custom memory allocation failure callback routine
- lets API users perform theirn own cleanup / exit through their own
  routines in case of allocation failure, or even try to free up
  some memory and retry allocation

Always use build + use our own allocator functions
- Consistent across platforms and will allow some things macros dont and
  memory checkers these days are smart enough to get decent output anyway
- Minimal namespacing with r-prefix, add compatibility macros to system.h
  for now so we dont have to change the entire codebase for this
- Also make rpmutil.h where the declarations and gcc __attribute__ macros
  are available everywhere

Eliminate unnecessary variable

Remove unused local variable

Remove db environment on close on chrooted operations
- As we open the db from outside the chroot, the environment ends up
  containing paths that are not valid once we enter the chroot causing
  dumb issues like RhBug:513699 and RhBug:507309.
- We'd be better off removing the environment always after access but
  limiting to chroots for now as the rpmdb open/close path is full of races
  when environment is not present. Chroots are somewhat special environemnts
  anyway and typically not concurrently accessed so it's less of an issue
  there.
- While this still has all sorts of issues, it at least leaves the rpmdb
  in chroot in a functional state after initial install.

Move the default transaction lock out of BDB environment namespace
- We dont want the transaction lock to get mixed with the BDB environment,
  nor do we want it wiped out with it as the ts lock protects things
  *around* the environment. Unlike the environment, any fcntl locks on
  the file get released when process dies, so if there is a lock,
  there's a process alive holding it and you'd better not remove it.
- Use the same lock path in macro configuration and built-in fallback
- Make it a hidden file to avoid people confusing lock file existence
  with it actually being locked

There's a function to test string equivalence...

Throw out useless db statistics structs
- besides being completely unused, these structs are defined by BDB
  public headers, no need to mirror them in rpm

Throw out pile of unused "stuff" from 2002

Minor cleanup to newRpmdb()
- no need to free what hasn't been allocated yet

One complicated way to write xstrdup() .. pff
- sqlite backend uses dbi subfile for table name, leave it alone although
  it is the same as the file name...
- allocate + free both file and table name separately

Eliminate long since unused db subfile from bdb code

Remove unnecessary arguments to db init and fini
- bdb environments are per db_home, not per file so dbfile etc dont make
  sense here

Also skip to be erased files in "netsharedpath"
fixes rhbz#494640

Initialize rpm earlier in rpm2cpio (RhBug:523260)
- url retrieval requires macros to be loaded

Take non-gi errors into account too (SuseBug:527191)
- patch from OpenSUSE / Michael Schroeder

Allow absolute paths in file lists again (SuseBug:535594, RhBug:521760)
- patch from OpenSUSE / Michael Schroeder
- build-time generated file lists should be placed in the build directory,
  but at least one valid use case for this is things like %files -f %{SOURCE10}

find-debuginfo.sh: use permissions 444 for all .debug files

Signed-off-by: Roland McGrath <roland@redhat.com>

Don't add dependency type into file requires and provides (RhBug:523282)
- unintended change in commit 3d8656f040d2c780c88d0cd63594898f0605a6fa

Clean up rpmalAllFileSatisfiesDepend() a bit
- avoid multiple strrchr() calls
- avoid _constfree(), the string size is known from strrchr() so we dont
  need to allocate on heap at all

Single point of exit for doSetupMacro() to clean up freeing allocations

Oops, accessing the wrong header in processBinaryFiles()
- dumb mistakes from commit 4177fdffd6e7414123062e8716afb22596608531

Rip the broken asynchronous psm machinery
- it was only "used" for the ldconfig optimization and actually been
  completely broken since the ordering rewrite

Rip the broken ldconfig optimization hack (RhBug:513224)
- it doesn't optimize much and what's worse, it sometimes optimizes out
  calls that would've been necessary

Rip unnecessary headerDump() and the last ancient artifacts using it
- xml format dump gives an output that can actually be further processed

Rip unused and useless headerGetLangs()
- nothing in the world uses it and the info is available through
  regular headerGet() anyway

Rip unused and useless headerUsageCount()

Eliminate unused goo from findEntry()

Handle NULL header in doHeaderUnload()
- avoids having to separately check elsewhere and avoids segfaulting
  on headerUnload() and headerReload() on NULL headers

Eliminate pointless increment

Eliminate yet more unused calculations from header code...

Eliminate unused pad calculations from header code

Eliminate yet another unused variable in header code

Eliminate last remaining leftovers from "8/98 bug" workaround
- the ieprev calculations aren't used for anything, should've been in
  commit 4d0d54492c7d539cc806ca9e7fa59e919c28e45d really

Eliminate unnecessary variable & related goo from regionSwab()

Initialize diff on declaration where it fits comfortably on one line

Move variable to block local scope, not needed elsewhere

Remove unnecessary assignment, diff is block local scope here

Reduce the number of exit points in parseFormat() to error and ok cases
- avoid a dozen redundant freeFormat() calls and dont bother NULLing
  local variable at the end

Add -Wempty-body to default options if gcc supports it
- ...to help protect against screwups like
  2ee45ef12aaa958b78c56b6d83bb18aa6a477d15

Eliminate unused variable

Eliminate unused variable

Minor rpmfcGenerateDependsHelper() cleanups
- initialize at declaration time, remove unnecessary variables etc

Move rl to the scope where its needed, dont bother assigning NULL

Single point of exit for doPatch() to clean up freeing

Eliminate bogus variable initialization
- i isn't used outside the loop at the end, move it there...

Add noreturn annotations to javadeps

Help compilers/analyzers a bit with luaL_error()
- luaL_error() never returns but as other tools dont know this it
  raises false alarms

Add noreturn annotations to argerror() helper functions

Add noreturn annotation to die() .. which itself should die