On Tue, 18 Oct 2016, Manav Bhatia wrote:

Seems like this was added in “65cde6a” on 5/17/14.
Was this intended?

Intended, but not correctly, I believe.  It looks like John was going
through the code looking for switch statements with no default (which
in many parts of the FE code would imply a silent error if no case
statement was hit) and adding a default error message to catch those
errors.

In the code you're tripping, though, the correct default behavior was
already there: the switch statement does a sign flip for
reverse-oriented edge dofs, but should do nothing for non-edge DoFs!

I'll wait for John's input but it looks like we ought to just delete
the default statement and add some test coverage for SZABAB.  You
could do the former yourself to get up and running quickly.
---
Roy
------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most 
engaging tech sites, SlashDot.org! http://sdm.link/slashdot
_______________________________________________
Libmesh-users mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/libmesh-users

Reply via email to