Closed Bug 73436 Opened 24 years ago Closed 23 years ago

npspy/common should not use win32api calls

Categories

(Core Graveyard :: Plug-ins, defect)

x86
Windows 2000
defect
Not set
normal

Tracking

(Not tracked)

VERIFIED FIXED
Future

People

(Reporter: timeless, Assigned: serhunt)

References

()

Details

Attachments

(1 file)

at least, it would seem to me that the common directory should be for common functions and not platform specific functions. /modules/plugin/npspy/common/npp_gate.cpp, line 79 -- NP_GETENTRYPOINTS real_NP_GetEntryPoints = (NP_GETENTRYPOINTS)GetProcAddress(hLib, "NP_GetEntryPoints");
This is a good catch. I overlooked it. Thanks. Will take care.
Depends on: 76602
Target Milestone: --- → Future
Attached patch first try (deleted) — Splinter Review
r=peterl
Checked in.
Status: NEW → RESOLVED
Closed: 23 years ago
Resolution: --- → FIXED
You're still including windowsx.h (which i suspect is windows related) and there's one other windowsism in logger.cpp (neither are wrapped in XP_WIN). -- I can file new bugs if you want.
Please, do.
verified fixed + filed Bugzilla Bug 96245 Including <windowsx.h> even for non windows platforms
Status: RESOLVED → VERIFIED
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: