[comp.sys.mac.programmer] Lightspeed C & Pedit

jcl@hpausla.HP.COM (Jeff Laing) (01/30/89)

Well, I finally got the time to sit down and use this neato Pedit package from
Think.  Has anyone else out there done the same and therefore headed off any
more of its problems so that I don't have to stumble over them.

First, the DA makes a big deal (12 lines of comment) about the tricks it uses
to ensure the Edit menu will work.  Then they botch the implementation so that
it always asks for a file TWICE.  [ the variable "first" needs to be set
before the call to DoOpenFile, not after; the problem is that SFGetFile()
will cause a DEACTIVATE for the current window which will call DoOpenFile
again. ]

Second, and worse, the function "Find Clipboard" crashes me with access
errors. [ the problem is the routine PEGetScrpLength().  it is used but 
IS NOT DEFINED in PE.h.  The manual talks about a routine PEGetScrapLen()
but it doesn't exist in the library.  Also, PEGetScrpLength() would appear
to have some troubles of its own.  To make it work, I had to declare it as
/*NONPASCAL*/ long PEGetScrpLength(long dummy);
and pass in a long dummy (which it would overwrite!!)  Note this hack does
not actually solve the problem, but fortunately the result accidentally
comes back in the right register ...

Anyone else out there had any problems, and/or know of any fixes?

Disclaimer: Although the above sounds terribly negative, I think the product
is great.  

oster@dewey.soe.berkeley.edu (David Phillip Oster) (01/31/89)

The PEdit source code is full of bugs. 

(o) The scrolling compares against 0 when it should compare against GetCtlMin()

(o) The code is very sloppy about failing to export/import the global text
scrap, so for example, you can DLGPaste(), or even get the global global
scrap on startup.

View it as an incentive to actually get in and modify the source. That's
what I've done.

--- David Phillip Oster            --"When we replace the mouse with a pen,
Arpa: oster@dewey.soe.berkeley.edu --3 button mouse fans will need saxophone
Uucp: {uwvax,decvax}!ucbvax!oster%dewey.soe.berkeley.edu --lessons." - Gasee