On Mon, Apr 08, 2002 at 01:41:33PM -0400, Dan Espen wrote:
> Uwe Pross <[EMAIL PROTECTED]> writes:
> > On Mon, Apr 08, 2002 at 09:47:42AM -0400, Dan Espen wrote:
> > > If someone were to read this, I think they'd be left with the
> > > impression that these edges are on all 4 edges of the display.
> ...
> > I adapted the checkPanFrames() function that if a command is bound  to
> > a  pan frame window it is always on the screen.
> 
> OK, good.
> 
> I think safestrdup does what the new duplicateString does.
> 
> The east/west stuff and the 1 letter abbreviations aren't
> documented.  You could removed the east/west stuff, I don't
> think its needed.

I still have to document the possible values of the direction
options.  It's just code from the ParseDirectionArgument()
function.

> I think there should be some mention of the additional frames
> when the commands are used.
> 
> The FvwmAuto man page refers to an example in the fvwm2 man
> page, but I couldn't find an example.
> 
> The function StrEquals can be used for doing compares without
> worring about case.
> 
> I haven't applied the patch, I just reviewed it.

If you think the patch is good, please apply it.  I've a backlog
of 150 mails, so it could take some time before I get to do it.

Bye

Dominik ^_^  ^_^

-- 
Dominik Vogt, email: [EMAIL PROTECTED]
LifeBits Aktiengesellschaft, Albrechtstr. 9, D-72072 Tuebingen
fon: ++49 (0) 7071/7965-0, fax: ++49 (0) 7071/7965-20
--
Visit the official FVWM web page at <URL:http://www.fvwm.org/>.
To unsubscribe from the list, send "unsubscribe fvwm-workers" in the
body of a message to [EMAIL PROTECTED]
To report problems, send mail to [EMAIL PROTECTED]

Reply via email to