Bugs / Найденные баги (2)

Skif_off · Post by **Skif_off** » Wed Oct 10, 2018 8:46 am

cordylus
Вопрос скорее к python.coder. Наверное, можно попробовать перенести правила подсветки строк из Quotes в QuotesRE, там, правда, ещё Escape char остаётся...

А зачем так писать? Напомнило мои потуги сделать подсветку Ruby: немало нерекомендуемых к использоварию вольностей синтаксиса ломали подсветку на раз.

cordylus · Post by **cordylus** » Fri Oct 12, 2018 7:51 pm

Skif_off
Переместил туда, спасибо.

А зачем писать - в том конкретном случае был многострочный HTML-шаблон параметром, почему бы и не так?

cordylus · Post by **cordylus** » Fri Oct 19, 2018 1:33 pm

Другая проблема: символ €, сохраненный в ANSI (код символа 88), не распознается как эта кодировка и исчезает при сохранении.

Проблема в первую очередь в работе с неизвестными символами, они не остаются как квадратики или знаки вопроса, а просто удаляются из текста, что приводит к потере данных при сохранении.

Воспроизвести так: сохранить текст с этим символом в ANSI, переименовать файл (иначе AkelPad запомнит кодировку) и переоткрыть. Откроется как UTF-8 без BOM без этого символа. Сохраняешь - и этого символа как не бывало, файл отличается по размеру от оригинала.

TNX · Post by **TNX** » Fri Nov 09, 2018 9:37 am

Instructor
Встроенные регулярки (Ctrl+F) глючат.

Такое выражение:
1(?!2?abc).+/z

Должно находить 2 последние строки:
1abc/z
12abc/z
1def/z
12def/z

А находит все 4. Захват \n отключен.

DV · Post by DV » Fri Nov 09, 2018 3:53 pm

TNX wrote:1(?!2?abc).+/z

Таки да.
А так работает правильно:

Code: Select all

1(?!(abc|2abc)).+/z

Enyby · Post by **Enyby** » Sat Dec 22, 2018 12:07 pm

Akelpad 4.9.8 x86.

Пользуюсь плагином Exit. Закрытие текущей MDI вкладки по Esc.

Все работает хорошо, кроме случая если идет загрузка крупного файла.
Если при этом нажать Esc, то Akelpad, после завершения загрузки, попробует закрыться сам (закрыв все вкладки), вместо того, чтобы вызвать плагин.

Очень сильно раздражает. Как это можно исправить?

Enyby · Post by **Enyby** » Sat Dec 22, 2018 12:10 pm

Другой момент.

Если есть окно поиска, то вызвать окно замены нельзя. И наоборот.

Серьезно мешает. Почему не сделать закрытие одного окна при вызове другого?

Enyby · Post by **Enyby** » Sat Dec 22, 2018 12:16 pm

Akelpad 4.9.8 x86.

Не работает замена пошагово, если регулярка содержит утверждения.

Воспроизведение:
Текст файла:

Code: Select all

Открываем диалог замены.
Отмечаем: Учитывать регистр и регулярные выражения.
Вводим в поиск

Code: Select all

456(?=123)

Как замену ставим пустую строку.
Жмем "Найти далее". Находит.
Жмем "заменить" несколько раз. Никакой замены нет, хотя переход к следующему варианту есть.

Если нажать "заменить все" то отработает.

Enyby · Post by **Enyby** » Sat Dec 22, 2018 12:23 pm

Akelpad 4.9.8 x86.

В некоторых случаях замена вообще никак не работает. Приходится делать замену руками.

Регулярка

Code: Select all

456 ((?!456 ).){3,}?

Текст:

Code: Select all

Учитывать регистр включено.
Нажимаю "Заменить все" - результатов 1. И происходит одна замена. (для другой регулярки вообще всегда 0).
Если нажать еще раз "Заменить все", снова будет ровно 1 замена.
Нажимаю "найти далее" - находит и выделяет.

Если убрать пробел из регулярки, то все работает.

Enyby · Post by **Enyby** » Sat Dec 22, 2018 12:30 pm

Akelpad 4.9.8 x86.

Нет ресайза окна поиска/замены.

Когда регулярка длинная это очень неудобно.

opk44 · Post by **opk44** » Sat Dec 22, 2018 6:07 pm

Enyby

Если есть окно поиска, то вызвать окно замены нельзя. И наоборот.

Окно "Замена" полностью перекрывает функционал окна "Поиск". Поэтому не вполне ясна цель таких манипуляций. Разве что из соображений безопасности... Если же нет, то от "Поиска" можно вообще отказаться (в пользу "Замены").

Жмем "заменить" несколько раз. Никакой замены нет, хотя переход к следующему варианту есть.

На версии 4.9.9 баг уже не воспроизводится.

(для другой регулярки вообще всегда 0)

Для какой?

Если убрать пробел из регулярки, то все работает.

Вы не могли бы пояснить, что именно начинает без пробела работать из того, что не работало с пробелом (кстати, с которым, а то их в вашей регулярке два)? И с пробелами и без оных (без обоих или без любого из двух) на приведенном вами примере отрабатывается (на моей стороне) идентично. Проверялось и на 4.9.8(x86) и 4.9.9(x86)[r4394]

Когда регулярка длинная это очень неудобно.

Тут не поспоришь.

Infocatcher · Post by **Infocatcher** » Sat Dec 22, 2018 6:11 pm

Enyby wrote:Другой момент.

Если есть окно поиска, то вызвать окно замены нельзя. И наоборот.

Серьезно мешает. Почему не сделать закрытие одного окна при вызове другого?

QSearch-Rus.txt wrote:Функция DialogSwitcher позволяет переключаться между диалогами "Поиск",
"Замена", "Перейти к строке" и панелью QSearch с помощью "горячих клавиш"
этих диалогов. Например, если нажать Ctrl+H в диалоге "Поиск", он
изменится на диалог "Замена". При использовании функции DialogSwitcher
рекомендуется поставить галочку для её автоматической загрузки в окне
настроек плагинов.

Enyby · Post by **Enyby** » Sat Dec 22, 2018 9:05 pm

Окно "Замена" полностью перекрывает функционал окна "Поиск".

У меня два монитора и куча окон. и я тыкаю ctrl+F и начинаю набивать текст не глядя. Так же Ctrl+H. А когда окно черти где и не того типа, это приводит к неожиданным результатам.

На версии 4.9.9 баг уже не воспроизводится.

4.9.9?
Разве 4.9.8 не последняя?

Нигде не увидел 4.9.9.

Для какой?

Code: Select all

\nGG: ((?!\nGG: ).){32536,}?Hash:((?!\nGG: ).)*?(?=\n\n)

Вот пример под нее будет подобрать несколько сложнее.

Вы не могли бы пояснить, что именно начинает без пробела работать из того, что не работало с пробелом (кстати, с которым, а то их в вашей регулярке два)? И с пробелами и без оных (без обоих или без любого из двух) на приведенном вами примере отрабатывается (на моей стороне) идентично. Проверялось и на 4.9.8(x86) и 4.9.9(x86)[r4394]

Вот видео как оно работает у меня: http://sendvid.com/ss0bp95s
Если убрать оба пробела, то проблемы не будет.

Тут не поспоришь.

Я сделал патч, который добавляет ресайз диалога поиска. Также сдвинул кнопки вниз. Его можно куда-то прислать?
Скрины: https://imgur.com/a/zP2uZRq

Code: Select all

diff --git a/AkelFiles/Langs/Resources/English.rc b/AkelFiles/Langs/Resources/English.rc
index 772ea96..35227d8 100644
--- a/AkelFiles/Langs/Resources/English.rc
+++ b/AkelFiles/Langs/Resources/English.rc
@@ -479,12 +479,12 @@ END
 
 IDD_FIND DIALOGEX 70, 70, 257, 108
 STYLE DS_MODALFRAME | DS_3DLOOK | WS_POPUP | WS_VISIBLE | WS_CAPTION | 
-    WS_SYSMENU
+    WS_SYSMENU | WS_THICKFRAME
 EXSTYLE WS_EX_DLGMODALFRAME
 CAPTION "Find"
 FONT 8, "MS Shell Dlg", 0, 0, 0x1
 BEGIN
