Skip to content

Conversation

MahadMuhammad
Copy link
Contributor

Currently, gccrs using the const rich_location &, and then using the cast to convert it to the functions which was used to emit the errors, having the overloaded is good, similiar to other frontends.


gcc/rust/ChangeLog:

* rust-diagnostics.cc (rust_be_error_at): Added overload function. (rust_error_at): non-const `rich_location *` function.
* rust-diagnostics.h (rust_error_at): Implementation of overloaded function. (rust_be_error_at): likewise.
* rust_error_codes.def: Added GNU license.

Currently, gccrs using the const `rich_location &`, and
then using the cast to convert it to the functions which
was used to emit the errors, having the overloaded is
good, similiar to other frontends.

gcc/rust/ChangeLog:

	* rust-diagnostics.cc (rust_be_error_at): Added overload function.
	(rust_error_at): non-const `rich_location *` function.
	* rust-diagnostics.h (rust_error_at): Implementation of overloaded
	function.
	(rust_be_error_at): likewise.
	* rust_error_codes.def: Added GNU license.

Signed-off-by: Muhammad Mahad <[email protected]>
Copy link
Member

@philberty philberty left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@philberty philberty added this pull request to the merge queue Aug 23, 2023
@philberty philberty added this to the GCC 14 Stage 3 milestone Aug 23, 2023
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to no response for status checks Aug 23, 2023
@P-E-P P-E-P added this pull request to the merge queue Aug 28, 2023
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to no response for status checks Aug 28, 2023
@P-E-P P-E-P added this pull request to the merge queue Aug 29, 2023
Merged via the queue into Rust-GCC:master with commit f2fc29d Aug 29, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants