Fix warnings in pkg.el

Reset symbol home packages

* lisp/emacs-lisp/pkg.el (delete-package): Set the package of
symbols whose home package is the deleted package to nil.
* test/src/pkg-tests.el (pkg-tests-delete-package):
(pkg-tests-use-package): Modify because we don't have export yet.

lkjlkj

* src/pkg.c (pkg_find_symbol): Simplify.
(Fpackage_percent_set_name): Prevent changing the name of standard
packages.
(Fpackage_percent_set_symbol_package): Allow nil for packages.

Print symbols habing a deleted home package

This can't happen in normal circumstances, but it might while
working on the package implementation itself.

* src/print.c (print_symbol): Handle symbol-packages that are
deleted, that is have a nil home package.
* src/lisp.h (PACKAGE_USE_LIST): New macro.

Fix error with ImageMagick

* src/image.c (imagemagick_filename_hint): Don't assume QCformat
is available.

shodow

* lisp/emacs-lisp/pkg.el (package-shadowing-symbols): Implement.
(shadow): Implement.
* src/pkg.c (Fpackage_percent_set_symbol_package): New function.
(syms_of_pkg): defsubr it.
* test/src/pkg-tests.el (pkg-tests-use-package): Fix byte compiler
warning.

do-symbols, do-external-symbols, do-all-symbols

* lisp/emacs-lisp/pkg.el (do-symbols): New macro.
(do-external-symbols): New macro.
(do-all-symbols): New macro.

Prepare for testing find-symbol

* src/pkg.c (pkg_find_symbol1): Remove.
(pkg_find_symbol): Lookup symbols differently.
* lisp/emacs-lisp/pkg.el: Prepare for find-symbol tests.
* test/src/pkg-tests.el (pkg-tests-use-package): New.

More work on Lisp and tests

Ongoing work on the Lisp side and tests

Uncomment tests after fixing bug#58714

