From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from localhost (unknown [IPv6:2602:4b:af6e:e500::35bb:7ca9]) by hurricane.the-brannons.com (Postfix) with ESMTPSA id B8FB577FF6; Mon, 10 Jul 2017 21:31:52 -0700 (PDT) From: Chris Brannon To: Dominique Martinet Cc: Edbrowse-dev@lists.the-brannons.com References: <20170703051927.GA1994@nautica> <20170709144030.GA24038@nautica> <20170609174513.eklhad@comcast.net> <20170710045636.GA3943@nautica> Date: Mon, 10 Jul 2017 21:32:38 -0700 In-Reply-To: <20170710045636.GA3943@nautica> (Dominique Martinet's message of "Mon, 10 Jul 2017 06:56:36 +0200") Message-ID: <87tw2joaop.fsf@the-brannons.com> User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/25.2 (gnu/linux) MIME-Version: 1.0 Content-Type: text/plain Subject: Re: [Edbrowse-dev] Disabling local echo for password fields X-BeenThere: edbrowse-dev@lists.the-brannons.com X-Mailman-Version: 2.1.24 Precedence: list List-Id: Edbrowse Development List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 11 Jul 2017 04:31:53 -0000 Dominique Martinet writes: > Right, I hadn't thought of radio and other type of fields as there > weren't any on the page I tested. > I agree something will likely break in that case, but I would like to > disable echo for informations entered with that even if the field is not > set -- I think in that case it makes more sense to revert INP_PW as a > minor type, I will implement that as you suggested in your earlier mail. The inp_types list in decorate.c is also duplicated in html.c, and you missed that one in the patch to add the new minor type. Couldn't we just get rid of the inp_types from html.c and make inp_types from decorate.c global? Maybe some of these constant strings belong in their own file, like html-constants.c or whatever, but that's a discussion for another day. I'd also say that the function infShow needs to be reworked a little, so that it shows the minor type, and not just text. I'd like Karl's opinion on that. If he agrees, I wouldn't mind doing that work after your patches are submitted. Aside from all of that, your patchset looks pretty good. I still want Adam's opinion on the auth negotiation. I don't know if his issue still exists, or whether it was subsequently fixed by libcurl / other software. Regards, -- Chris