remove SFMTXUNLOCK() no-op

remove no-op SFMTXLOCK()

remove useless extern declarations

These variables already have extern linkage.

remove now unused vthread.h

remove unused sf_mutex field from Sfextern_t

remove unused mtx parameter to SFCLEAR() macro

remove unused mutex field from SFIO structures

remove unused Sfonce functionality

remove SFONCE() no-op

remove vmtx mutex no-ops

remove unused SFIO vthread prototypes

remove unused sftell()

remove unused _sfstrtod()

Merge branch 'orientation' into 'master'

Combine `orientation` attr for graphs and nodes

See merge request graphviz/graphviz!1640

Combine `orientation` attr for graphs and nodes

This simplifies the generation quite a lot, gets rid of the duplicated
anchors for differently-named.

Removes a little confusion too, in my opinion.

It's not perfect: they have different datatypes.

Merge branch 'stray-a' into 'master'

attrs.html: Close TT tag

See merge request graphviz/graphviz!1639

attrs.html: Close TT tag

Merge branch 'stray-a' into 'master'

attrs.html: Remove stray anchor close tag

See merge request graphviz/graphviz!1638

attrs.html: Remove stray anchor close tag

Merge branch 'ps' into 'master'

attrs.html: Rename 'kind' to 'types'.

See merge request graphviz/graphviz!1636

attrs.html: Rename 'kind' to 'types'.

Matches the title of the table, and colloquial use.

Merge branch 'ps' into 'master'

attrs.html: Rename attr.uses to used_by

See merge request graphviz/graphviz!1635

attrs.html: Rename attr.uses to used_by

Matches table header.

Makes more sense when you're looking at an attribute definition

Merge branch 'ps' into 'master'

Add consistent indentation to attrs.html

See merge request graphviz/graphviz!1634

Add consistent indentation to attrs.html

No-op as far as appearance goes.

But makes looking at diffs much easier!

Merge branch 'ps' into 'master'

attrs.html: don't special-case double/int/string

See merge request graphviz/graphviz!1633

attrs.html: don't special-case double/int/string

Instead, add (very basic) type definitions for them.

This simplifies the template.

Merge branch 'ps' into 'master'

attrs.html: Stop special-casing bitmap output

See merge request graphviz/graphviz!1632

attrs.html: Stop special-casing bitmap output

No-op change.

Merge branch 'ps' into 'master'

attrs: Remove special-casing postscript template

See merge request graphviz/graphviz!1631

attrs: Remove special-casing postscript template

Just spell out postscript in the source

Merge branch 'smattr/07caebcd-6088-4479-89d4-80d864a567fa' into 'master'

speed up Neato

See merge request graphviz/graphviz!1619

squash a -Wfloat-conversion warning

inline and fuse a number of loops in majorization code

This accelerates the Neato example from #1652 by ~9.5%.

inline some vector computations and fuse two loops

This slightly accelerates a current test case by ~0.1%. Related to #1652.

Merge branch 'nslimit6' into 'master'

Remove no-op bool case

See merge request graphviz/graphviz!1628

Remove no-op bool case

It behaved the same as the 'else'

Merge branch 'nslimit5' into 'master'

Replace TT with CODE element

See merge request graphviz/graphviz!1627

Replace TT with CODE element

TT is deprecated

Merge branch 'nslimit4' into 'master'

attrs.html: clean ID and NAMEs

See merge request graphviz/graphviz!1626

attrs.html: clean ID and NAMEs

- Convert NAMEs to IDs, as recommended for non-form elements.
https://stackoverflow.com/a/7470325/171898
- Quote HREFs and IDs

Merge branch 'nslimit3' into 'master'

Remove two-spacing from front of HTML continuation

See merge request graphviz/graphviz!1625

Merge branch 'nslimit2' into 'master'

Simplify parsing, now all attributes have one name

Closes graphviz.gitlab.io#64

See merge request graphviz/graphviz!1623

Remove two-spacing from front of HTML continuation

No need for this, just adds complexity to the parser.

Merge branch 'nslimie1' into 'master'

Split nslimit and nslimit1 attributes

See merge request graphviz/graphviz!1622

Simplify parsing, now all attributes have one name

Closes #64

Split nslimit and nslimit1 attributes

They're the only attributes that don't already have their own sections,
they complicate parsing and emitting, and on balance I think they're
better explained separately (but next to each other).

Closes #64

Merge branch '9ddbb9ea-f0a4-481d-9c40-b9709aa1680c' into HEAD

squash a -Wimplicit-fallthrough warning

GCC recognizes "fall through" and "fallthrough" comments to suppress this
warning but not "falling through."

remove now-unnecessary use of volatile on some locals

Related to #1801.

remove now-unused ljbuf

Related to #1801.

undo an unnecessary use of setjmp/longjmp

This code actually correctly marks the variables that are affected by the
setjmp/longjmp as volatile, but there is still no need for this complexity. A
standard error code return does the trick much simpler. Related to #1801.

Merge branch 'remove-fedora-29-and-30-jobs-from-ci' into 'master'

Remove Fedora 29 and 30 jobs from CI

See merge request graphviz/graphviz!1617

Remove EOL Fedora 30 Dockerfile

Remove EOL Fedora 29 Dockerfile

Stop building & testing for EOL Fedora 30 in CI

Stop building & testing for EOL Fedora 29 in CI

Merge branch '91908b3f-1fea-401e-a238-435c742d3589' into HEAD

fix Lefty's reliance on dup() internals

This addresses the following Coverity warnings:

  Error: RESOURCE_LEAK (CWE-772): [#def39]
  graphviz-2.40.1/cmd/lefty/os/unix/io.c:362: open_fn: Returning handle opened by "dup".
  graphviz-2.40.1/cmd/lefty/os/unix/io.c:362: leaked_handle: Failing to save or close handle opened by "dup(fd[1])" leaks it.
  #  360|           panic2 (POS, "ptyopen", "cannot fork");
  #  361|       case 0:
  #  362|->         close (fd[0]), close (0), dup (fd[1]);
  #  363|           close (1), dup (fd[1]), close (fd[1]);
  #  364|           execl (shell, shbname, "-c", cmd, NULL);

  Error: RESOURCE_LEAK (CWE-772): [#def40]
  graphviz-2.40.1/cmd/lefty/os/unix/io.c:363: open_fn: Returning handle opened by "dup".
  graphviz-2.40.1/cmd/lefty/os/unix/io.c:363: leaked_handle: Failing to save or close handle opened by "dup(fd[1])" leaks it.
  #  361|       case 0:
  #  362|           close (fd[0]), close (0), dup (fd[1]);
  #  363|->         close (1), dup (fd[1]), close (fd[1]);
  #  364|           execl (shell, shbname, "-c", cmd, NULL);
  #  365|           panic2 (POS, "ptyopen", "child cannot exec: %s\n", cmd);

  Error: RESOURCE_LEAK (CWE-772): [#def41]
  graphviz-2.40.1/cmd/lefty/os/unix/io.c:429: open_fn: Returning handle opened by "dup".
  graphviz-2.40.1/cmd/lefty/os/unix/io.c:429: leaked_handle: Failing to save or close handle opened by "dup(p1[1])" leaks it.
  #  427|                   panic2 (POS, "pipeopen", "child cannot exec: %s\n", cmd2);
  #  428|               }
  #  429|->         close (1), dup (p1[1]), close (p1[1]);
  #  430|           close (0), dup (p2[0]), close (p2[0]);
  #  431|           execl (shell, shbname, "-c", cmd, NULL);

  Error: RESOURCE_LEAK (CWE-772): [#def42]
  graphviz-2.40.1/cmd/lefty/os/unix/io.c:430: open_fn: Returning handle opened by "dup".
  graphviz-2.40.1/cmd/lefty/os/unix/io.c:430: leaked_handle: Failing to save or close handle opened by "dup(p2[0])" leaks it.
  #  428|               }
  #  429|           close (1), dup (p1[1]), close (p1[1]);
  #  430|->         close (0), dup (p2[0]), close (p2[0]);
  #  431|           execl (shell, shbname, "-c", cmd, NULL);
  #  432|           panic2 (POS, "pipeopen", "child cannot exec: %s\n", cmd);

Fixes #1823. Related to #1464.

Merge branch 'master' into 'master'

Since the string from strerror should never be modified, use const.

See merge request graphviz/graphviz!1618

Merge branch 'b2f0e4be-93c5-4d03-9a33-16c7f45988a8' into HEAD

remove unused sfsize()

remove unused sfset()

remove unused SFSEEK() macro

remove unused sfpurge()

remove unused sfprints()

remove unused sfpool()

remove unused sfpoll()

remove unused sfnotify()

remove SFIO references to missing directories

The only one of these that ever existed in the source tree, Sfio_dc, was removed
in 3cc75428f78e4b66554d3a266939ad3e6113a1f4.

Merge branch 'add-centos7-and-fedora31-cmake-build-and-test-jobs-to-ci' into 'master'

Add Centos 7 and Fedora 31 CMake build and test jobs to CI

See merge request graphviz/graphviz!1616

Add centos7-cmake-test job to CI

Add centos7-cmake-build job to CI

Add fedora31-cmake-test job to CI

Add fedora31-cmake-build job to CI

Move ubuntu18-04-cmake-test later in .gitlab-ci.yml

The Linux CMake build jobs are kept together after all the autotools
jobs and the test jobs should follow the same principle.

Add install of RPM package from Centos and Fedora CMake builds

Add generation of an RPM package from Centos and Fedora CMake builds

Install git version 2 instead of version 1 in Centos 7 Docker image

The option --date=format-local:... used in gen_version.py is not
supported in git version 1.

Install CMake 3 instead of CMake 2 in Centos 7 Docker image

Some constructs used in the CMake builds are not supported in CMake 2, e.g. the $<TARGET_OBJECTS:common_obj> generator expression.

See also:

* https://centos.pkgs.org/7/epel-x86_64/cmake3-3.17.3-3.el7.x86_64.rpm.html#howto
* https://stackoverflow.com/questions/48831131/cmake-on-linux-centos-7-how-to-force-the-system-to-use-cmake3

Add changelog notes for CMake dependency bump

Change minimum required CMake version to 3.1

The $<TARGET_OBJECTS:objLib> generator expression was introduced in
CMake version 3.1.

Merge branch '0f71798c-c756-4c99-8ef0-c88e42201de0' into HEAD

add a test case for sequential PS graphs

Related to #1845.

add changelog entry for #1845

Merge branch '9efca080-662e-4990-a1ff-40ebd0fd1530' into HEAD

remove unnecessary destructor

The compiler-provided default destructor is identical to this, except that the
compiler-provided one can also be inlined.

remove user configuration files in the Visio plugin

These are Visual Studio configuration files for local machine setup. They are
not intended to be committed to version control.

Since the string from strerror should never be modified, use const.

take an early exit when gvprintdouble-ing a "0"

In large graphs, the value being printed is frequently zero and the snprintf in
this function shows up as a minor hotspot. Profiling with
tests/regression_tests/large/long_chain this drops the total snprintf calls
within gvprintdouble from 924019 to 858017. This lowers the contributing runtime
of gvprintdouble to overall execution from 28.89% to 28.12%.

Merge branch 'B494AFA0-22B1-4119-A751-D39B7E9BEE9D' into 'master'

clean up some unused things

See merge request graphviz/graphviz!1603

remove commented out cleanup1 function

remove deprecated AppVeyor config

Windows CI now runs on Gitlab.

remove commented out legacy CI jobs

Merge branch 'b3d77ab1-ae13-4658-abf2-1dc937774152' into 'master'

remove long lived static global allocation

See merge request graphviz/graphviz!1604

remove now unused AF global

use a local instead of reusing global AF in gvrender_polyline

There is a static global, AF, that is reused for a number of gvrender functions,
but none need to retain previous data stored in this array. This hack presumably
was from a time when allocators were much slower. Refactoring this into a local
allocation makes this function thread safe and removes the need to unnecessarily
prolong the lifetime of this allocation, thus decreasing Graphviz memory usage.
This commit introduces a -Wshadow warning about AF, but that will be removed
when we soon remove the static global.

use a local instead of reusing global AF in gvrender_beziercurve

There is a static global, AF, that is reused for a number of gvrender functions,
but none need to retain previous data stored in this array. This hack presumably
was from a time when allocators were much slower. Refactoring this into a local
allocation makes this function thread safe and removes the need to unnecessarily
prolong the lifetime of this allocation, thus decreasing Graphviz memory usage.
This commit introduces a -Wshadow warning about AF, but that will be removed
when we soon remove the static global.

use a local allocation instead of global hack in gvrender_polygon

There is a static global, AF, that is reused for a number of gvrender functions,
but none need to retain previous data stored in this array. This hack presumably
was from a time when allocators were much slower. Refactoring this into a local
allocation makes this function thread safe and removes the need to unnecessarily
prolong the lifetime of this allocation, thus decreasing Graphviz memory usage.
This commit introduces a -Wshadow warning about AF, but that will be removed
when we soon remove the static global.

remove commented out code in dia plugin