threads / bug / 39022

git diff bug? what problem may be here

Subject: git diff bug? what problem may be here

## tl;dr

2 messages between Apr 7, 2015 and Apr 7, 2015.

replies: 1people: 1as markdown or json

KES· Apr 7, 2015, 06:04 UTC · lore
When after editing I get this:
 mdi.Application= this;
 mdi.Class.TApplication.baseConstructor.apply( this, arguments );
git diff shows:
 
  mdi.Application= this;
- mdi.Class.TApplication.baseConstructor.apply( this, arguments );
 
- //We must not create/show application node because of this is the 'document.bo
- //It exists always so apply application properties to 'document.body' only
- this.updateDomNode();
+ mdi.Class.TApplication.baseConstructor.apply( this, arguments );
 },{
Notice same lines:
- mdi.Class.TApplication.baseConstructor.apply( this, arguments );
+ mdi.Class.TApplication.baseConstructor.apply( this, arguments );
But when I edit like this (notice no new line between mdi.Application and mdi.Class):
 mdi.Application= this;
 mdi.Class.TApplication.baseConstructor.apply( this, arguments );
Git diff shows:
  mdi.Application= this;
  mdi.Class.TApplication.baseConstructor.apply( this, arguments );
-
- //We must not create/show application node because of this is the 'document.body'.
- //It exists always so apply application properties to 'document.body' only
- this.updateDomNode();
 },{
Why new line cause same line in patch added and removed?
KES· Apr 7, 2015, 17:27 UTC · re: KES · lore
When after editing I get this:
 mdi.Application= this;
 mdi.Class.TApplication.baseConstructor.apply( this, arguments );
git diff shows:
  mdi.Application= this;
- mdi.Class.TApplication.baseConstructor.apply( this, arguments );
- //We must not create/show application node because of this is the 'document.bo
- //It exists always so apply application properties to 'document.body' only
- this.updateDomNode();
+ mdi.Class.TApplication.baseConstructor.apply( this, arguments );
 },{
Notice same lines:
- mdi.Class.TApplication.baseConstructor.apply( this, arguments );
+ mdi.Class.TApplication.baseConstructor.apply( this, arguments );
But when I edit like this (notice no new line between mdi.Application and mdi.Class):
 mdi.Application= this;
 mdi.Class.TApplication.baseConstructor.apply( this, arguments );
Git diff shows:
  mdi.Application= this;
  mdi.Class.TApplication.baseConstructor.apply( this, arguments );
-
- //We must not create/show application node because of this is the 'document.body'.
- //It exists always so apply application properties to 'document.body' only
- this.updateDomNode();
 },{
Why new line cause same line in patch added and removed?

← back to recent threads