-    COMBOBOX        IDC_SEARCH_FIND,27,9,160,100,CBS_DROPDOWN | 
+    COMBOBOX        IDC_SEARCH_FIND,27,9,223,100,CBS_DROPDOWN | 
                     CBS_AUTOHSCROLL | WS_VSCROLL | WS_TABSTOP
     CONTROL         "Match &case",IDC_SEARCH_MATCHCASE,"Button",
                     BS_AUTOCHECKBOX | WS_TABSTOP,9,29,109,12
@@ -510,22 +510,22 @@ BEGIN
     CONTROL         "All file&s",IDC_SEARCH_ALLFILES,"Button",
                     BS_AUTORADIOBUTTON,126,87,60,10
     GROUPBOX        "",IDC_SEARCH_ALLFILES_GROUP,121,80,66,19
-    DEFPUSHBUTTON   "&Find next",IDC_SEARCH_FIND_BUTTON,196,6,54,14,
+    DEFPUSHBUTTON   "&Find next",IDC_SEARCH_FIND_BUTTON,196,27,54,14,
                     BS_CENTER
-    PUSHBUTTON      "Cancel",IDCANCEL,196,23,54,14,BS_CENTER
+    PUSHBUTTON      "Cancel",IDCANCEL,196,44,54,14,BS_CENTER
     LTEXT           "What:",IDC_STATIC,4,11,22,12,NOT WS_GROUP
 END
 
 IDD_REPLACE DIALOGEX 70, 70, 257, 122
 STYLE DS_MODALFRAME | DS_3DLOOK | WS_POPUP | WS_VISIBLE | WS_CAPTION | 
-    WS_SYSMENU
+    WS_SYSMENU | WS_THICKFRAME
 EXSTYLE WS_EX_DLGMODALFRAME
 CAPTION "Replace"
 FONT 8, "MS Shell Dlg", 0, 0, 0x1
 BEGIN
-    COMBOBOX        IDC_SEARCH_FIND,27,9,160,100,CBS_DROPDOWN | 
+    COMBOBOX        IDC_SEARCH_FIND,27,9,223,100,CBS_DROPDOWN | 
                     CBS_AUTOHSCROLL | WS_VSCROLL | WS_TABSTOP
-    COMBOBOX        IDC_SEARCH_REPLACE,27,23,160,100,CBS_DROPDOWN | 
+    COMBOBOX        IDC_SEARCH_REPLACE,27,23,223,100,CBS_DROPDOWN | 
                     CBS_AUTOHSCROLL | WS_VSCROLL | WS_TABSTOP
     CONTROL         "Match &case",IDC_SEARCH_MATCHCASE,"Button",
                     BS_AUTOCHECKBOX | WS_TABSTOP,9,43,109,12
@@ -551,13 +551,13 @@ BEGIN
     CONTROL         "All file&s",IDC_SEARCH_ALLFILES,"Button",
                     BS_AUTORADIOBUTTON,126,101,60,10
     GROUPBOX        "",IDC_SEARCH_ALLFILES_GROUP,121,94,66,19
-    DEFPUSHBUTTON   "&Find next",IDC_SEARCH_FIND_BUTTON,196,6,54,14,
+    DEFPUSHBUTTON   "&Find next",IDC_SEARCH_FIND_BUTTON,196,41,54,14,
                     BS_CENTER
-    PUSHBUTTON      "&Replace",IDC_SEARCH_REPLACE_BUTTON,196,23,54,14,
+    PUSHBUTTON      "&Replace",IDC_SEARCH_REPLACE_BUTTON,196,58,54,14,
                     BS_CENTER
-    PUSHBUTTON      "Replace &all",IDC_SEARCH_ALL_BUTTON,196,39,54,14,
+    PUSHBUTTON      "Replace &all",IDC_SEARCH_ALL_BUTTON,196,74,54,14,
                     BS_CENTER
-    PUSHBUTTON      "Cancel",IDCANCEL,196,55,54,14,BS_CENTER
+    PUSHBUTTON      "Cancel",IDCANCEL,196,90,54,14,BS_CENTER
     LTEXT           "What:",IDC_STATIC,4,11,22,12,NOT WS_GROUP
     LTEXT           "With:",IDC_STATIC,4,25,22,12,NOT WS_GROUP
 END
