Attempt to fix GTK hid zooming/panning/flipping

hid/gtk: Make SwapSides action operate with passed coordinates.

Previosuly we used the last known X,Y coordinates from a mouse event seen
by the event handlers.

Revert "gtkhid-main.c: remove seemingly useless double conversion."

This reverts commit f4b511daed4bcc07df7445a345f2d19105b2be70.

They don't do anything.  The locals event_x/y are just copied
out then straight back into gport->pcb_x/y, none of which are
use at all in the interim.

Delete the debian/ directory

Including disro-packaging upstream is discouraged by debian policy, as it
makes packaging harder.

Deleting at the request of our upstream maintainers.

Revert "hid/ps/ps.c: avoid division by zero."

This reverts commit 6e24a51370c17eb99e8d823c670b77cbb9eba515.

Given a linewidth of zero, 'inf' was written as line width to the
PostScript file, which made the file invalid. Now we at least get
valid PostScript.

Not yet solved: why is there an attempt to draw a circle of zero
size?

This is related to bug LP1497963.

Revert "hid/ps/ps.c: deal with zero sized circles even better."

This reverts commit 2eb76110abcc1bbed375b169808ef72c472cb604.

Instead of hardcoding a size, actually calculate the best fit.

This also makes the PostScript output of the circles test case
matching what's shown on the screen.

Revert "gui-top-window.c: no need to check arguments if not used."

This reverts commit 92d4bab703cd0894421bc8c8fe1aa7a67dc1c679.

Revert "ActionDisplay: show help on failure."

This reverts commit 0978828fcf27be98d4edee22e57a13756a684b62.

We need to be strict about argument checking.

drc: Fix polygon-polygon clearance checks

IsPolygonInPolygon was applying 2x bloat to the line it uses to test with
against the second polygon, and IsLineInPolygon was applying a further
bloat offset - making the clearance used incorrect.

This caused false DRC violation reports of polygons being too close to
one-another, despite there being adequate clearance.

Note for future - a cleaner fix regarding "Bloat", and the DRC, is that
we should probably pass it more explicitly as a parameter where it is
to be used, in order to avoid surprises as to where it gets applied.

Added keywords to the appdata file.

Signed-off-by: bert <bert.timmerman@xs4all.nl>

Add measurement units to route-styles string in preferences file.

This avoids unit conversion to cmil on save of preferences, and
avoids the rounding errors associated with that.

Fix Auto-enforce-DRC check to allow the exact minimum spacing

Removes an improper "+ 1" (COORD - e.g. nm) condition applied in the auto-drc
code, and similar from the XOR rendering code.

Reviewed-by: Peter Clifton <Peter.Clifton@clifton-electronics.co.uk>
Closes-bug: lp-1013358

src/hid.h: Converted plain comments into doxygen comments.

Signed-off-by: bert <bert.timmerman@xs4all.nl>

src/heap.[ch]: Converted plain comments into doxygen comments.

Signed-off-by: bert <bert.timmerman@xs4all.nl>

src/global.h: correction of a typo in the Doxygen header.

Signed-off-by: bert <bert.timmerman@xs4all.nl>

Added QFN24_5 and TQFN24_5.

Signed-off-by: bert <bert.timmerman@xs4all.nl>

Added LQFP80-10.

Signed-off-by: bert <bert.timmerman@xs4all.nl>

Added escaping of Attribute value

Signed-off-by: bert <bert.timmerman@xs4all.nl>

Corrections in documentation.

Reported-by: dima
Signed-off-by: bert <bert.timmerman@xs4all.nl>

Add some comments and improve default refdes silk screen

Add generator for Aries Electronics, Inc. footprints

Added footprint generator and definitions for the
High-Temp Universal ZIF DIP Burn-in and Test sockets
from Aries Electronics.
See for example www.arieselec.com for information on their
ZIF sockets.

Fix invalid jump to next VNODE in Collect1() function

When constructing polygon Collect1() function jumps to cur->next vertex
when calling Gather() which results in inclusion of invalid vertex into
resulting polygon.

Fixed by moving the test and the cur->next expression into parameters of
first call to Collect1() only where it is valid (in contrast to second
call after jump() test).

Signed-off-by: bert <bert.timmerman@xs4all.nl>

unit_to_coord(): explicitely cast the result.

Previously 32-bit machines would return 999999999 as the result
of "1.0 m", making the unit test fail.

