Skip to content

[24122] Fix warning on ubuntu noble (backport #133)#136

Open
mergify[bot] wants to merge 1 commit into1.2.xfrom
mergify/bp/1.2.x/pr-133
Open

[24122] Fix warning on ubuntu noble (backport #133)#136
mergify[bot] wants to merge 1 commit into1.2.xfrom
mergify/bp/1.2.x/pr-133

Conversation

@mergify
Copy link

@mergify mergify bot commented Jan 27, 2026

Description

This PR fixes:

../../eprosima/discovery-server/src/DiscoveryServerManager.cpp: In member function ‘void eprosima::discovery_server::DiscoveryServerManager::loadClient(tinyxml2::XMLElement*)’:
../../eprosima/discovery-server/src/DiscoveryServerManager.cpp:1106:28: error: suggest parentheses around assignment used as truth value [-Werror=parentheses]
 1106 |                     if (p4 = std::dynamic_pointer_cast<TCPv4TransportDescriptor>(sp))
      |                         ~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

@Mergifyio backport 2.1.x 2.0.x 1.2.x

Contributor Checklist

  • Commit messages follow the project guidelines.
  • The added tests pass locally.
  • Changes are backwards compatible.
  • New feature has been documented/Current behavior is correctly described in the documentation.
  • Applicable backports have been included in the description.

Reviewer Checklist

  • The PR has a milestone assigned.
  • The title and description correctly express the PR's purpose.
  • Check contributor checklist is correct.
  • Check CI results: changes do not issue any warning.
  • Check CI results: failing tests are unrelated with the changes.

This is an automatic backport of pull request #133 done by [Mergify](https://mergify.com).

Signed-off-by: Ricardo González Moreno <ricardo@richiware.dev>
(cherry picked from commit cb2addc)

# Conflicts:
#	src/DiscoveryServerManager.cpp
@mergify mergify bot added the conflicts label Jan 27, 2026
@mergify
Copy link
Author

mergify bot commented Jan 27, 2026

Cherry-pick of cb2addc has failed:

On branch mergify/bp/1.2.x/pr-133
Your branch is up to date with 'origin/1.2.x'.

You are currently cherry-picking commit cb2addc.
  (fix conflicts and run "git cherry-pick --continue")
  (use "git cherry-pick --skip" to skip this patch)
  (use "git cherry-pick --abort" to cancel the cherry-pick operation)

Unmerged paths:
  (use "git add <file>..." to mark resolution)
	both modified:   src/DiscoveryServerManager.cpp

no changes added to commit (use "git add" and/or "git commit -a")

To fix up this pull request, you can check it out locally. See documentation: https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/reviewing-changes-in-pull-requests/checking-out-pull-requests-locally

@mergify mergify bot mentioned this pull request Jan 27, 2026
10 tasks
@cferreiragonz cferreiragonz added this to the v1.2.3 milestone Jan 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants