java-defects4j-closure-100-checkglobalthis

A painting is a person's account of this edit as text, recorded independently of the node mapping - a rendering often has several equally defensible answers where the mapping has one. Deletions and insertions carry no position on the opposite side in a painting, so that panel draws no caret for them.

Before

program

package_declaration

package "package"

scoped_identifier

identifier "com"

. "."

identifier "google"

. "."

identifier "javascript"

. "."

identifier "jscomp"

; ";"

import_declaration

import "import"

scoped_identifier

identifier "com"

. "."

identifier "google"

. "."

identifier "javascript"

. "."

identifier "jscomp"

. "."

identifier "CheckLevel"

; ";"

import_declaration

import "import"

scoped_identifier

identifier "com"

. "."

identifier "google"

. "."

identifier "javascript"

. "."

identifier "jscomp"

. "."

identifier "NodeTraversal"

. "."

identifier "Callback"

; ";"

import_declaration

import "import"

scoped_identifier

identifier "com"

. "."

identifier "google"

. "."

identifier "javascript"

. "."

identifier "rhino"

. "."

identifier "JSDocInfo"

; ";"

import_declaration

import "import"

scoped_identifier

identifier "com"

. "."

identifier "google"

. "."

identifier "javascript"

. "."

identifier "rhino"

. "."

identifier "Node"

; ";"

import_declaration

import "import"

scoped_identifier

identifier "com"

. "."

identifier "google"

. "."

identifier "javascript"

. "."

identifier "rhino"

. "."

identifier "Token"

; ";"

block_comment "/**\n * Checks for certain uses of the {@code this} keyword t"...

class_declaration

modifiers

final "final"

class "class"

identifier "CheckGlobalThis"

super_interfaces

implements "implements"

type_list

type_identifier "Callback"

class_body

{ "{"

field_declaration (+25 nodes collapsed)

field_declaration

modifiers

private "private"

final "final"

type_identifier "AbstractCompiler"

variable_declarator

identifier "compiler"

; ";"

field_declaration

modifiers

private "private"

final "final"

type_identifier "CheckLevel"

variable_declarator

identifier "level"

; ";"

block_comment "/**\n * If {@code assignLhsChild != null}, then the node be"...

field_declaration

modifiers

private "private"

type_identifier "Node"

variable_declarator

identifier "assignLhsChild"

= "="

null_literal "null"

; ";"

constructor_declaration (+33 nodes collapsed)

block_comment "/**\n * Since this pass reports errors only when a global {"...

method_declaration

modifiers

public "public"

boolean_type "boolean"

identifier "shouldTraverse"

formal_parameters

( "("

formal_parameter

type_identifier "NodeTraversal"

identifier "t"

, ","

formal_parameter

type_identifier "Node"

identifier "n"

, ","

formal_parameter

type_identifier "Node"

identifier "parent"

) ")"

block

{ "{"

if_statement

if "if"

parenthesized_expression

( "("

binary_expression

method_invocation

identifier "n"

. "."

identifier "getType"

argument_list

( "("

) ")"

== "=="

field_access

identifier "Token"

. "."

identifier "FUNCTION"

) ")"

block

{ "{"

line_comment "// Don't traverse functions that are constructors or have th"...

line_comment "// annotation."

local_variable_declaration

type_identifier "JSDocInfo"

variable_declarator

identifier "jsDoc"

= "="

method_invocation

identifier "getFunctionJsDocInfo"

argument_list

( "("

identifier "n"

) ")"

; ";"

if_statement (+37 nodes collapsed)

line_comment "// Don't traverse functions unless they would normally"

line_comment "// be able to have a @this annotation associated with them. "...

line_comment "// var a = function() { }; // or"

line_comment "// function a() {} // or"

line_comment "// a.x = function() {};"

} "}"

if_statement (+190 nodes collapsed)

return_statement

return "return"

true "true"

; ";"

} "}"

method_declaration (+92 nodes collapsed)

method_declaration

modifiers

private "private"

boolean_type "boolean"

identifier "shouldReportThis"

formal_parameters

( "("

formal_parameter

type_identifier "Node"

identifier "n"

, ","

formal_parameter

type_identifier "Node"

identifier "parent"

) ")"

block

{ "{"

if_statement

if "if"

parenthesized_expression

( "("

binary_expression

identifier "assignLhsChild"

!= "!="

null_literal "null"

) ")"

block

{ "{"

line_comment "// Always report a THIS on the left side of an assign."

return_statement

return "return"

true "true"

; ";"

} "}"

line_comment "// Also report a THIS with a property access."

return_statement

return "return"

false "false"

; ";"

} "}"

block_comment "/**\n * Gets a function's JSDoc information, if it has any."...

method_declaration (+173 nodes collapsed)

} "}"

After

program

package_declaration

package "package"

scoped_identifier

identifier "com"

. "."

identifier "google"

. "."

identifier "javascript"

. "."

identifier "jscomp"

; ";"

import_declaration

import "import"

scoped_identifier

identifier "com"

. "."

identifier "google"

. "."

identifier "javascript"

. "."

identifier "jscomp"

. "."

identifier "CheckLevel"

; ";"

import_declaration

import "import"

scoped_identifier

identifier "com"

. "."

identifier "google"

. "."

identifier "javascript"

. "."

identifier "jscomp"

. "."

identifier "NodeTraversal"

. "."

identifier "Callback"

; ";"

import_declaration

import "import"

scoped_identifier

identifier "com"

. "."

identifier "google"

. "."

identifier "javascript"

. "."

identifier "rhino"

. "."

identifier "JSDocInfo"

; ";"

import_declaration

import "import"

scoped_identifier

identifier "com"

. "."

identifier "google"

. "."

identifier "javascript"

. "."

identifier "rhino"

. "."

identifier "Node"

; ";"

import_declaration

import "import"

scoped_identifier

identifier "com"

. "."

identifier "google"

. "."

identifier "javascript"

. "."

identifier "rhino"

. "."

identifier "Token"

; ";"

block_comment "/**\n * Checks for certain uses of the {@code this} keyword t"...

class_declaration

modifiers

final "final"

class "class"

identifier "CheckGlobalThis"

super_interfaces

implements "implements"

type_list

type_identifier "Callback"

class_body

{ "{"

field_declaration (+25 nodes collapsed)

field_declaration

modifiers

private "private"

final "final"

type_identifier "AbstractCompiler"

variable_declarator

identifier "compiler"

; ";"

field_declaration

modifiers

private "private"

final "final"

type_identifier "CheckLevel"

variable_declarator

identifier "level"

; ";"

block_comment "/**\n * If {@code assignLhsChild != null}, then the node be"...

field_declaration

modifiers

private "private"

type_identifier "Node"

variable_declarator

identifier "assignLhsChild"

= "="

null_literal "null"

; ";"

constructor_declaration (+33 nodes collapsed)

block_comment "/**\n * Since this pass reports errors only when a global {"...

method_declaration

modifiers

public "public"

boolean_type "boolean"

identifier "shouldTraverse"

formal_parameters

( "("

formal_parameter

type_identifier "NodeTraversal"

identifier "t"

, ","

formal_parameter

type_identifier "Node"

identifier "n"

, ","

formal_parameter

type_identifier "Node"

identifier "parent"

) ")"

block

{ "{"

if_statement

if "if"

parenthesized_expression

( "("

binary_expression

method_invocation

identifier "n"

. "."

identifier "getType"

argument_list

( "("

) ")"

== "=="

field_access

identifier "Token"

. "."

identifier "FUNCTION"

) ")"

block

{ "{"

line_comment "// Don't traverse functions that are constructors or have th"...

line_comment "// annotation."

local_variable_declaration

type_identifier "JSDocInfo"

variable_declarator

identifier "jsDoc"

= "="

method_invocation

identifier "getFunctionJsDocInfo"

argument_list

( "("

identifier "n"

) ")"

; ";"

if_statement (+37 nodes collapsed)

line_comment "// Don't traverse functions unless they would normally"

line_comment "// be able to have a @this annotation associated with them. "...

line_comment "// var a = function() { }; // or"

line_comment "// function a() {} // or"

line_comment "// a.x = function() {};"

local_variable_declaration

integral_type

int "int"

variable_declarator

identifier "pType"

= "="

method_invocation

identifier "parent"

. "."

identifier "getType"

argument_list

( "("

) ")"

; ";"

if_statement

if "if"

parenthesized_expression

( "("

unary_expression

! "!"

parenthesized_expression

( "("

binary_expression

identifier "pType"

== "=="

field_access

identifier "Token"

. "."

identifier "BLOCK"

|| "||"

binary_expression

identifier "pType"

== "=="

field_access

identifier "Token"

. "."

identifier "SCRIPT"

|| "||"

binary_expression

identifier "pType"

== "=="

field_access

identifier "Token"

. "."

identifier "NAME"

|| "||"

binary_expression

identifier "pType"

== "=="

field_access

identifier "Token"

. "."

identifier "ASSIGN"

) ")"

block

{ "{"

return_statement

return "return"

false "false"

; ";"

} "}"

if_statement (+190 nodes collapsed)

return_statement

return "return"

true "true"

; ";"

} "}"

method_declaration (+92 nodes collapsed)

method_declaration

modifiers

private "private"

boolean_type "boolean"

identifier "shouldReportThis"

formal_parameters

( "("

formal_parameter

type_identifier "Node"

identifier "n"

, ","

formal_parameter

type_identifier "Node"

identifier "parent"

) ")"

block

{ "{"

if_statement

if "if"

parenthesized_expression

( "("

binary_expression

identifier "assignLhsChild"

!= "!="

null_literal "null"

) ")"

block

{ "{"

line_comment "// Always report a THIS on the left side of an assign."

return_statement

return "return"

true "true"

; ";"

} "}"

line_comment "// Also report a THIS with a property access."

return_statement

return "return"

binary_expression

identifier "parent"

!= "!="

null_literal "null"

&& "&&"

method_invocation

identifier "NodeUtil"

. "."

identifier "isGet"

argument_list

( "("

identifier "parent"

) ")"

; ";"

} "}"

block_comment "/**\n * Gets a function's JSDoc information, if it has any."...

method_declaration (+173 nodes collapsed)

} "}"