Code

Go Cleanup Built-in Rules#265

Code

Merged

dvmarcilio merged 56 commits intomasteruber/piranha:masterfrom go_developuber/piranha:go_developCopy head branch name to clipboardDec 19, 2022

+1,848-75

Commits

Commits on Dec 18, 2022

• Pointing to modified Go grammar for now

  dvmarcilio

  committed
  872687c

• Go: Initial boolean_expression_simplify rules

  dvmarcilio

  committed
  fb8e013

• Unary_expression has "hidden" field operator

  Show description for bf726c1
  dvmarcilio

  committed
  bf726c1

• Go: simplify {true|false}_and_something...

  dvmarcilio

  committed
  3833c4e

• Go: simplify {true|false} or something...

  dvmarcilio

  committed
  769ba90

• Go: converging to a standard rule formatting

  dvmarcilio

  committed
  fee496d

• Go: if_cleanup && remove_unneccessary_nested_block

  dvmarcilio

  committed
  8acc48d

• Go: TODO comment to address if w/ short stmt

  dvmarcilio

  committed
  215f11d

• Go: temp start for delete_short_v_decl

  dvmarcilio

  committed
  3e4e10f

• 'delete_immediate_if_err' working. clean rules/edges/scope later

  dvmarcilio

  committed
  b2b0674

• replace_double_short_var_decl working!

  dvmarcilio

  committed
  f4d9bac

• delete_variable_declaration

  dvmarcilio

  committed
  51500e1

• One intra cleanup scenario working!

  dvmarcilio

  committed
  eb85d56

• Merging two rules into one

  dvmarcilio

  committed
  6cb92ed

• Fix hole; Scope Function -> Function-Method

  dvmarcilio

  committed
  a9530c5

• Adding graph for current built-in rules

  dvmarcilio

  committed
  8e85e59

• Go: constant same file

  dvmarcilio

  committed
  e086048

• delete_bool_blank_identifier_short_var_declaration

  dvmarcilio

  committed
  6875fbb

• Go: Removing unused param used in call

  dvmarcilio

  committed
  dbc86f7

• Go: delete_all_statements_after_return

  dvmarcilio

  committed
  93d0bf2

• Simplifying identity equality and inequality for {true|false|nil}

  Show description for 02200fa
  dvmarcilio

  committed
  02200fa

• Go: Cleanup rules work for single short var decl

  Show description for 9c89491
  dvmarcilio

  committed
  9c89491

• delete_variable_declaration needs a constraint!

  dvmarcilio

  committed
  a0435b5

• replace_identifier_with_value needs a constraint

  dvmarcilio

  committed
  c5de253

• More meaningful names for identity rules

  dvmarcilio

  committed
  d4249aa

• Moving Go tests

  dvmarcilio

  committed
  e7276bc

• Go delete_statement_after_return self-edge

  dvmarcilio

  committed
  fcf2633

• Removing outdated image

  dvmarcilio

  committed
  de9ad4c

• --ignore-parser-error option. (rebased so this commit is very weird)

  Show description for 0cefd06
  dvmarcilio

  committed
  0cefd06

• Reverting ignore_parse_error arg (rebased; commit is weird)

  dvmarcilio

  committed
  ec97026

• Initial cleanup of go rules

  dvmarcilio

  committed
  5ccc4b1

• Go simplify_identity: wildcard for lhs and rhs

  dvmarcilio

  committed
  a6255e2

• Go: test for simplify if false with no alternative

  dvmarcilio

  committed
  d049ed8

• Removing .s from delete_parent_assignment

  dvmarcilio

  committed
  8a24525

• * and + rules with \n terminator

  Show description for fcb1c48
  dvmarcilio

  committed
  fcb1c48

• Go grammar: branch on Uber tree-sitter-go repo

  dvmarcilio

  committed
  9439bca

• return_cleanup <-> delete_stmt_after_return

  dvmarcilio

  committed
  533a1be

• Go: removing * from lhs|rhs captures

  dvmarcilio

  committed
  8dba1a2

• constraint for replace_identifier_with_value

  dvmarcilio

  committed
  e2c14ec

• simplify_something_or_true excluding calls

  dvmarcilio

  committed
  7dbde04

• simplify_something_and_false excluding calls

  dvmarcilio

  committed
  2fbd662

• Removing 'delete_parent_assignment' !

  dvmarcilio

  committed
  5224867

• Adding Go in PiranhaLanguage

  dvmarcilio

  committed
  189990c

• dummy 'return_cleanup' -> 'return_statement_cleanup

  dvmarcilio

  committed
  171d4dc

• Simplifying rules for test "const_same_file"

  dvmarcilio

  committed
  7759e90

• const_same_file tests following conventions

  Show description for fb72b1f
  dvmarcilio

  committed
  fb72b1f

• Removing empty edges.toml

  dvmarcilio

  committed
  9e8faf3

• test_piranha_go: Adding needed import

  dvmarcilio

  committed
  67b509a

• Using treated/treated_complement as arguments in test

  dvmarcilio

  committed
  1f93f33

• Fix .toml formatting

  dvmarcilio

  committed
  8cbbe3c

• cleaning remove_unnecessary_nested_block

  dvmarcilio

  committed
  0a1d8da

• Cleaning delete_statement_after_return

  dvmarcilio

  committed
  bea7f6f

• add missing license

  dvmarcilio

  committed
  4ce2547

Commits on Dec 19, 2022

• README: checkmarks for Go

  dvmarcilio

  committed
  2b584aa

• TODO on Cargo.toml to change tree-sitter-go dependency

  dvmarcilio

  committed
  c8f9942

• Merge branch 'master' into go_develop

  dvmarcilio

  authored
  f0b1f55