LibGUI: Remove unnecessary type cast in JsonArrayModel.

Since TCP sequence numbers are randomly choosen 32-bit numbers, it often happend that the most significant bit was set. The cast to a 32-bit signed integer then made the number negative. Thus TCP sequence were shown negative in the SystemMonitor every so often.
akimdi · Sep 20, 2020 · 03a27bc · 03a27bc
1 parent 3ce97b9
commit 03a27bc
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/Libraries/LibGUI/JsonArrayModel.cpp b/Libraries/LibGUI/JsonArrayModel.cpp
@@ -107,7 +107,7 @@ Variant JsonArrayModel::data(const ModelIndex& index, ModelRole role) const
  if (field_spec.massage_for_display)
  return field_spec.massage_for_display(object);
  if (data.is_number())
- return data.to_i32();
+ return data;
  return object.get(json_field_name).to_string();
  }
 

diff --git a/Libraries/LibGUI/JsonArrayModel.h b/Libraries/LibGUI/JsonArrayModel.h
@@ -64,7 +64,7 @@ class JsonArrayModel final : public Model {
  return adopt(*new JsonArrayModel(json_path, move(fields)));
  }
 
- virtual ~JsonArrayModel() override {}
+ virtual ~JsonArrayModel() override { }
 
  virtual int row_count(const ModelIndex& = ModelIndex()) const override { return m_array.size(); }
  virtual int column_count(const ModelIndex& = ModelIndex()) const override { return m_fields.size(); }