open_databases() and close_databases() are now cdb_open_databases() and cdb_close_databases()

usetable.c: moved CheckIfAlreadySeen() out of database.c

Also remove cdb_allocate_tsd(), cdb_free_tsd(), cdb_cull_logs()

It is important to get this right because it's going to become a driver interface for databases

Removed checkpoint_thread() prototype from database.h , this is not a real function anymore

database.h: assert that it is an internal API that should remain consistent across multiple back ends.

Renamed database.c to database_bdb.c

serv_nntp.c: change everything to C99 comment style

Marked code that will require some rework for LMDB.

When BDB returns data, the caller owns the memory and is required to
free it, which we usually do.  I marked the places where we take ownership
of memory pointers and then set the BDB ptr to NULL.  If we switch to
LMDB, those pointers remain valid forever, so we cannot simply free them
later.

point out sessions that are timed out but non-idle

verify that 'begin' and 'end' lines were found

began work on enumerating attachments in the mail view

Enumerate attachments in JSON message output

Merge branch 'ConstStr_Access_Checks' into 'master'

IMAP memory issues with use of ConstStr

See merge request citadel/citadel!5

Release version 988 generated by do-release.sh

ctdldump/ctdlload: -h now specifies the top-level Citadel directory, just like all the other programs, and defaults to the compiled-in Citadel directory, just like all the other programs

More reliable search for guesstimezone.sh

Release version 987 generated by do-release.sh

ctdldump/ctdlload: more testing

ctdlload: finished remaining record types; need to test

skip the database open for the unused table to keep it from erroring out

ctdlload: euidindex

ctdlload: usersbynumber

add some //FIXME tags where needed

import fulltext index records

stub functions for new import routines

Export euidindex records

export fulltext records

Both banners now use the same color scheme

Go back to exporting dir and usersbynumber records.

We will also be exporting fulltext, euidindex, etc records.
Re-generating them on the target server after import is just way too time consuming.

When exporting visit records, if there is corrupt (non-printable) data in seen/answered arrays, cut that out before exporting

Release version 986 generated by do-release.sh

Fixed bad import in thing

Release version 985 generated by do-release.sh

Aggressively sync

Be more aggressive about closing the databases during a segfault

Try to close the databases if we segfault

Release version 984 generated by do-release.sh

ctdldump/ctdlload: fixed on Raspberry Pi, reintroduced to build

add prototype for b64unalphabet() to silence warning

Merge branch 'Memory_Leak_SmtpClient' into 'master'

Memory leak in SMTP Client string trim

See merge request citadel/citadel!4

Release version 983 generated by do-release.sh

Release version 982 generated by do-release.sh

Release version 981 generated by do-release.sh

IMAP memory issues with use of ConstStr

Had a coredump in IMAP Search parsing.  Not sure of root cause, but parameter string parsing
  of IMAP Search coredumped on ConstStr pointer content access.

When ConstStr are created, pointer is null and length is zero.  Some code accesses the pointer
  contents without checking length first.  Not sure how parameter parsing passed a zero length
parameter, but added length checks before accessing throughout IMAP code to be safe.

While adding checks, found a typo in checking string for end of string in imap_list().
  Was using address of character instead of looking for null termination.  Corrected.

Memory leak in SMTP Client string trim

serv_smtpclient.c: Removed unused variable 't' from trim_response() which was malloc'ing memory

Removed serv_migrate.c and ctdlmigrate.  They are replaced by ctdldump and ctdlload.

remove extra msgs

ctdldump/ctdlload is functionally complete.

happy happy: convert_msgtext() also works for convert_bigmsgs()

ctdlload: import use table records

citserver: regenerate secondary indices if the regenerate_secondary_indices config key is set to nonzero

ctdldump/ctdlload: removed tables that can be re-generated by the server.

Added a new config key to the server that forces it to do that the next time it is run.

Beautiful startup banners in vivid color

ctdlload: import visits

import msglists

ctdlload: import floor table

ctdlload: import room records

Replaced all the pointer arithmetic in getuser/putuser with readable.

Pretty statistics

Now I have a machine gun that can write ctdluser records.  Ho-ho-ho.

Moved makeuserkey() into its own file because we need it in ctdlload

ctdlload: load message metadata

ctdlload: load msgtext

ctdlload: skeleton code for table ingestion

./ctdldump -y -h./data | ./ctdlload -y -h./newdata

./ctdldump -y -h./data | ./ctdlload -y -h./newdata

ctdlload: wrote the code to ingest dump lines

Cleaned up config dump so there are no nulls in the dump

ctdldump: convert binary to base64 instead of hex

ctdldump: dump config records.  This is now functionally complete.

ctdldump: export usersbynumber (secondary index)

ctdldump: export euidindex

Require Berkeley DB v18 or newer

Removed some stray comments

ctdldump: export use table

Export directory records

ctdldump: export visit table

ctdldump: export floor records

ctdldump: export room records

ctdldump.c: export users

Wrote the first of 14 export functions (msgmain)

Makefile: added some more pretty things

ctdldump.c: skeletonize

ctdl3264: grab files from this commit if we bring it back.

I'm going to experiment with replacing this somewhat kludgey utility
with proper dump and load utilities.  Then we can use it as a more
reliable replacement for ctdlmigrate.  Who the hell needs online migration
on a running server anyway?

bootstrap: fixed incorrect source file name

forum_view.c: log unhandled headers

citadel/Makefile: parallelizable build with wildcards

view_mail.js: set variables to local scope

webcit-ng: drag-and-drop mail to other folders is complete.

The fact that this could be completed without writing any new C code is
a good indicator that the DAV foundation is in good shape.

DAV MOVE method wasn't documented.  Documented it.

Carry list of divs being moved in drag and drop

webcit-ng makefile now handles changes to webcit.h properly

view_mail.js: handle drag of row that was not selected.

If the row being dragged was previously selected, we will drag all
selected rows and show that count next to the icon.  If the row being
dragged is NOT selected, we drag only that row and set count to 1.

This appears to make the behavior consistent with other webmails.

webcit-ng/Makefile: made it awesomer

client_passwords.c: style cleanup

textclient: indent -kr -i8 -brf -bbb -fnc -l132 -nce

Removed stub functions that are determined not to be needed.

Highlight drop target folders

All events are firing correctly.  Ready to write the action.

Removed some tests that I tried and are no longer needed