Skip to content
Snippets Groups Projects

Updating leap mode disk writing, leap mode folding and MPG-SKA dish 10 bit merger

Merged Jason Wu requested to merge LEAP into devel
1 unresolved thread

Merge request reports

Pipeline #92302 passed

Pipeline passed for 71fc285b on LEAP

Approval is optional

Merged by Tobias WinchenTobias Winchen 4 years ago (Jan 28, 2021 10:15am UTC)

Merge details

  • Changes merged into devel with b3619f8d.
  • Deleted the source branch.

Pipeline #92335 failed

Pipeline failed for b3619f8d on devel

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
9 namespace psrdada_cpp {
10 namespace effelsberg {
11 namespace edd {
12 class DiskSinkLeap
13 {
14 public:
15 DiskSinkLeap(std::string prefix, int nchan);
16 ~DiskSinkLeap();
17 void init(RawBytes& block);
18 bool operator()(RawBytes& block);
19
20 public:
21 std::string _prefix;
22 std::size_t _counter;
23 int _nchan;
24 char _header[4096];
  • If possible, use a global define or const to specify header sizes, block sizes, etc. Even if they will never change:

     char _header[leap_header_size];
    ...
    for (int i =0; i < leap_header_size; i++) {...};
    ...

    is much clearer than

     char _header[4096];  // 4096 is fixed leap header size
    ...
    
    // 4096 is fixed leap header size
    for (int i =0; i < 4096; i++) {...};
    ...

    even with the comments.

  • changed this line in version 2 of the diff

  • Please register or sign in to reply
  • Jason Wu added 1 commit

    added 1 commit

    Compare with previous version

  • Jason Wu added 1 commit

    added 1 commit

    • 82fc80ca - got rid of unused parameters and definfing global constant

    Compare with previous version

  • Jason Wu added 1 commit

    added 1 commit

    Compare with previous version

  • Jason Wu added 1 commit

    added 1 commit

    Compare with previous version

  • All fixed, I believe.

  • Tobias Winchen mentioned in commit b3619f8d

    mentioned in commit b3619f8d

  • Please register or sign in to reply
    Loading