This introduces the DOUBLE_TO_COORD() macro which should be used
everywhere where doubles are used for calculating coordinates.

Wisdom about rounding found at:

  https://www.cs.tut.fi/~jkorpela/round.html

Added RenumberBlock() and RenumberBuffer().

Converted from plug-in sources from DJ Delorie.
Source file adapted for nm resolution and added user documentation
and a file header for Doxygen.
Added to the build system, so it gets always compiled in.

Found on http://www.delorie.com/pcb/renumberblock.c

Reported-by: DJ Delorie <dj@delorie.com>
Signed-off-by: bert <bert.timmerman@xs4all.nl>

Merge branch 'master' of ssh://git.geda-project.org:65/pcb

let git ignore *.o and *.a files in the gts folder

+ translation file de.po

Add German to the list of translation goals

Update of the translation files

Add translation infrastructure to the empty layout error message

png HID should exit with warning when layout contains no data

Instead of segfault exit decently with an error message on stdout.

Original patch by Amit Dhingra

fixes bug #1048256
https://bugs.launchpad.net/geda-project/+bug/1048256

sync my local master with upstream master

Use hypot() instead of sqrt(pow, pow)

Use some computed vars to make them used

Use wide types for comfortable Coord output

Added missing #include

Changed name of teardrop example pcb file as to avoid make interference.

Signed-off-by: bert <bert.timmerman@xs4all.nl>

Added an example figure and more edits to the user doumentation.

Signed-off-by: bert <bert.timmerman@xs4all.nl>

Added user documentation for teardrops.

Signed-off-by: bert <bert.timmerman@xs4all.nl>

Renamed to teardrops.c.

Signed-off-by: bert <bert.timmerman@xs4all.nl>

Added teardrop.c to POTFILES.in.

Signed-off-by: bert <bert.timmerman@xs4all.nl>

Add teardrops.c from DJ Delorie.

Source file unchanged, just prettyfied file header for Doxygen.
Added to the build system, so it gets always compiled in.

Found on http://www.delorie.com/pcb/teardrops/

Make compiler warning go away.

hid/common/hidgl.c:455: warning: format ‘%lu’ expects type
‘long unsigned int’, but argument 2 has type ‘unsigned int’

Signed-off-by: bert <bert.timmerman@xs4all.nl>

Remove the unwanted N_("..."); macro cruft from the user documentation.

This commit solves LP bug #1505262.

Signed-off-by: bert <bert.timmerman@xs4all.nl>

Added user documentatation for SmartDisperse.

Signed-off-by: bert <bert.timmerman@xs4all.nl>

src/global.h: Converted plain comments into doxygen comments.

Signed-off-by: bert <bert.timmerman@xs4all.nl>

src/gettext.c: Converted plain comments into doxygen comments.

Signed-off-by: bert <bert.timmerman@xs4all.nl>

src/free_atexit.[ch]: Converted plain comments into doxygen comments.

Signed-off-by: bert <bert.timmerman@xs4all.nl>

gtkhid-main.c: swap sprintf() for snprintf().

It's good practice to use the 'n' variant even if it's reasonably
sure the string will never overflow.

This should satisfy bug 1069317.

Gtk: right-aligned static strings in layer groups dialogue.

The static strings are a little more easy to the eye when right
aligned. A colon was added for extra legibility.

Gtk: use radio buttons in layer groups dialogue.

Gtk: more explicit comment in the layer group code.

Gtk: change the order of top and bottom in preference dialogue.

gtk/gui-config.c: swap the order of silk layer groups in GUI.

The silk layers double as layer groups for top and bottom
copper. Current pcb/gtk GUI shows them bottom above top in
File -> Preferences -> Layers -> Groups.  This is inconsistent
with the general order of layers.

The patch swaps the order the silk layers are displayed in the
Groups dialogue.

hid/png/png.c: more checks wether a layer actually exists.

Especially when considering the move to layer types there is no
guarantee a mask and/or silk layer actually exists.

hid/png/png.c: check wether a bottom layer actually exists.

This should fix LP796389: fixes segfault with --photo-mode
                          --photo-flip if the layer "solder" is
                          not in the "Solder side".

global.h: eliminate GCC_VERSION macro.

It was used in this place, only, so no need to clutter the
namespace.

global.h: remove forgotten comment.

RCS IDs and the macro described by the comment were removed with
commit 02a720280796e439015f in December 2011.

hid/ps/ps.c: deal with zero sized circles even better.

Instead of hardcoding a size, actually calculate the best fit.

This also makes the PostScript output of the circles test case
matching what's shown on the screen.

Tests:more challenging circles.pcb.

This time several circles of varying size, along with ruler lines
for measuring them on non-screen output, e.g. a PostScript file.

Note that the zero sized circle with 50 mil line width is drawn
differently in PostScript than on screen.

Introduce regression tests for PostScript exporter.

Not the most fancy testing, but at least it recognizes invalid
PostScript. It triggers with the bug reported in LP1497963.

hid/ps/ps.c: avoid division by zero.

Given a linewidth of zero, 'inf' was written as line width to the
PostScript file, which made the file invalid. Now we at least get
valid PostScript.

Not yet solved: why is there an attempt to draw a circle of zero
size?

This is related to bug LP1497963.

swap the order of silk layer groups in GUI
The silk layers double as layer groups for top and bottom copper. Current
pcb/gtk GUI shows them bottom above top in File->Preferences->Layers->Groups.
This is inconsistent with the general order of layers.
The patch swaps the order the silk layers are displayed in the groups dialog.

Follow autoconf's recommendation for including glu.h.

Follow autoconf's recommendation for including gl.h.

Debian: 'gerbv' isn't a package build dependency.

It's needed for running regression tests, only, and these tests
aren't run when building packages.

Tested with pbuilder.

README.ppa: also set the actual uploader in debian/control.

... or at least provide instructions on how to do so.

debian/control: remove obsolete field DM-Upload-Allowed.

This also fixes the corresponding warning while building the
package.

Debian: add dependencies for building straight from Git.

When building from the bare repository there are quite a few
additional dependencies, almost all of them for documentation and
tutorials.

README.ppa: add instructions on how to build in a chroot.

After many failures with many different tools, 'pbuilder' was
finally found to be the one to work as described.

Debian: remove Quilt entirely.

All patches were removed earlier already, because there should
never be a need for packaging patches when packaging straight from
the Git repository. Accordingly there's no need to have a patch
system at all.

Debian: add instructions on how to build for the PPA.

Debian: remove all patches.

When building straight from the Git repo there should never be a
need for packaging patches.

Debian: additional rules to build straight from Git.

Previous rules were set to build from a distribution tarball, so
'configure' was expected to exist already.

These fixes are just enough to get the packages building, no
or sanity checks happened.

Debian: adjust build dependencies to recent development.

 - Relax dependency on 'tk8.5' to just 'tk'.

 - Add 'libgts-dev' and 'texlive-binaries'.

So far only changes to get it building. All these dependencies
likely need a review, like trying to build without them in a
chroot, one by one, and keeping only the neccessary ones.

Debian: import debian/ as-is from Eugene Mikhantievs PPA.

Fetched from
https://launchpad.net/~mehanik/+archive/ubuntu/geda-unstable

Move gdk_window_process_all_updates to safest spot

Because it might be a tiny bit inefficient for zoom where it was.  This
way nothing hits this except the case where we're about to do a pointer
warp

Do not warp pointer too easily...

* Pointer warping bug people so now you have to double-click to get it

* Tooltip for the drc window

Fix CenterWindow, DRC activation pointer warping.

CenterWindow didn't do what it's name said, it just warped the pointer.
Now it does both for the gtk hid, so DRC checking works consistently
when the violation is off-screen or not.  It's still broken in lesstif
HID but no worse than it was, and there's a warning about the
pan-and-warp action not being implemented there.

gtkhid-main.c: remove seemingly useless double conversion.

They don't do anything.  The locals event_x/y are just copied
out then straight back into gport->pcb_x/y, none of which are
use at all in the interim.

hid/gtk/gui.h: fix definition of LAYER_BUTTON_SILK

LAYER_BUTTON_SILK should give the number of the layer after all copper
layers. Since the copper layers are enumerated starting with zero, this
is equal to the number of copper layers MAX_LAYER.

The previous definition erreously added LAYER_BUTTON_SILK to this number.
This only worked, because src/const.h sets this constant to zero.

Fix the recent addition of smartdisperse.c.

Before committing something, please run a 'make distcheck'. This
is considered to be the most safe way we have to make sure there
is no regression.

If this isn't possible, please build with and without NLS and run
a 'make check' each time. Like:

  ./autogen.sh && ./configure --disable-nls --disable-doc && make
  make check
  ./autogen.sh && ./configure && make
  make check

