Skip to content

Commit a854443

Browse files
authored
Merge pull request #390 from vdidenko/cpp_simple_bool_returns
[C++]: Simplify bool method returns
2 parents 6b567c7 + 94389b1 commit a854443

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

sbe-tool/src/main/java/uk/co/real_logic/sbe/generation/cpp/CppGenerator.java

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -525,7 +525,7 @@ private void generateVarDataDescriptors(
525525
indent + " }\n\n" +
526526
indent + " bool %1$sInActingVersion(void)\n" +
527527
indent + " {\n" +
528-
indent + " return (m_actingVersion >= %1$sSinceVersion()) ? true : false;\n" +
528+
indent + " return m_actingVersion >= %1$sSinceVersion();\n" +
529529
indent + " }\n\n" +
530530
indent + " static SBE_CONSTEXPR const std::uint16_t %1$sId(void)\n" +
531531
indent + " {\n" +
@@ -661,7 +661,7 @@ private CharSequence generateChoices(final String bitsetClassName, final List<To
661661
" bool %1$s(void) const\n" +
662662
" {\n" +
663663
"%2$s" +
664-
" return (%3$s(*((%4$s *)(m_buffer + m_offset))) & (0x1L << %5$s)) ? true : false;\n" +
664+
" return %3$s(*((%4$s *)(m_buffer + m_offset))) & (0x1L << %5$s);\n" +
665665
" }\n\n",
666666
choiceName,
667667
generateChoiceNotPresentCondition(token.version(), BASE_INDENT),
@@ -1490,7 +1490,7 @@ private CharSequence generateFields(final String containingClassName, final List
14901490
indent + " }\n\n" +
14911491
indent + " bool %1$sInActingVersion(void)\n" +
14921492
indent + " {\n" +
1493-
indent + " return (m_actingVersion >= %1$sSinceVersion()) ? true : false;\n" +
1493+
indent + " return m_actingVersion >= %1$sSinceVersion();\n" +
14941494
indent + " }\n\n",
14951495
propertyName,
14961496
(long)signalToken.version()

0 commit comments

Comments
 (0)