Signed-off-by: Fabian Maurer <dark.shadow4@web.de>

If we get an error, we might get a double free otherwise.
Signed-off-by: Fabian Maurer <dark.shadow4@web.de>

Signed-off-by: Fabian Maurer <dark.shadow4@web.de>

Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=53677

Fix an uninitialized variable warning.

Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=53861

Instead of silently ignoring crashes.

Signed-off-by: Eric Pouech <eric.pouech@gmail.com>

Signed-off-by: Eric Pouech <eric.pouech@gmail.com>

And call it recursively with force = TRUE instead of calling graphics
driver pUpdateDisplayDevices separately.

The previously used syntax, -Wl,-pdb,<file>, was the originally
supported one in lld (since 2018). Later (in 2019) the second
syntax was added, allowing both -Wl,-pdb=<file> and -Wl,-pdb,<file>.
(This other parameter syntax allows easier distinguishing an empty
argument, for letting the linker implicitly pick the file name.)

Move over to using the more modern syntax - reducing the usage of
the old form of the option. This potentially allows deprecating
the original syntax and maybe allows using --pdb as a boolean
flag for implying an automatically named PDB file (which currently
requires using the awkward syntax "--pdb=").

Also prefer the long two dashes form, i.e. --pdb instead of -pdb;
the single dash form is only allowed by getopt when there
are no conflicts with single-letter options, while the form with
two dashes is unambiguous.

Basically:
- extending symt_function to enable storage of multiple address ranges
- symt_function and sym_inlinesite now share the same fields, so
  get rid to the later.

Note that only the first range of a top level function is actually
stored and used (even if the structure allows for more).
Signed-off-by: Eric Pouech <eric.pouech@gmail.com>

Signed-off-by: Eric Pouech <eric.pouech@gmail.com>