Hacktoberfest 2026: những issue maintainer đã đánh dấu cho tháng Mười, đang mở và phù hợp người mới. Xem issue Hacktoberfest

cpplint checks fail for services and actions with buffers in the response/result

Đang mở
  • Có gì không ổn?
#264 2 bình luận 0 reaction 0 người được giao Xem trên GitHub

Chưa có ai nhận issue này.

Đánh giá

Độ khó
3/5
Thời gian dự kiến
1-2 ngày
Mức phù hợp với người mới
56/100
Loại issue
Lỗi
Độ rõ ràng
Khá rõ ràng
Mức độ hoạt động
Sôi nổi
Công nghệ
c
Lĩnh vực
build-system, testing-qa

Hướng nghiên cứu

Bắt đầu với tệp rosidl_generator_py được tạo build/.../rosidl_generator_py/srv/_get_buffer_s.c và lỗi build/include_order của cpplint_rosidl_generated_py. So sánh việc tạo service/action được mô tả trong #263 với việc chèn stdint.h từ #250, bao gồm cách xử lý response/result/feedback so với request/goal. Được xem là hoàn tất khi các kiểm tra cpplint đối với mã được tạo bị ảnh hưởng đều đạt mà không đưa thêm lỗi vào các trường hợp khác.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Mô tả

bug
Generated by Generative AI

No

Operating System:

any

ROS version or commit hash:

rolling, lyrical

RMW implementation (if applicable):

No response

RMW Configuration (if applicable):

No response

Client library (if applicable):

No response

'ros2 doctor --report' output

No response

Steps to reproduce issue

See #263

Expected behavior

Tests pass

Actual behavior

Tests fail

- rosidl_generator_py.cpplint_rosidl_generated_py build/include_order [4] (/ws/build/rosidl_generator_py/rosidl_generator_py/rosidl_generator_py/srv/_get_buffer_s.c:83)
  <<< failure message
    Found C system header after other header. Should be: _get_buffer_s.h, c system, c++ system, other.
  >>>
Additional information

#250 injects <stdint.h> into the headers, which might appear in the generated code after non-C headers, if the code is for services and actions, which have buffer type in their response/result/feedback, but not in the response/goal.

There are two simple fixes:

  1. silence build/include_order in the cpplint checks
  2. always add <stdint.h>

A more complicated fix would add stdint.h as in the request/goal part, if the parts use buffers.

Ngôn ngữ chính
EmberScript
Star
26
Fork
68
Merge trung bình
1 ngày 9 giờ
Pull request đã merge (30 ngày)
3

Chuẩn bị môi trường

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Issue khác của ros2/rosidl_python

Tất cả issue của ros2/rosidl_python

Issue tương tự

Thêm issue về Build System

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.