pnedev / comparePlus

Compare plugin for Notepad++
GNU General Public License v3.0
988 stars 139 forks source link

Crash when compare two completely different files. #231

Closed ArkadiuszMichalski closed 2 years ago

ArkadiuszMichalski commented 4 years ago

Hi, I get crash when compare two completely different files. I use Build time: Mar 2 2020, 15:16:47 compare version. Step to reproduce:

1. Open first localize file, like english.xml.
2. Open second localize file, like german.xml.
3. Do compare.

NPP will crash every time when do above steps.

ArkadiuszMichalski commented 4 years ago

One another case when I get crash, discovered accidentally, that's why I will describe it here.

  1. Save first file with this data:

    
    Notepad++ v7.8.2 enhancements and bug-fixes:
  2. Add key shortcuts in Save or not dialog.

  3. Add find in files filter excluding ability.

  4. Add "Open UDL folder" command.

  5. Fix issue of showing save dialog twice while cloned file is dirty.

  6. Enhance "Open Explorer" command in Folder as workspace with expected path.

  7. Fix tab dragging issues.

  8. Enhance "Close all but This" and "Close all to the left/right" commands while files are dirty.

  9. Enhance "In Selection" option in Find dialog.

  10. Fix Notepad++ blocked in notification zone issue.

  11. Make end-of-line conversion operations macroable.

  12. Fix "View Current File In" commands disabled problem in some situations.

Included plugins:

  1. NppExport v0.2.9
  2. Converter 4.2.1
  3. Mime Tool 2.5

Updater (Installer only):

  1. Fix "View Current File In" commands disabled problem in some situations.

// Komentarz Included plugins:

  1. NppExport v0.2.9
  2. Converter 4.2.1
  3. Mime Tool 2.5

Updater (Installer only):

NPP will crash every time when do above steps.

pnedev commented 4 years ago

@ArkadiuszMichalski ,

Perhaps your version of Notepad++ is above 7.7. ComparePlus plugin is still in development - it is not released yet. The development build is not suited for Notepad++ versions above 7.7 (included) - the changes needed by the new Scintilla component are not integrated yet. It will be done for the official release.

BR

pnedev commented 2 years ago

Implemented/fixed and will be included in the first ComparePlus plugin release coming in a month or so.