kicad-developers team mailing list archive
-
kicad-developers team
-
Mailing list archive
-
Message #42061
[PATCH 0/2] Remove useless casts
Hi,
I have two patches that remove useless casts. The first covers cases where
it is quite obvious that the cast can be removed with no adverse effects,
the second contains the cases where that is less clear.
In theory, both can be applied and should not cause code changes, but the
second needs review to see if something else was originally intended by
that code.
Simon
Simon Richter (2):
Remove useless casts
Remove questionable useless casts
3d-viewer/3d_cache/sg/sg_helpers.h | 4 ++--
3d-viewer/3d_canvas/create_layer_items.cpp | 10 ++++-----
.../3d_model_viewer/c3d_model_viewer.cpp | 2 +-
.../c3d_render_ogl_legacy.cpp | 2 +-
.../3d_render_ogl_legacy/clayer_triangles.h | 4 ++--
.../accelerators/cbvh_pbrt.cpp | 2 +-
.../c3d_render_createscene.cpp | 8 +++----
.../c3d_render_raytracing.cpp | 4 ++--
.../shapes2D/citemlayercsg2d.cpp | 6 ++---
.../shapes2D/cpolygon2d.cpp | 2 +-
.../shapes3D/clayeritem.cpp | 4 ++--
3d-viewer/3d_rendering/cimage.cpp | 4 ++--
3d-viewer/3d_rendering/cpostshader.h | 2 +-
3d-viewer/3d_rendering/trackball.cpp | 12 +++++-----
bitmap2component/bitmap2cmp_main.cpp | 2 +-
common/base_struct.cpp | 2 +-
common/bezier_curves.cpp | 4 ++--
common/board_printout.cpp | 2 +-
common/draw_panel_gal.cpp | 4 ++--
common/gal/opengl/opengl_compositor.cpp | 6 ++---
common/gal/opengl/opengl_gal.cpp | 2 +-
common/gal/opengl/shader.cpp | 4 ++--
common/lib_table_base.cpp | 2 +-
common/tool/picker_tool.cpp | 2 +-
common/utf8.cpp | 2 +-
common/view/view.cpp | 4 ++--
common/widgets/footprint_preview_widget.cpp | 4 ++--
common/widgets/net_selector.cpp | 2 +-
common/xnode.cpp | 4 ++--
cvpcb/cvpcb.cpp | 2 +-
cvpcb/display_footprints_frame.cpp | 2 +-
dxflib_qcad/dl_dxf.cpp | 2 +-
eeschema/dialogs/dialog_erc.cpp | 2 +-
eeschema/generate_alias_info.cpp | 2 +-
eeschema/hierarch.cpp | 2 +-
eeschema/libedit/toolbars_libedit.cpp | 2 +-
.../netlist_exporter_pspice.cpp | 2 +-
eeschema/sch_eagle_plugin.cpp | 4 ++--
eeschema/sch_marker.cpp | 2 +-
eeschema/sch_sheet_path.h | 2 +-
eeschema/toolbars_viewlib.cpp | 2 +-
eeschema/tools/ee_inspection_tool.cpp | 2 +-
eeschema/tools/ee_point_editor.cpp | 10 ++++-----
eeschema/tools/ee_selection.cpp | 2 +-
eeschema/tools/ee_selection_tool.cpp | 2 +-
eeschema/tools/lib_edit_tool.cpp | 2 +-
eeschema/tools/sch_drawing_tools.cpp | 6 ++---
eeschema/tools/sch_edit_tool.cpp | 4 ++--
eeschema/tools/sch_editor_control.cpp | 2 +-
gerbview/gerbview_frame.cpp | 2 +-
gerbview/tools/gerbview_control.cpp | 2 +-
gerbview/tools/gerbview_selection_tool.cpp | 2 +-
include/gal/graphics_abstraction_layer.h | 4 ++--
include/gal/opengl/shader.h | 2 +-
include/utf8.h | 6 ++---
pagelayout_editor/pl_editor.cpp | 2 +-
pagelayout_editor/pl_editor_frame.cpp | 2 +-
pagelayout_editor/tools/pl_edit_tool.cpp | 2 +-
pagelayout_editor/tools/pl_editor_control.cpp | 2 +-
pagelayout_editor/tools/pl_point_editor.cpp | 2 +-
pagelayout_editor/tools/pl_selection_tool.cpp | 2 +-
pcb_calculator/pcb_calculator.cpp | 2 +-
.../class_board_stackup.cpp | 4 ++--
pcbnew/class_board.cpp | 4 +---
pcbnew/class_dimension.cpp | 2 +-
pcbnew/class_drawsegment.cpp | 4 ++--
pcbnew/class_marker_pcb.cpp | 8 +++----
pcbnew/class_module.cpp | 4 ++--
pcbnew/class_module.h | 4 ++--
pcbnew/class_pcb_target.cpp | 2 +-
pcbnew/class_pcb_text.cpp | 2 +-
pcbnew/class_track.cpp | 4 ++--
pcbnew/class_zone.cpp | 2 +-
pcbnew/connectivity/connectivity_algo.cpp | 8 +++----
.../convert_drawsegment_list_to_polygon.cpp | 4 ++--
pcbnew/dialogs/dialog_create_array.cpp | 4 ++--
pcbnew/exporters/export_gencad.cpp | 4 +---
pcbnew/files.cpp | 2 +-
pcbnew/footprint_info_impl.cpp | 2 +-
pcbnew/legacy_plugin.cpp | 2 +-
pcbnew/pad_print_functions.cpp | 2 +-
pcbnew/pcb_base_frame.cpp | 10 ++++-----
pcbnew/pcb_draw_panel_gal.cpp | 2 +-
pcbnew/pcb_view.cpp | 2 +-
pcbnew/plot_board_layers.cpp | 3 +--
pcbnew/plot_brditems_plotter.cpp | 2 +-
pcbnew/router/pns_diff_pair_placer.cpp | 4 ++--
pcbnew/router/pns_line_placer.cpp | 2 +-
pcbnew/router/pns_meander.cpp | 4 ++--
pcbnew/router/pns_sizes_settings.cpp | 2 +-
pcbnew/specctra_import_export/specctra.cpp | 4 ++--
pcbnew/specctra_import_export/specctra.h | 8 +++----
pcbnew/swig/pcbnew_action_plugins.cpp | 4 ++--
pcbnew/tools/drc.cpp | 2 +-
pcbnew/tools/edit_tool.cpp | 4 ++--
pcbnew/tools/pcb_tool_base.cpp | 2 +-
pcbnew/tools/pcbnew_control.cpp | 6 ++---
pcbnew/tools/pcbnew_picker_tool.cpp | 2 +-
pcbnew/tools/selection_tool.cpp | 2 +-
pcbnew/undo_redo.cpp | 2 +-
plugins/3d/idf/s3d_plugin_idf.cpp | 2 +-
polygon/clipper.cpp | 22 +++++++++----------
qa/libs/sexpr/test_sexpr_parser.cpp | 4 ++--
utils/idftools/idf2vrml.cpp | 4 ++--
104 files changed, 179 insertions(+), 188 deletions(-)
--
2.20.1
Follow ups