diff --git a/AkelPad.c b/AkelPad.c
index e52fce3..c5a712b 100644
--- a/AkelPad.c
+++ b/AkelPad.c
@@ -444,6 +444,7 @@ BOOL bNoSearchFinishMsg=FALSE;
 BOOL bLockSearchSetTextCatch=FALSE;
 WORD wLastReplaceButtonID=IDC_SEARCH_FIND_BUTTON;
 WNDPROC lpOldComboboxEdit;
+RECT rcFindAndReplaceMinMaxDialog={257, 108, 0, 0};
 
 //Options dialog
 HHOOK hHookPropertySheet;
diff --git a/Edit.c b/Edit.c
index 5497116..57706ac 100644
--- a/Edit.c
+++ b/Edit.c
@@ -229,6 +229,7 @@ extern BOOL bNoSearchFinishMsg;
 extern BOOL bLockSearchSetTextCatch;
 extern WORD wLastReplaceButtonID;
 extern WNDPROC lpOldComboboxEdit;
+extern RECT rcFindAndReplaceMinMaxDialog;
 
 //Options dialog
 extern HHOOK hHookPropertySheet;
@@ -9383,6 +9384,9 @@ BOOL CALLBACK FindAndReplaceDlgProc(HWND hDlg, UINT uMsg, WPARAM wParam, LPARAM
   int nReplaceFiles;
   int i;
   INT_PTR nResult;
+  static RESIZEDIALOG rds[]={{&hWndFind,       		RDS_SIZE|RDS_X, 0},
+                             {&hWndReplace,         RDS_SIZE|RDS_X, 0},
+                             {0, 0, 0}};
 
   if (uMsg == WM_INITDIALOG)
   {
@@ -9898,6 +9902,8 @@ BOOL CALLBACK FindAndReplaceDlgProc(HWND hDlg, UINT uMsg, WPARAM wParam, LPARAM
       }
     }
   }
+  ResizeDialogMessages(&rds[0], &rcFindAndReplaceMinMaxDialog, &moCur.rcSearchCurrentDialog, RDM_PAINTSIZEGRIP, hDlg, uMsg, wParam, lParam);
+  
   return FALSE;
 }

Функция DialogSwitcher позволяет переключаться между диалогами "Поиск",

То что надо. Спасибо.

Enyby · Post by **Enyby** » Sat Dec 22, 2018 10:04 pm

Я так понимаю, никакого адекватного прерывания загрузки для MDI нет?
То что есть, явно не для MDI:

