Замечания и предложения

Среда PascalABC.NET
#1207

Спасибо, я читал, что делает эта директива. Но это не ответ на мои вопросы: зачем, когда и как часто нужно?

#1208

Нужно любому, кто делает крупный проект. Пустая строчка с описанием задевает мои чувства перфекциониста).

image
image1920×762 72.3 KB

Болталка PascalABC.NET
#1209

Понятно. Я считаю, что реагировать на чувства отдельно взятых перфекционистов - исключительно прерогатива разработчиков, поэтому от дальнейших комментариев и вопросов воздержусь. Замечу только, что разработчики уже не раз отмечали: PascalABC.NЕТ предназначен для небольших и среднего размера проектов.

#1210

Зато подобная практика помогает находить баги среды.

#1211

Ну это у Вас уже сродни бородатейшему советскому анекдоту:

  • А почему у вас в СССР в магазинах мяса нет?
  • Зато у Вас в Америке негров линчуют!
1 лайк
#1212

Предлагаю запретить создавать проекты Windows Forms, поскольку:

  1. Они уже не поддерживаются разработчиками.
  2. Содержат много багов, связанных, например, с переименованием и анализатором кода.
#1213

Переименование не относится к Windows Forms. Переименование относится к переименованию.

Анализатор кода не относится к Windows Forms. Он относится к средствам Intellisense. Особых багов я там не знаю - может, Sun Serega их напишет.

#1214

Переименование там ломается часто. Я об этом.

#1215

Да, я понял. Дизайнер форм в этом не виноват. Виновато переименование.

#1216

Был бы дизайнер для WPF-приложений… :slight_smile:

#1217

Да, я б с удовольствием поручил его кому-то интегрировать.

3 лайка
#1218

Директива {$description ‘’}
Короче, всё, что нужно сделать:
в TreeConverter\TreeConversion\compiler_string_consts.cs в 340-ую строку добавить

        public static string description_string = "description";

в Compiler\Compiler.cs в строку 1922 добавить:

                if (compilerDirectives.TryGetValue(TreeConverter.compiler_string_consts.description_string, out cds))
                {
                    cdo.Description = cds[0].directive;
                }

в 1986-ую:

                    if (!string.IsNullOrEmpty(project.description))
                        cdo.Description = project.description;

в 2012-ой надо заменить sw.WriteLine("VALUE \"FileDescription\"," + "\"" + "" + "\""); на sw.WriteLine("VALUE \"FileDescription\"," + "\"" + cdo.Description + "\""); В Compiler\ProjectInfo.cs в 33-ю строку:

        private string _description;

в 194-ой организовать свойство по тому полю:

        public string description
        {
            get
            {
                return _description;
            }
            set
            {
                _description = value;
            }
        }

в 483-ю добавить:

            writer.WriteAttributeString("Description", description);

и в 549-ую:

            description = reader.GetAttribute("Description");
            if (description == null) description = "";

в NETGenerator\NETGenerator.cs в 89-ую:

        private string _Description = "";

        public string Description
        {
            get { return _Description; }
            set { _Description = value; NeedDefineVersionInfo = true; }
        }

и последнее – необходимо как-то записать описание в сборку, с этим я не разобрался.
Вроде это всё, что требуется.

#1219

А почему не пулл реквестом?

#1220

Потому что гитхаб “хорошо” показывает изменения в файлах:

Да и я так и не разобрался как впилить описание в итоговую сборку.

#1221

Да-да. А потом что-то где-то вылезет в другом месте и разработчики будут еще полгода выковыривать баги на основе полсотни issure, которые им радостно нарисуют добрые люди. Зато будет удовлетворена сомнительная хотелка.

1 лайк
#1222

@RAlex, я бывал часто с Вами не согласен в чём-либо, но сейчас поддерживаю Вашу позицию.

#1223

Потому что не против твоей хотелки сейчас высказывания летят.

#1224

Я всё готов протестировать. Вряд ли добавление описания процессу может повлечь за собой какие-то критические ошибки.

#1225

Это хорошо, что готовы. Тогда тестируйте. Только, разберитесь с тем, с чем ещё не разобрались.

#1226

Пока это все - “сотрясания воздуха”. Разработчики по Вашему предложению не отписались, а тут главное - это их мнение.

1 лайк