From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <chris@the-brannons.com>
Received: from localhost (unknown [IPv6:2602:4b:a4d8:4b00:12bf:48ff:fe7c:5584])
 by hurricane.the-brannons.com (Postfix) with ESMTPSA id 4101577AA5
 for <Edbrowse-dev@lists.the-brannons.com>;
 Thu, 30 Jan 2014 14:19:11 -0800 (PST)
From: Chris Brannon <chris@the-brannons.com>
To: Edbrowse-dev@lists.the-brannons.com
References: <20140030165853.eklhad@comcast.net>
Date: Thu, 30 Jan 2014 14:18:46 -0800
In-Reply-To: <20140030165853.eklhad@comcast.net> (Karl Dahlke's message of
 "Thu, 30 Jan 2014 16:58:53 -0500")
Message-ID: <8761p1kta1.fsf@mushroom.PK5001Z>
User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/24.3 (gnu/linux)
MIME-Version: 1.0
Content-Type: text/plain
Subject: Re: [Edbrowse-dev] [PATCH] Use the list class from the C++ STL,...
X-BeenThere: edbrowse-dev@lists.the-brannons.com
X-Mailman-Version: 2.1.17
Precedence: list
List-Id: Edbrowse Development List <edbrowse-dev.lists.the-brannons.com>
List-Unsubscribe: <http://lists.the-brannons.com/mailman/options/edbrowse-dev>, 
 <mailto:edbrowse-dev-request@lists.the-brannons.com?subject=unsubscribe>
List-Archive: <http://lists.the-brannons.com/mailman/private/edbrowse-dev/>
List-Post: <mailto:edbrowse-dev@lists.the-brannons.com>
List-Help: <mailto:edbrowse-dev-request@lists.the-brannons.com?subject=help>
List-Subscribe: <http://lists.the-brannons.com/mailman/listinfo/edbrowse-dev>, 
 <mailto:edbrowse-dev-request@lists.the-brannons.com?subject=subscribe>
X-List-Received-Date: Thu, 30 Jan 2014 22:19:11 -0000

Karl Dahlke <eklhad@comcast.net> writes:

> Like name is assigned the string produced by htmlAttrVal() in format.c,
> returning a c string of course,

The only call sites for htmlAttrVal are in html.cpp.  It would be best
to move it there and make it static.  But point well taken.  There are
probably other cross-module issues waiting in the wings.

-- Chris