Skip to content

[Fixed issue] - Detect Line ending changes on Reload#4034

Closed
SinghRajenM wants to merge 1 commit intonotepad-plus-plus:masterfrom
SinghRajenM:src_EOL_IssueFix
Closed

[Fixed issue] - Detect Line ending changes on Reload#4034
SinghRajenM wants to merge 1 commit intonotepad-plus-plus:masterfrom
SinghRajenM:src_EOL_IssueFix

Conversation

@SinghRajenM
Copy link
Copy Markdown
Contributor

Fixed issue #4033

@CookiePLMonster
Copy link
Copy Markdown
Contributor

CookiePLMonster commented Jan 13, 2018

Ouch, haven't noticed your pull request before submitting #4077... not sure what to do now, since I also corrected detecting encoding on reload (so default encoding for ANSI files is respected when reloading) :/

I can close my PR if there is a fix for #2843 out there somewhere - and since nobody mentioned it in the issue, I don't think there is.

@SinghRajenM
Copy link
Copy Markdown
Contributor Author

SinghRajenM commented Jan 15, 2018

Ouch, haven't noticed your pull request before submitting #4077... not sure what to do now

No problem. I feel it is ok to keep the both PR active and let @donho decide which/how to merge. Because, this PR just handle the UI updation, while PR #4077 deals with actual encoding in certain scenarios.

@CookiePLMonster
Copy link
Copy Markdown
Contributor

That's cool then =) I guess if your PR gets merged first then mine will throw some merge issues, which actually won't be merge issues (as it's 1:1 code), and thus both can work together just fine.

@donho
Copy link
Copy Markdown
Member

donho commented Jan 28, 2018

46d0f12

@donho donho closed this Jan 28, 2018
@SinghRajenM SinghRajenM deleted the src_EOL_IssueFix branch January 29, 2018 02:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants