Skip to content

Commit

Permalink
remove parameters from EmptyArray form
Browse files Browse the repository at this point in the history
  • Loading branch information
ManasviGoyal committed Aug 4, 2023
1 parent aba0b8d commit dd3699b
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 5 deletions.
2 changes: 1 addition & 1 deletion header-only/layout-builder/awkward/LayoutBuilder.h
Original file line number Diff line number Diff line change
Expand Up @@ -415,7 +415,7 @@ namespace awkward {
/// contents in the form of a JSON-like string.
std::string
form() const noexcept {
return "{ \"class\": \"EmptyArray\", \"parameters\": {} }";
return "{ \"class\": \"EmptyArray\" }";
}

private:
Expand Down
6 changes: 2 additions & 4 deletions header-only/tests/test_1494-layout-builder.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -460,8 +460,7 @@ test_Empty() {

assert(builder.form() ==
"{ "
"\"class\": \"EmptyArray\", "
"\"parameters\": {} "
"\"class\": \"EmptyArray\" "
"}");

clear_buffers(buffers);
Expand Down Expand Up @@ -532,8 +531,7 @@ test_ListOffset_Empty() {
"\"class\": \"ListOffsetArray\", "
"\"offsets\": \"i64\", "
"\"content\": { "
"\"class\": \"EmptyArray\", "
"\"parameters\": {} "
"\"class\": \"EmptyArray\" "
"}, "
"\"form_key\": \"node1\" "
"}, "
Expand Down

0 comments on commit dd3699b

Please sign in to comment.