Fix &key parameters called without arguments (bug#58714)

* lisp/emacs-lisp/cl-macs.el (cl--do-arglist): Check for missing
argument.
* test/lisp/emacs-lisp/cl-macs-tests.el (cl-&key-arguments): New test.

One more place with ":..." in C strings

* src/font.c (font_unparse_fcname): Use LISP_SYMBOL_NAME.
(font_filter_properties): Use LISP_SYMBOL_NAME.

Fix error opening Gnus gmail imap connection

The error is caused by comparing names of (possibly) keyword symbols
with a table of symbols names containing ":" for symbol names.

* src/process.c (set_socket_option): Use LISP_SYMBOL_NAME.

Fix some pkg-tests

Don't define tests that consist of should nil only.  Don't assume
packages are registered by make-package.

Fixes in package functions

* lisp/emacs-lisp/pkg.el (pkg-stringify-names): De-duplicate names.
(delete-package): Unregister package.

Improve Lisp interface on the C side of packages

* src/pkg.c (Fpackage_percent_set_name): Allow nil as name.
(Fpackage_percent_register): Return the package.

Expose package-%register to Lisp

* src/pkg.c (Fpackage_percent_register): New DEFUN.
(syms_of_pkg): defsubr it.

Mark module--test-assertions--call-emacs-from-gc unstable

This test succeeds for me with --enable-checking and fails otherwise
with a segfault.  See also comment there.

* test/src/emacs-module-tests.el
(module--test-assertions--call-emacs-from-gc): Mark unstable
if feature symbol-packages is present.

Remove an invalid eassert

* src/pkg.c (pkg_emacs_intern): Don't assert that symbol names
can never start with a colon.

Add tests for packagep and package-name

* test/src/pkg-tests.el (pkg-tests-packagep)
(pkg-tests-package-name): New tests.
(pkg-tests-list-all-packages): Expand test.

Fix cl-intern/cl-unintern tests

* test/src/pkg-tests.el (pkg-tests-cl-intern)
(pkg-tests-cl-unintern): Fix tests.

* src/lisp.h (pkg_error): Fix GCC warning in declaration.

Fix priting of :1

* src/print.c (print_symbol_name, print_symbol): Don't check for
symbol names looking like a number when we have already printed a
package prefix.
* test/src/editfns-tests.el (format-%s-keywords): Test for :1.

Make format %s for keywords compatible

* src/editfns.c (styled_format): Use LISP_SYMBOL_NAME.

Add test for format %s with keywords

* test/src/editfns-tests.el (format-%s-keywords): New test.

Make lisp/progmodes/elisp-mode-tests succeed

Shorthands are not supported with packages.  Add :expected-result
accordingly.

* test/lisp/progmodes/elisp-mode-tests.el (elisp-shorthand-read-buffer):
(elisp-shorthand-read-from-string):
(elisp-shorthand-load-a-file):
(elisp-shorthand-byte-compile-a-file):
(elisp-shorthand-completion-at-point):
(elisp-shorthand-escape):
(elisp-dont-shadow-punctuation-only-symbols): Expect to fail
if (featurep 'symbol-packages).

Provide 'symbol-packages'

* src/pkg.c (init_pkg_once): DEFSYM Qsymbol_packages.
(syms_of_pkg): Fprovide it.

Fixes for minibuf-tests

This was a remnant of a time when I made obarrays packages.

* src/minibuf.c (Ftry_completion): Handle case collection being obarray.
(Fall_completions): Same.
(Ftest_completion): And same again.

Fixes for obarray-tests

* src/pkg.c (pkg_emacs_unintern): Take args, and return a value
consistent with traditional unintern.

Fix printing of empty keywords

* src/print.c (print_symbol): Don't print ## for keywords with
empty symbol name.

print-tests failure for esoteric symbols

* src/print.c (print_symbol_name): Fix printing of symbols that
look like numbers.

print-test fixes

* src/print.c (print_symbol): Fix printing of empty symbol names
to be compatible.

Work on Lisp interface

Handle keywords in image specs

* src/image.c (parse_image_spec): Don't assume that keywords have
a ':' in their symbol name.

Remove a call to pkg_break

Add pkg_set_status and Lisp defun for it

Fix completion for new symbol table layout

De-duplicate packages for mapatoms

Introduce PACKAGE_NAMEX

PACKAGE_NAME Is already taken :-(.

Some cleanup in pkg.c and lisp.h

Intrdduce pkg_find_symbol

Print package prefixes right

Remove Lisp_Symbol::external

Remove unused function prototype from lisp.h

Change package symbol table layout

The symbol table now stored symbol as key, and symbol
status (:internal, :external) as values.

Quite some changes due to that.

Fix some warnings

hash_remove_from_table returns bool

* src/fns.c (hash_remove_from_table): Return true if something was
deleted.
* src/lisp.h: Change declaration.

Fix intern-soft

* src/pkg.c (conflicting_package): Remove unused function.
(pkg_emacs_intern_soft): If checking a symbol, return non-nil only if
we found exactly that symbol.

Fix missing ')'

Restore obarray.el to return vectors

New predefined hash table test for string-equal

* src/fns.c (cmpfn_string_equal): New.
(hashfn_string_equal): New.
(hashtest_string_equal): New.
(Fmake_hash_table): Recognize test type Qstring_equal.
(syms_of_fns): DEFSYM Qstring_qual.

Move more package stuff to Lisp

Move make-package to Lisp

* lisp/emacs-lisp/pkg.el: Implement make-package.
* lisp/obarray.el (obarray-make): Use make-%package.
* src/pkg.c: Various changes to move make-package to Lisp.

Make faces work with keywords not having : in symbol names

* src/xfaces.c (set_lface_from_font): Use LISP_SYMBOL_NAME.
(merge_face_vectors): Use LISP_SYMBOL_NAME.
(merge_face_ref): Use SYMBOL_KEYWORD_P instead of checking ':'.
(gui_supports_face_attributes_p): Use LISP_SYMBOL_NAME.

Add amcros for keyword symbols

* src/lisp.h (SYMBOL_KEYWORD_P): Returns true if symbol is a keyword.
(LISP_SYMBOL_NAME): Basically the same as Fsymbol_name, a shortcut.
* src/data.c (Fsymbol_name): Use SYMBOL_KEYWORD_P.
* src/pkg.c (pkg_intern_keyword): Use SYMBOL_KEYWORD_P.

Make internals of Lisp_Package visible to Lisp

Done to be able to do as much as possible from Lisp.

* src/lisp.h (Lisp_Package): Rename used_packages to use_list.
* src/pkg.c: Rename used_packages to use_list.
(Fpackage_percent_name): (Fpackage_percent_set_name):
(Fpackage_percent_nicknames):
(Fpackage_percent_set_nicknames):
(Fpackage_percent_use_list):
(Fpackage_percent_set_use_list):
(Fpackage_percent_shadowing_symbols):
(Fpackage_percent_set_shadowing_symbols):
(Fpackage_percent_symbols): New functions for getting and setting
internals of Lisp_Package.
(syms_of_pkg): defsubr them.

Work on defpackage

Increase default symbol-table sizes

* src/pkg.c (Fpackage_symbols): New function.
(syms_of_pkg): defsubr it.
(init_pkg_once): Up symbol table sizes.

DEFVAR some variables

* src/lisp.h: Remove extern declarations for some vars.
* src/pkg.c (syms_of_pkg): DEFVAR_LISP_NOPRO some variables.
*package*, *package-registry*, *emacs-package*,
*keyword-package*, package-prefixes

Support specifying package size

Revert some unimportant changes

Partially revert "Remove obarrays"

More scribbling

Revert "Dpn't assume symbol-name of keywords starts with colon"

This reverts commit bb6b5db2b7af3f1aaf8866d4dd6ce944d10244f8.

Add cö-symbol-name

Revert "Use make-package instead of make-vector"

This reverts commit 072e89afa119c0e35321edfb8bc0dfff89b6960d.

Fix printing uninterned symbols

symbol-name returning ':' for keywords

* src/data.c (Fsymbol_name): Return a name starting with ':' for
keywords.

Recognize vectors as obarrays

* lisp/obarray.el (obarray-make): Ignore parameter.
(obarray-size): Ignore parameter.
(obarrayp): Accept vectors.

Fix printing symbols

Fake obarrays

Fix printing of confusing symbol names

Merge remote-tracking branch 'origin/master' into pkg

More scribbling

Allow intern with ":xyz" again

Handle packages in completion

* lisp/minibuffer.el (completion-table-with-context): Add packagep
case.
* src/minibuf.c (Ftry_completion, Fall_completions),
Ftest_completion): Take into account that predicate functions used
with packages are the same that were used for obarrays.

Some scribbling

intern-soft with ':' trick

* src/lread.c (Fintern): Move everything to pkg.c
* src/pkg.c (pkg_emacs_intern): Handle ':' in symbol names.
(pkg_emacs_intern_soft): Ditto.

Mapatoms differently

Also, assume that in some cases, (intern ":xy") means that old code
wants to intern a keyword.

Workaround for something Stefan missed

See comment there for an explanation.

* src/alloc.c (xmake_pure_vector): Former make_pure_vector.
(init_vectors): Use it to allocate zero_vecgtor.

src/alloc.c: Remove all uses of `pure_alloc`

First step of removal of the purespace: stop using it.
The more delicate parts are the handling of 0-length strings and
vectors which we used to allocate in purespace but now need to be
allocated elsewhere, but the existing code makes us work harder to
allocate them in the normal way.

* src/alloc.c: Remove all uses of `pure_alloc`.
(init_strings): Alloc empty strings in the normal heap.
(init_vectors): Allocate the zero_vector in the normal heap.
(make_pure_string, make_pure_c_string, pure_cons): Rewrite to create
normal heap objects.
(find_string_data_in_pure, make_pure_float, make_pure_bignum)
(make_pure_vector, purecopy_hash_table): Delete functions.
(purecopy): Return without purecopying.

Use build_pure_c_string

* src/pkg.c (init_pkg_once): Use build_pure_c_string instead of
build_string.

Consider shorthands out of scope

* lisp/loadup.el ("emacs-lisp/shorthands"): Comment out.

Blabla

* src/lread.c (Fintern): Remove comment.
* src/pkg.c (pkg_emacs_intern_soft): Don't assume *package* if
a vector was passed in for a package.  Assert instead elsewhere.

Dpn't assume symbol-name of keywords starts with colon

* lisp/emacs-lisp/cl-seq.el (cl--parsing-keywords):
Don't substring the keyword symbol name.

Use make-package instead of make-vector

* lisp/cedet/semantic/lex.el (semantic-lex-make-type-table):
Don't use obarray functionality.

Remove Lisp_Symbol::next pointer

* src/lisp.h (struct Lisp_Symbol): Remvoe next field.
(next_free_symbol, set_next_free_symbol): New.
(set_symbol_next): Remove.
* src/alloc.c (init_symbol): symbol_free_list done differently.
* src/pdumper.c (dump_symbol): Don't dump Lisp_Symbol::next.

Remove obarrays

* lisp/emacs-lisp/eldoc.el (eldoc-message-commands): Make a package
instead of an obarray-vector.
* src/doc.c (Fsnarf_documentation): Don't use oblookup.
* src/eval.c (Fsignal): Debuggin helper code.
* src/font.c (font_intern_prop):  Don't use oblookup.
* src/lisp.h (intern_c_string): Delete inline function, use the one
in fread.c.
* src/lread.c: Remove everything realted to obarray, except Vobarray,
which is now set to Vemacs_package.
* src/minibuf.c (Ftry_completion, Fall_completions, Ftest_completion):
Accept packages.
* src/pkg.c: Adapted to other changes.

Use packages instead of obarrays in obarray.el

Initialize package system earlier

Mapping over symbols in a package, mapatoms

Don't register packages in make-package

Remove Lisp_Symbol::interned

Can now pdumg withput warnings from cl-defstruct

Revert the escaping of symbol names in lisp files

Can now pdump

More symbol reading

Intern keywords differently

Instead of something like (intern (format ":%s" ...)) do
(intern (format "%s" :keyword).  Likewise in C.

And more fixes