[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: lynx-dev Patch for pre.9
From: |
Doug Kaufman |
Subject: |
Re: lynx-dev Patch for pre.9 |
Date: |
Sun, 11 Oct 1998 14:52:57 -0700 (PDT) |
On Sun, 11 Oct 1998 address@hidden wrote:
> > While looking at the temp directory code, I found that DOS viewers
> > didn't work if the temp directory was given unix style. The mixed
> > forward and backward slashes confused the system. The patch makes
>
> The www names aren't synonymouse with unix (there's additional parsing).
> You may want to revisit that...
I think that this only applies to the temp files created, which were
being made with mixed style (e.g. "f:/whatever/tmp\lytemp.xxx"). Now the
path is uniform in DOS style (e.g. "f:\whatever\tmp\lytemp.xxx"). Am I
missing something here?
> > LYMain.c: In function `main':
> > LYMain.c:688: warning: passing arg 2 of `HTSACopy' makes pointer \
> > from integer without a cast
> > LYMain.c:690: warning: passing arg 2 of `HTSACopy' makes pointer \
> > from integer without a cast
> > LYMain.c:692: warning: passing arg 2 of `HTSACopy' makes pointer \
> > from integer without a cast
>
> that's a missing #include (I'll look/see). doesn't happen when I build
> on Unix, so it's due to one of the #ifdef DOSPATH includes.
This only applied to the HTDOS_name function (which was arg 2 of
HTSACopy). The following revised patch for LYMain.c seems to fix it.
It needed (char *) before HTDOS_name. I also put TMPDIR as one of the
options for DOS, applying only if TEMP and TMP are not defined in the
environment.
Doug
*** lynx2-8-1/src/LYMain.c Sat Oct 10 13:53:16 1998
--- lynx2-8-1/src/LYMain.c.new Sun Oct 11 14:35:52 1998
***************
*** 683,702 ****
StrAllocCat(lynx_version_putenv_command, LYNX_VERSION);
putenv(lynx_version_putenv_command);
#endif /* VMS */
- if ((cp = getenv("LYNX_TEMP_SPACE")) != NULL)
- StrAllocCopy(lynx_temp_space, cp);
- else
#if defined (DOSPATH) || defined (__EMX__)
! if ((cp = getenv("TEMP")) != NULL)
! StrAllocCopy(lynx_temp_space, cp);
else if ((cp = getenv("TMP")) != NULL)
! StrAllocCopy(lynx_temp_space, cp);
else
{
printf("You MUST define a valid TMP or TEMP area!\n");
exit(-1);
}
#else
StrAllocCopy(lynx_temp_space, TEMP_SPACE);
#endif
if ((cp = strchr(lynx_temp_space, '~'))) {
--- 683,708 ----
StrAllocCat(lynx_version_putenv_command, LYNX_VERSION);
putenv(lynx_version_putenv_command);
#endif /* VMS */
#if defined (DOSPATH) || defined (__EMX__)
! if ((cp = getenv("LYNX_TEMP_SPACE")) != NULL)
! StrAllocCopy(lynx_temp_space, (char *)HTDOS_name(cp));
! else if ((cp = getenv("TEMP")) != NULL)
! StrAllocCopy(lynx_temp_space, (char *)HTDOS_name(cp));
else if ((cp = getenv("TMP")) != NULL)
! StrAllocCopy(lynx_temp_space, (char *)HTDOS_name(cp));
! else if ((cp = getenv("TMPDIR")) != NULL)
! StrAllocCopy(lynx_temp_space, (char *)HTDOS_name(cp));
else
{
printf("You MUST define a valid TMP or TEMP area!\n");
exit(-1);
}
#else
+ if ((cp = getenv("LYNX_TEMP_SPACE")) != NULL)
+ StrAllocCopy(lynx_temp_space, cp);
+ else if ((cp = getenv("TMPDIR")) != NULL)
+ StrAllocCopy(lynx_temp_space, cp);
+ else
StrAllocCopy(lynx_temp_space, TEMP_SPACE);
#endif
if ((cp = strchr(lynx_temp_space, '~'))) {
__
Doug Kaufman
Internet: address@hidden (preferred)
address@hidden