Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Commit 3720495

Browse filesBrowse files
authored
Merge pull request #859 from fjatWbyT/fix-issue-718
Fix #718
2 parents a2ffd4c + ef3104a commit 3720495
Copy full SHA for 3720495

File tree

3 files changed

+13
-1
lines changed
Filter options

3 files changed

+13
-1
lines changed
+2Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,2 @@
1+
- `A2-7-3` - `UndocumentedUserDefinedType.ql`
2+
- Fixes #718. Include trailing characters after group comment endings with ///@{ ... ///@}.

‎cpp/autosar/src/rules/A2-7-3/UndocumentedUserDefinedType.ql

Copy file name to clipboardExpand all lines: cpp/autosar/src/rules/A2-7-3/UndocumentedUserDefinedType.ql
+1-1Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,7 @@ private predicate isInFunctionScope(Declaration d) {
2828
private string doxygenCommentGroupStrings(boolean opening) {
2929
opening = true and result = ["///@{", "/**@{*/"]
3030
or
31-
opening = false and result = ["///@}", "/**@}*/"]
31+
opening = false and result = ["///@}%", "/**@}*/"]
3232
}
3333

3434
pragma[inline]

‎cpp/autosar/test/rules/A2-7-3/test.cpp

Copy file name to clipboardExpand all lines: cpp/autosar/test/rules/A2-7-3/test.cpp
+10Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -225,4 +225,14 @@ class ClassG2 { // COMPLIANT
225225
class ClassG3 { // COMPLIANT
226226
public:
227227
friend int foo3() { return 1; } // NON_COMPLIANT
228+
};
229+
230+
/// @brief A Doxygen comment.
231+
class ClassH { // COMPLIANT
232+
public:
233+
/// @brief Group with comment at the end.
234+
///@{
235+
void m(); // COMPLIANT
236+
void n(); // COMPLIANT
237+
///@} End of group
228238
};

0 commit comments

Comments
0 (0)
Morty Proxy This is a proxified and sanitized view of the page, visit original site.