|
From: | Jacob Bachmeyer |
Subject: | Re: [PATCH 1/2] Use 'file join' to concatenate directory and filename |
Date: | Thu, 07 Mar 2019 20:22:53 -0600 |
User-agent: | Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.8.1.22) Gecko/20090807 MultiZilla/1.8.3.4e SeaMonkey/1.1.17 Mnenhy/0.7.6.0 |
Andrew Burgess wrote:
In search_and_load_file in once place 'file join' is used to concatenate a directory and a filename, while elsewhere in the function the concatenate is hard coded as a string "$dir/$initfile", assuming that '/' is used to join components. Presumably this is fine for all the targets dejagnu supports, but its probably better to stick to one approach or the other. This patch makes use of 'file join' for building paths.
Thanks; that is one of the eventual minor cleanups that I have been doing as I can fit them in. While I doubt that DejaGnu will ever be ported to a system that does not use the POSIX file name convention, it is possible that Expect may eventually be so ported and, in any case, [file join] is more clear than concatenating strings. I suspect that [file join] did not exist in Tcl when DejaGnu was first written.
-- Jacob
[Prev in Thread] | Current Thread | [Next in Thread] |