[Bug rust/23650] New: rust field name access error mentions "foo"

[tromey at sourceware dot org]

https://sourceware.org/bugzilla/show_bug.cgi?id=23650

Bug ID: 23650
Summary: rust field name access error mentions "foo"
Product: gdb
Version: HEAD
Status: NEW
Severity: normal
Priority: P2
Component: rust
Assignee: unassigned at sourceware dot org
Reporter: tromey at sourceware dot org
Target Milestone: ---

This is in rust-lang.c:

error (_("Attempting to access named field foo of tuple "

This should mention the field name, not "foo".

[tromey at sourceware dot org]

https://sourceware.org/bugzilla/show_bug.cgi?id=23650

Tom Tromey <tromey at sourceware dot org> changed:

What |Removed |Added
----------------------------------------------------------------------------
Assignee|unassigned at sourceware dot org |tromey at sourceware dot org

[cvs-commit at gcc dot gnu.org]

https://sourceware.org/bugzilla/show_bug.cgi?id=23650

--- Comment #1 from cvs-commit at gcc dot gnu.org <cvs-commit at gcc dot gnu.org> ---
The master branch has been updated by Tom Tromey <***@sourceware.org>:

https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;h=4a3fe98f885a2d1db804584e7ea265ab3ccef4d7

commit 4a3fe98f885a2d1db804584e7ea265ab3ccef4d7
Author: Tom Tromey <***@tromey.com>
Date: Thu Sep 13 10:56:34 2018 -0600

Make Rust error message mention the field name

I noticed a spot in rust-lang.c where the placeholder "foo" was used
instead of the actual field name. This patch fixes the bug.

gdb/ChangeLog
2018-09-13 Tom Tromey <***@tromey.com>

PR rust/23650:
* rust-lang.c (rust_evaluate_subexp): Use field name, not "foo".

gdb/testsuite/ChangeLog
2018-09-13 Tom Tromey <***@tromey.com>

PR rust/23650:
* gdb.rust/simple.exp: Add test for enum field access error.

[cvs-commit at gcc dot gnu.org]

https://sourceware.org/bugzilla/show_bug.cgi?id=23650

--- Comment #2 from cvs-commit at gcc dot gnu.org <cvs-commit at gcc dot gnu.org> ---
The gdb-8.2-branch branch has been updated by Tom Tromey
<***@sourceware.org>:

https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;h=fde24e1a534b7df3edd0fc167d0ace7d240021f8

commit fde24e1a534b7df3edd0fc167d0ace7d240021f8
Author: Tom Tromey <***@tromey.com>
Date: Thu Sep 13 10:56:34 2018 -0600

Make Rust error message mention the field name

I noticed a spot in rust-lang.c where the placeholder "foo" was used
instead of the actual field name. This patch fixes the bug.

2018-09-13 Tom Tromey <***@tromey.com>

PR rust/23650:
* rust-lang.c (rust_evaluate_subexp): Use field name, not "foo".

gdb/testsuite/ChangeLog
2018-09-13 Tom Tromey <***@tromey.com>

PR rust/23650:
* gdb.rust/simple.exp: Add test for enum field access error.

[tromey at sourceware dot org]

https://sourceware.org/bugzilla/show_bug.cgi?id=23650

Tom Tromey <tromey at sourceware dot org> changed:

What |Removed |Added
----------------------------------------------------------------------------
Status|NEW |RESOLVED
Resolution|--- |FIXED
Target Milestone|--- |8.2.1

--- Comment #3 from Tom Tromey <tromey at sourceware dot org> ---
Fixed.