Misleading indentation in RelayState JSON output code
Basics
Technical
Logistics
Basics
Technical
Logistics
Description
The JSON output isn't really conditional in the current code, contrary to what the formatting suggests. If it should be conditional, I propose the following patch:
Environment
None
Activity
Show:
Rod Widdowson February 14, 2017 at 11:37 AM
applied 5dcb0ee
Scott Cantor February 1, 2017 at 5:59 PM
Have they moved to Firefox versioning? Every patch is a new major version? The latest Red Hat is still GCC 4 for heaven's sake.
Ferenc Wágner February 1, 2017 at 5:51 PM
Edited
Careful, just yesterday I got reports of OpenSAML and the SP not building with GCC 7 (that's a seven). (Don't worry, they don't look genuine, just trying to link with an XMLTooling built with GCC 6.)
Scott Cantor January 27, 2017 at 7:10 PM
Heh, retirement may be right about the time I actually run into GCC 6.
Ferenc Wágner January 27, 2017 at 7:04 PM
Not necessarily: GCC 6 introduced several new warnings for this kind of mistakes.
Fixed
Pinned fields
Click on the next to a field label to start pinning.
The JSON output isn't really conditional in the current code, contrary to what the formatting suggests. If it should be conditional, I propose the following patch: