Skip to content

Commit f967141

Browse files
committed
Remove useless if branch in sign map creation
1 parent a61d24b commit f967141

2 files changed

Lines changed: 7 additions & 3 deletions

File tree

NEWS.md

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,12 @@ All notable changes to this project will be documented in this file.
55
The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
66
and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
77

8+
## [Unreleased]
9+
10+
### Fixed
11+
12+
- Fixed `is_slave == false` (comparison instead of assignment) in `DivConformingFESpaces`, and simplified the boundary-facet branch by removing the now-redundant `if/else`.
13+
814
## [0.4.11] - 2026-02-20
915

1016
### Added

src/DivConformingFESpaces.jl

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -126,9 +126,7 @@ function _generate_sign_flips(model,cell_reffes)
126126
cells_around_facets,
127127
facet_gid)
128128
is_slave=false
129-
if (length(facet_cells_around)==1)
130-
is_slave == false
131-
else
129+
if length(facet_cells_around) > 1
132130
mx=maximum(loc_to_glo[facet_cells_around])
133131
is_slave = (loc_to_glo[cell] == mx)
134132
end

0 commit comments

Comments
 (0)