Fixed bug with preprocessor including the same file multiple times. Fixes 363

[Konstantin8105]

See #363

---

This change is 

[codecov]

Codecov Report

Merging #369 into master will increase coverage by 0.01%.
The diff coverage is 89.47%.

@@            Coverage Diff             @@
##           master     #369      +/-   ##
==========================================
+ Coverage   81.71%   81.73%   +0.01%     
==========================================
  Files         144      144              
  Lines        6192     6208      +16     
==========================================
+ Hits         5060     5074      +14     
- Misses        907      908       +1     
- Partials      225      226       +1

Impacted Files	Coverage Δ
preprocessor/preprocessor.go	70.7% <89.47%> (+3.23%)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 9aa8212...cfc3c7f. Read the comment docs.

[elliotchance]

Reviewed 1 of 1 files at r1, 3 of 3 files at r2.
Review status: all files reviewed at latest revision, 1 unresolved discussion.

---

preprocessor/preprocessor.go, line 41 at r2 (raw file):

				allItems[i].other == allItems[j].other &&
				len(allItems[i].lines) == len(allItems[j].lines) {
				indentical := true

"identical". Can you also explain a bit what this code is doing with a comment?

---

Comments from Reviewable

[Konstantin8105]

Review status: all files reviewed at latest revision, 1 unresolved discussion.

---

preprocessor/preprocessor.go, line 41 at r2 (raw file):

Previously, elliotchance (Elliot Chance) wrote…

"identical". Can you also explain a bit what this code is doing with a comment?

Yes, you are right . This part of code, is huge. So I separate to function isSame.

---

Comments from Reviewable

[elliotchance]

Reviewed 1 of 1 files at r3.
Review status: all files reviewed at latest revision, all discussions resolved.

---

Comments from Reviewable