Signed-off-by: Hugh McMaster <hugh.mcmaster@outlook.com>

This bug was exposed by 3b1faf59Signed-off-by: Hugh McMaster <hugh.mcmaster@outlook.com>

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

We used to hide current WineDbg instance when displaying processes' list
(command 'info proc'). This can potentially generate some "dangling"
processes in the hierarchy (related to this WineDbg instance):
- conhost.exe
- start.exe (when launched from unix shell without full path
  to winedbg.exe)

Also, print a more comprehensive error message when trying to attach to
itself (now that debugger's PID is more easily available).
Signed-off-by: Eric Pouech <eric.pouech@gmail.com>
Signed-off-by: Alexandre Julliard <julliard@winehq.org>

Signed-off-by: Aurimas Fišeras <aurimas@members.fsf.org>
Signed-off-by: Alexandre Julliard <julliard@winehq.org>

Implementation copied from msvcrt.
Signed-off-by: Alexandre Julliard <julliard@winehq.org>

Signed-off-by: Alexandre Julliard <julliard@winehq.org>

Signed-off-by: Alexandre Julliard <julliard@winehq.org>

Implementation copied from msvcrt.
Signed-off-by: Alexandre Julliard <julliard@winehq.org>

Implementation copied from msvcrt.
Signed-off-by: Alexandre Julliard <julliard@winehq.org>

Implementation copied from msvcrt.
Signed-off-by: Alexandre Julliard <julliard@winehq.org>

Implementation copied from msvcrt.
Signed-off-by: Alexandre Julliard <julliard@winehq.org>

Implementation copied from msvcrt.
Signed-off-by: Alexandre Julliard <julliard@winehq.org>

Signed-off-by: Alexandre Julliard <julliard@winehq.org>

Signed-off-by: Rémi Bernon <rbernon@codeweavers.com>

Signed-off-by: Rémi Bernon <rbernon@codeweavers.com>

Signed-off-by: Rémi Bernon <rbernon@codeweavers.com>

Signed-off-by: Rémi Bernon <rbernon@codeweavers.com>

Signed-off-by: Rémi Bernon <rbernon@codeweavers.com>

Signed-off-by: Alexandre Julliard <julliard@winehq.org>

Signed-off-by: Alexandre Julliard <julliard@winehq.org>

Application dialog procedures are called in UXTHEME_DefDlgProc(), then may be called again in
USER_DefDlgProcA/W(). Lotus Approach doesn't expect this and run into infinite recursion.

Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=52586Signed-off-by: Zhiyi Zhang <zzhang@codeweavers.com>

Signed-off-by: Zhiyi Zhang <zzhang@codeweavers.com>

Increment pixel pointer for every *pixel*, not every *stride*.
Signed-off-by: Jinoh Kang <jinoh.kang.kr@gmail.com>

The PDB hash stream from TPI header contains information to force a remap
to a given type record (whatever the order in the bucket hash list).
This is generated by the incremental linker in some occasions.

Use that information to remap the corresponding types.
Signed-off-by: Eric Pouech <eric.pouech@gmail.com>

The PDB types can contain several times a type definition with an identical
name. It seems to appear when modifying a type in source (like adding new
fields to a struct):
- as the PDB file (generated from first compilation) is updated (and not
  fully rewritten), the debug information for the old type is not flushed;
  a new record (for the same struct name) is emitted, and inserted
  before the old one in the hash table (bucket list).

Even if dbghelp's hash table is different from PDB's internal one (ie
number of buckets & bucket lists are different), we must maintain the order
of records of identical names (they end up in the same bucket) as a lookup
by name *must* return the first record in PDB's order.
Lookup by name is used:
- when resolving a forward definition (to get the full UDT definition
  including for example a struct/class fields's list)
- when searching by type name from dbghelp APIs, like SymGetTypeFromName()

This patch implements the preservation of that order.
Signed-off-by: Eric Pouech <eric.pouech@gmail.com>

- create (contentless) UDT & enum in first pass
- fill UDT&enum content and load the rest of types in second pass.
Signed-off-by: Eric Pouech <eric.pouech@gmail.com>

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

Code now follows these guidelines:
- define PDB & Codeview internals in cvconst.h and mscvinfo.h (instead
  of having definitions in .c files, some of them being duplicate of .h
  content, and their "duplicate" values eventually diverged over time)
- index of first type comes from PDB type header (instead of always being
  hardcoded as FIRST_DEFINABLE_TYPE)
- use index of last typex from type header (instead of guessing the right
  value while parsing types, which also allows a single allocation
  instead of enlarging buffer while parsing).
Signed-off-by: Eric Pouech <eric.pouech@gmail.com>

Rename a couple of fields in PDB structures for clarity.
Signed-off-by: Eric Pouech <eric.pouech@gmail.com>

Signed-off-by: Arkadiusz Hiler <ahiler@codeweavers.com>

Signed-off-by: Arkadiusz Hiler <ahiler@codeweavers.com>

Signed-off-by: Julian Rüger <jr98@gmx.net>
Signed-off-by: Alexandre Julliard <julliard@winehq.org>

This is currently broken on WoW64.

Signed-off-by: Stefan Dösinger <stefan@codeweavers.com>