This is a bit more manual work but gives similar confidence.

All these tests pass now again.

Ignore completely clipped polygons in PS export

This avoids dereferencing a null pointer and crashing.

Signed-off-by: Keith Packard <keithp@keithp.com>
Signed-off-by: bert <bert.timmerman@xs4all.nl>

Added smartdisperse.c from Ben Jackson.

Removed bitrot, and prettyfied the file header for Doxygen.
Added to the build system, so it gets always compiled in.

Signed-off-by: bert <bert.timmerman@xs4all.nl>

configure.ac: report tests enablement in the summary.

Also moved CPP_FLAGS & Co. upwards to have the more important
part last.

Consider build tests not run to be a failure.

Previously a missing test environment would go unnoticed, the
distinction between a successful test run and a test not run at
all was (almost) not noticeable.

src/fontmode.c: Converted plain comments into doxygen comments.

Signed-off-by: bert <bert.timmerman@xs4all.nl>

src/flags.c: Converted plain comments into doxygen comments.

Signed-off-by: bert <bert.timmerman@xs4all.nl>

src/find.[ch]: Converted plain comments into doxygen comments.

Signed-off-by: bert <bert.timmerman@xs4all.nl>

src/file.[ch]: Converted plain comments into doxygen comments.

Signed-off-by: bert <bert.timmerman@xs4all.nl>

run_tests.sh: make sure the test candidate is writable.

This makes a distinction when running 'make distcheck', because
the file is written read-only to the distribution.

Remove these almost 25 Makefiles for just copying tests.

Before there was a Makefile/.am/.in in each test directory,
just to define the EXTRA_DIST variable for this directory. A
rather complex approach.

The new strategy is to list all these files in one place, in
tests/Makefile.am. This almost doubles the speed of compilation
after small changes and reduces clutter sent to the command
line a lot, so a great improvement for developers.

Creation of new tests is also simplified, creating a Makefile.am
and editing configure.ac is no longer neccessary.

Previous versions of this commit used wildcards to include new
tests automatically. This strategy was discarded in favour of a
hand-edited list, because some developers aren't exactly excited
by using wildcards.

As we're on the topic, here are a few informative readings
regarding recursive makes and regarding wildcards:

  "Recursive Make Considered Harmful"
  Peter Miller, 2008
  http://aegis.sourceforge.net/auug97.pdf

  GNU automake about recursion:
  http://www.gnu.org/software/automake/manual/html_node/Directories.html

  GNU make about wildcards:
  http://www.gnu.org/software/make/manual/html_node/Wildcard-Function.html

  GNU automake about wildcards:
  http://www.gnu.org/software/automake/manual/html_node/Wildcards.html

w32/Makefile.am: refine the move to wildcards.

As per discussion in #geda IRC, liberal use of wildcards isn't
exactly a good idea. Some even prefer to use no wildcards at all.
Others want to make situations, where files are often added or
removed, less forget-prone.

As a compromise, move towards more strict, unambiguous wildcards.

Makefile.am: refine the move from INCLUDES to AM_CPPFLAGS.

The key is apparently this sentence in the autoconf docs:

 "AM_CPPFLAGS is ignored in preference to a per-executable
  (or per-library) _CPPFLAGS variable if it is defined."

Accordingly we have to add -I$(top_srcdir) to every
target-specific *_CPPFLAGS.

This fixes a 'make distcheck' more, it builds now, but fails on
the checks.

w32/Makefile.am: use wildcards to include minipack stuff.

This should end the need for the often required commits to let the
Makefile catch up with actual development.

This partially fixes 'make distcheck', it runs a lot further.

src/error.[ch]: Converted plain comments into doxygen comments.

Signed-off-by: bert <bert.timmerman@xs4all.nl>

src/drill.[ch]: Converted plain comments into doxygen comments.

Signed-off-by: bert <bert.timmerman@xs4all.nl>

src/draw.[ch]: Converted plain comments into doxygen comments.

Signed-off-by: bert <bert.timmerman@xs4all.nl>

configure.ac: remove test for maintainer mode.

The test is now pointless since usage of maintainer mode was
removed.

A README shouldn't be run as shell script.

Not that the file had the executable bit set, but having
a #!/bin/sh on top fools some syntax coloring editors.

Fix a configuration warning.

This was the warning message:

src/Makefile.am:224: warning: 'INCLUDES' is the old name for
  'AM_CPPFLAGS' (or '*_CPPFLAGS')