🚀 go-pugleaf

I took the types I used from his messages lines that are reproduced below.

> ...  If I try to compile with this:
> (Proc) (szRunTypes, (char *)&rntyptable);
> which is how it ought to be ...

You are correct that it is
    int __stdcall(*
and not
    int (* __stdcall

Good catch!

You are not correct in your objections to the cast to type FARPROC.  It is a
cast on an lvalue, not a declaration.  As you can see below it works ok.

----------------------------------
C:\DOCUME~1\ADMINI~1\MYDOCU~1\Lookat\q006
>type testit.cpp
  #include <windows.h>
  #include <cstdio>

  using std::printf;

  int main()
    {
    HMODULE Lib = LoadLibrary("NYSiteP.dll");
    int __stdcall (*Proc)(char *, char *);

    if (Lib != NULL)
        {
        (FARPROC) Proc = GetProcAddress(Lib, "SITE");

        if (Proc != NULL)
            {
            printf("should be 8: %d\n", Proc("123", "45678"));
            printf("should be 4: %d\n", Proc(NULL, "1234"));
            }

        FreeLibrary(Lib);
        }

    return 0;
    }


C:\DOCUME~1\ADMINI~1\MYDOCU~1\Lookat\q006
>type nysitep.cpp
    #include <windows.h>

    BOOL WINAPI DllEntryPoint(HINSTANCE, DWORD, LPVOID)
      {
      return TRUE;
      }

    extern "C"
    int __declspec(dllexport) __stdcall SITE(char *Test, char *data)
      {
      int result = (Test) ? lstrlen(Test) : 0;

      if (data)
        {
        result += lstrlen(data);
        }

      return result;
      }

C:\DOCUME~1\ADMINI~1\MYDOCU~1\Lookat\q006
>bcc32 -WCR testit
Borland C++ 5.6.4 for Win32 Copyright (c) 1993, 2002 Borland
TESTIT.CPP:
Turbo Incremental Link 5.64 Copyright (c) 1997-2002 Borland

C:\DOCUME~1\ADMINI~1\MYDOCU~1\Lookat\q006
>bcc32 -WDR nysitep
Borland C++ 5.6.4 for Win32 Copyright (c) 1993, 2002 Borland
NYSITEP.CPP:
Turbo Incremental Link 5.64 Copyright (c) 1997-2002 Borland


C:\DOCUME~1\ADMINI~1\MYDOCU~1\Lookat\q006
>testit
should be 8: 8
should be 4: 4

C:\DOCUME~1\ADMINI~1\MYDOCU~1\Lookat\q006
>

----------------------------------

.  Ed

> Remy Lebeau wrote in message
> news:4033dc64$3@newsgroups.borland.com...
>
>
> >   int (*__stdcall Proc)(szRunTypes, char*&);
>
> Shouldn't be be more like this instead?
>
>     int __stdcall (*Proc)(char*, char*);
>
> From Peter's description, szRunTypes is a variable, not a data type.  And
> there is no reference used for the second parameter.
>
> >       Proc = (int(*__stdcall)(szRunTypes,char*&))
> >           GetProcAddress(Lib, "SITE");
>
> You declared the Proc variable to be FARPROC.  Wouldn't that cast fail
then?
> Also, since it is a FARPROC, wouldn't you need to past the pointer again
> when actually calling the exported function?