From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 32283 invoked by alias); 29 Oct 2010 20:16:25 -0000 Received: (qmail 32274 invoked by uid 22791); 29 Oct 2010 20:16:23 -0000 X-SWARE-Spam-Status: No, hits=-2.1 required=5.0 tests=AWL,BAYES_00,T_RP_MATCHES_RCVD X-Spam-Check-By: sourceware.org Received: from mail.codesourcery.com (HELO mail.codesourcery.com) (38.113.113.100) by sourceware.org (qpsmtpd/0.43rc1) with ESMTP; Fri, 29 Oct 2010 20:16:19 +0000 Received: (qmail 14460 invoked from network); 29 Oct 2010 20:16:17 -0000 Received: from unknown (HELO caradoc.them.org) (dan@127.0.0.2) by mail.codesourcery.com with ESMTPA; 29 Oct 2010 20:16:17 -0000 Date: Fri, 29 Oct 2010 20:16:00 -0000 From: Daniel Jacobowitz To: Doug Evans Cc: Nathan Froyd , gdb-patches@sourceware.org Subject: Re: [PATCH] fix py-inferior.exp for remote cross-endian testing Message-ID: <20101029201604.GA813@caradoc.them.org> Mail-Followup-To: Doug Evans , Nathan Froyd , gdb-patches@sourceware.org References: <1288380039-21997-1-git-send-email-froydnj@codesourcery.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.5.20 (2009-06-14) Mailing-List: contact gdb-patches-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sourceware.org X-SW-Source: 2010-10/txt/msg00394.txt.bz2 On Fri, Oct 29, 2010 at 07:49:44PM +0000, Doug Evans wrote: > Hi. This patch is ok with me. > > [btw, one could get the value of endianness from python with > gdb.parameter("endian"). > Or at least should. On my machine it returns "". > We probably want to fix that. Your patch is fine with me as is though.] (gdb) py print gdb.parameter("endian") (gdb) set endian big The target is assumed to be big endian (gdb) py print gdb.parameter("endian") big (gdb) set endian little The target is assumed to be little endian (gdb) py print gdb.parameter("endian") little (gdb) set endian auto The target endianness is set automatically (currently little endian) (gdb) py print gdb.parameter("endian") auto (gdb) show endian The target endianness is set automatically (currently little endian) So that's at least one obvious bug ("" vs "auto"), and one more complex bug (what should it return for auto?). I'm inclined to say that it should return auto, and there should be some other way to find out "auto what?"; OTOH, it's a general problem with parameters... -- Daniel Jacobowitz CodeSourcery