Code: Select all

              //Check for stop processing
              //if (dwProgressType == AEPGS_SETTEXT || dwProgressType == AEPGS_STREAMIN)
              {
                while (PeekMessageWide(&msg, hMainWnd, WM_KEYFIRST, WM_KEYLAST, PM_REMOVE))
                {
                  if (msg.message == WM_KEYDOWN && msg.wParam == VK_ESCAPE)
                  {
                    bStop=TRUE;
                  }
                }
              }

              //Win7: prevent system from mark program as hanged
              PeekMessageWide(&msg, hMainWnd, 0, 0, PM_NOREMOVE);
            }

            //End progress
            if (nCurrent >= aenp->nMaximum || bStop)
            {
              ShowWindow(hProgress, SW_HIDE);
              UpdateWindow(hStatus);
              dwProgressType=0;

              if (bStop)
              {
                PostMessage(hMainWnd, WM_COMMAND, IDM_FILE_EXIT, 0);
                *lResult=1;
                return TRUE;
              }

Типа если нажал ескейп, то смерть всем.
Если заменить IDM_FILE_CLOSE на IDM_WINDOW_FRAMECLOSE лучше не становится. Поскольку срабатывает дефолтный ускоритель.
Если дефолтный ускоритель также переставить на IDM_WINDOW_FRAMECLOSE, то сообщения буферизируются, а потом выполняются. Нажал 5 раз эск - 5 вкладок и закрылось.

Только если пристрелить ускоритель, тогда по эску в простое ничего не происходит, а при загрузке происходит отмена и вкладка закрывается.

Code: Select all

diff --git a/AkelFiles/Langs/Resources/English.rc b/AkelFiles/Langs/Resources/English.rc
index 35227d8..f92fabb 100644
--- a/AkelFiles/Langs/Resources/English.rc
+++ b/AkelFiles/Langs/Resources/English.rc
@@ -61,7 +61,7 @@ BEGIN
             MENUITEM "_",                           IDM_RECENT_FILES
         END
         MENUITEM SEPARATOR
-        MENUITEM "E&xit\tEsc",                  IDM_FILE_EXIT
+        MENUITEM "E&xit",                       IDM_FILE_EXIT
     END
     POPUP "&Edit"
     BEGIN
@@ -334,7 +334,7 @@ BEGIN
                                                     NOINVERT
     VK_BACK,        IDM_EDIT_DELETE_FIRST_CHAR, VIRTKEY, SHIFT, NOINVERT
     VK_DELETE,      IDM_EDIT_CUT,           VIRTKEY, SHIFT, NOINVERT
-    VK_ESCAPE,      IDM_FILE_EXIT,          VIRTKEY, NOINVERT
+//  VK_ESCAPE,      IDM_FILE_EXIT,          VIRTKEY, NOINVERT
     VK_F10,         IDM_SELECTWINDOW,       VIRTKEY, NOINVERT
     VK_F12,         IDM_FILE_CODEPAGEMENU,  VIRTKEY, NOINVERT
     VK_F2,          IDM_FILE_SAVE,          VIRTKEY, NOINVERT
diff --git a/AkelPad.c b/AkelPad.c
index c5a712b..7bd62d2 100644
--- a/AkelPad.c
+++ b/AkelPad.c
@@ -6193,7 +6193,7 @@ BOOL CALLBACK EditParentMessages(HWND hWnd, UINT uMsg, WPARAM wParam, LPARAM lPa
 
               if (bStop)
               {
-                PostMessage(hMainWnd, WM_COMMAND, IDM_FILE_EXIT, 0);
+                PostMessage(hMainWnd, WM_COMMAND, IDM_WINDOW_FRAMECLOSE, 0);
                 *lResult=1;
                 return TRUE;
               }

Это для русского:

Code: Select all

diff --git a/AkelFiles/Langs/Resources/Russian.rc b/AkelFiles/Langs/Resources/Russian.rc
index de76ddd..1eade4d 100644
--- a/AkelFiles/Langs/Resources/Russian.rc
+++ b/AkelFiles/Langs/Resources/Russian.rc
@@ -61,7 +61,7 @@ BEGIN
             MENUITEM "_",                           IDM_RECENT_FILES
         END
         MENUITEM SEPARATOR
-        MENUITEM "�&����\tEsc",                 IDM_FILE_EXIT
+        MENUITEM "�&����",                      IDM_FILE_EXIT
     END
     POPUP "&������"
     BEGIN
@@ -337,7 +337,7 @@ BEGIN
     VK_BACK,        IDM_EDIT_DELETE_FIRST_CHAR, VIRTKEY, SHIFT, NOINVERT
     VK_DELETE,      IDM_EDIT_CUT,           VIRTKEY, SHIFT, NOINVERT
     VK_MULTIPLY,    IDM_VIEW_FONTSIZE_RESTORE, VIRTKEY, CONTROL, NOINVERT
-    VK_ESCAPE,      IDM_FILE_EXIT,          VIRTKEY, NOINVERT
+//  VK_ESCAPE,      IDM_FILE_EXIT,          VIRTKEY, NOINVERT
     VK_F10,         IDM_SELECTWINDOW,       VIRTKEY, NOINVERT
     VK_F12,         IDM_FILE_CODEPAGEMENU,  VIRTKEY, NOINVERT
     VK_F2,          IDM_FILE_SAVE,          VIRTKEY, NOINVERT
@@ -481,12 +481,12 @@ END
 
 IDD_FIND DIALOGEX 70, 70, 257, 108
 STYLE DS_MODALFRAME | DS_3DLOOK | WS_POPUP | WS_VISIBLE | WS_CAPTION | 
-    WS_SYSMENU
+    WS_SYSMENU | WS_THICKFRAME
 EXSTYLE WS_EX_DLGMODALFRAME
 CAPTION "�����"
 FONT 8, "MS Shell Dlg", 0, 0, 0x1
 BEGIN
-    COMBOBOX        IDC_SEARCH_FIND,27,9,160,100,CBS_DROPDOWN | 
+    COMBOBOX        IDC_SEARCH_FIND,27,9,223,100,CBS_DROPDOWN | 
                     CBS_AUTOHSCROLL | WS_VSCROLL | WS_TABSTOP
     CONTROL         "&��������� �������",IDC_SEARCH_MATCHCASE,"Button",
                     BS_AUTOCHECKBOX | WS_TABSTOP,9,29,109,12
@@ -512,20 +512,20 @@ BEGIN
     CONTROL         "��� &�����",IDC_SEARCH_ALLFILES,"Button",
                     BS_AUTORADIOBUTTON,126,87,60,10
     GROUPBOX        "",IDC_SEARCH_ALLFILES_GROUP,121,80,66,19
-    DEFPUSHBUTTON   "&����� �����",IDC_SEARCH_FIND_BUTTON,196,6,54,14,
+    DEFPUSHBUTTON   "&����� �����",IDC_SEARCH_FIND_BUTTON,196,27,54,14,
                     BS_CENTER
-    PUSHBUTTON      "������",IDCANCEL,196,23,54,14,BS_CENTER
+    PUSHBUTTON      "������",IDCANCEL,196,44,54,14,BS_CENTER
     LTEXT           "���:",IDC_STATIC,5,11,22,12,NOT WS_GROUP
 END
 
 IDD_REPLACE DIALOGEX 70, 70, 257, 122
 STYLE DS_MODALFRAME | DS_3DLOOK | WS_POPUP | WS_VISIBLE | WS_CAPTION | 
-    WS_SYSMENU
+    WS_SYSMENU | WS_THICKFRAME
 EXSTYLE WS_EX_DLGMODALFRAME
 CAPTION "������"
 FONT 8, "MS Shell Dlg", 0, 0, 0x1
 BEGIN
-    COMBOBOX        IDC_SEARCH_FIND,27,9,160,100,CBS_DROPDOWN | 
+    COMBOBOX        IDC_SEARCH_FIND,27,9,223,100,CBS_DROPDOWN | 
                     CBS_AUTOHSCROLL | WS_VSCROLL | WS_TABSTOP
-    COMBOBOX        IDC_SEARCH_REPLACE,27,23,160,100,CBS_DROPDOWN | 
+    COMBOBOX        IDC_SEARCH_REPLACE,27,23,223,100,CBS_DROPDOWN | 
                     CBS_AUTOHSCROLL | WS_VSCROLL | WS_TABSTOP
@@ -553,13 +553,13 @@ BEGIN
     CONTROL         "��� &�����",IDC_SEARCH_ALLFILES,"Button",
                     BS_AUTORADIOBUTTON,126,101,60,10
     GROUPBOX        "",IDC_SEARCH_ALLFILES_GROUP,121,94,66,19
-    DEFPUSHBUTTON   "&����� �����",IDC_SEARCH_FIND_BUTTON,196,6,54,14,
+    DEFPUSHBUTTON   "&����� �����",IDC_SEARCH_FIND_BUTTON,196,41,54,14,
                     BS_CENTER
-    PUSHBUTTON      "&��������",IDC_SEARCH_REPLACE_BUTTON,196,23,54,14,
+    PUSHBUTTON      "&��������",IDC_SEARCH_REPLACE_BUTTON,196,58,54,14,
                     BS_CENTER
-    PUSHBUTTON      "�������� &��",IDC_SEARCH_ALL_BUTTON,196,39,54,14,
+    PUSHBUTTON      "�������� &��",IDC_SEARCH_ALL_BUTTON,196,74,54,14,
                     BS_CENTER
-    PUSHBUTTON      "������",IDCANCEL,196,55,54,14,BS_CENTER
+    PUSHBUTTON      "������",IDCANCEL,196,90,54,14,BS_CENTER
     LTEXT           "���:",IDC_STATIC,5,11,22,12,NOT WS_GROUP
     LTEXT           "���:",IDC_STATIC,5,25,22,12,NOT WS_GROUP
 END

Enyby · Post by **Enyby** » Sat Dec 22, 2018 10:59 pm

На версии 4.9.9 баг уже не воспроизводится.

Выгрузил [r4399] из СВН, собрал в 2008 студии (2005 у меня не собирает на Вин7), никакой разницы - как баг был так и остался. Все как на моем видео. Хотя теперь в эбаут боксе и написана заветная 4.9.9 версия.

Кстати оба патча накатил своих - проблем не возникло.