Could not compile KOL 2.10

← →
Dodfr (2005-07-12 01:30) [0]

Hi,

I get "not enought parameters" on a lot (may be all) of Rpt() calls within mck mirrors when I try to compile last 2.10 KOL.

In fact Rpt is called only with string parameter, no color

procedure Rpt( const S: String; Color: Integer );

Is that normal or did I miss something ?

← →
Dodfr (2005-07-12 01:41) [1]

I added WHITE parameter to all missing Rpt color values and it compile fine, I hope I choosed right color.

← →
Dodfr (2005-07-12 01:53) [2]

Now it"s MCKFakeclasses.icn that return "Undeclared identifier"

TKOLActionList = PActionList;

I already had this problem with 2.09 and I just copied the ActionList source from KOL 2.00 version.

← →
Dodfr (2005-07-12 02:41) [3]

Владимир Кладов said (sorry but this name is totally unreadable for me so I don"t who it is) that TACionList moved to KOLadd but so...what to do with MCKFakeClass.inc ? remove ActionList lines from it ? My copy/past from older kol.pas verison works fine but may not be best way.

← →
Владимир Кладов (2005-07-12 05:02) [4]

Don"t believe. Such error is for 2.09 (Delphi 2,3). In 2.10 all these are fixed.

← →
Dodfr (2005-07-12 12:17) [5]

OK, you trust what you see :-)

Rpt problems found in :

mckMenuEditor.inc lines 262, 296, 299, 303, 308, 311, 334, 338, 350, 353, 358
mckToolbarEditor.inc lines 720, 882, 896, 899, 904
mckLVColumnsEditor.inc lines 332, 347, 357, 360 ,365

← →
Dodfr (2005-07-12 13:51) [6]

I forgot mckActionListEditor.inc

← →
avs © (2005-07-12 21:03) [7]

К сожадению, у меня то же самое... Rpt( и т.д.
Delhi 7,WinXp Sp2

← →
Dodfr (2005-07-12 21:39) [8]

sorry avs I don"t read russian.

← →
ECM © (2005-07-13 11:30) [9]

Dodfr (12.07.05 02:41) [3]
TACionList moved to KOLadd but so...what to do with MCKFakeClass.inc ?
...it`s a bug in 2.10 (disapper in sequential upgrade 2.0x -> 2.07 -> 2.10)
just comment following line in MCKfakeClasses.inc:
...
//! TKOLActionList = PActionList;
//! TKOLAction = PAction;

← →
Dodfr (2005-07-13 12:43) [10]

yes, Finally that is what I simply did.

Strange that "complete" 2.10 package have bug and sequential update do not...

← →
ECM © (2005-07-13 13:02) [11]

Dodfr (13.07.05 12:43) [10]
...
Strange that "complete" 2.10 package have bug and sequential update do not...

Vladimir lost new version MCKfakeClasses.inc in versions > 2.07

← →
fellix (2005-07-13 13:39) [12]

> Dodfr
> ECM

Another way (w/o patching MCKfakeClasses.inc) is to include KOLadd in uses clause in every project"s unit.
Example:
{$IFDEF KOL_MCK} uses Windows, Messages, ShellAPI, KOL {$IFNDEF KOL_MCK}, mirror, Classes {$ENDIF}, KOLadd; {$ELSE} {$I uses.inc} ; {$ENDIF}

← →
ECM © (2005-07-13 13:53) [13]

2fellix (13.07.05 13:39) [12]:
", KOLadd" ( 8 chars) * N (units)
vs
"//" * 2 = 4 chars in MCKFakeClasses.inc
??? K.I.S.S

← →
ECM © (2005-07-13 13:54) [14]

P.S.
( 8 chars) * N (units) * M (projects)

← →
fellix (2005-07-13 14:13) [15]

>ECM
Yes, agree, that"s ugly. Truly speaking, I also always patch MCKFakeClasses.inc. That"s just another way.
The best method is to do it automatically. As for me, Vladimir Kladov, extracting KOLAdd.pas from KOL.pas, had to change
"uses Windows, Messages, ShellAPI, KOL"
to
"uses Windows, Messages, ShellAPI, KOL, KOLAdd"
in mirror.pas/TKOLForm.GeneratePAS function.

← →
ECM © (2005-07-13 15:07) [16]

2fellix (13.07.05 14:13) [15]
The best method is to do it automatically

Yes... But not in all MCK-projects KOLadd.pas it is really necessary. It should be added automatically only for projects with KOLAction(List).It is already realized in a mirror for KOLActionList. With use of function AdditionalUnits

see http://delphimaster.net/view/11-1110119068/ post [49]

← →
fellix (2005-07-13 19:40) [17]

> ECM © (13.07.05 15:07) [16]

KOL ideology is "if something isn"t used, the code isn"t generated". Even if declared. (Actually, including KOLAdd into "uses" clause adds around 70 bytes to the code.)
If you require ActionList, you need to "unpatch" MCKFakeClasses.inc back. So adding "KOLAdd" into every unit containing KOLForm is more universal decision.

← →
Dodfr (2005-07-13 20:17) [18]

talking about bytes, nobody noticed increase of it"s EXE size from 2.09 to 2.10 ? I tested by recompiling in 2.09 and back again to 2.10 and 2.10 produce 30KB bigger EXE.

← →
fellix (2005-07-13 20:22) [19]

>Dodfr
Nothing mentioned noticed on my projects. Try to analyze detailed map file.

← →
Dodfr (2005-07-13 20:41) [20]

I checked the RCDATA PACKAGEINFO resource from both EXE but they import same units.

← →
Sniper17 (2005-07-13 20:57) [21]

так как в итоге скомпилить KOL 2.10?

← →
fellix (2005-07-13 20:58) [22]

Forget it and check the map. :-)
You can find size of every module used.

← →
Dodfr (2005-07-13 21:50) [23]

sorry sniper17 I can"t read russian

← →
Dodfr (2005-07-14 00:45) [24]

fellix, which tool to analyze detailed map ?

← →
fellix (2005-07-14 01:48) [25]

>Dodfr

Try to do this manually. :-)
Set Project -> Options -> Linker -> Detailed and compile the project both with KOL 2.09 and 2.10. Of course, ensure that other options are identical. Then compare obtained *.map files manually or using any text file comparer, e.g., standard fc.exe or DiffLines ( http://bonanzas.rinet.ru/DiffLines.zip ).

← →
Sniper17 (2005-07-14 11:02) [26]

Dodfr:
How can I compile KOL2.10 then? %-)

← →
Dodfr (2005-07-14 13:54) [27]

Sniper17 : if you have same problem than me, just // the rpt() lines and also // the two actionlist in mckFakeCtrls.inc

← →
Barloggg (2005-07-18 13:55) [28]

well... a just set comment to every line with this error... no more RPT...
it"s wrong?
---------in RU
Я закоментировал все строчки с такой ошибкой и всего делов.
Хотя лучше добавлю коладд в усес. или дернуть из 2.07 этот инк файл?

2Barloggg Если дернуть из 2.07, то это ни к чему не приведет.

Лучше поправить объявление функции Rpt так:
rpt(S:String; color:TColor = WHITE);
{это если у вас Delphi>=4}

Сама функция в модуле mirror.pas.

2RA вопрос не о Rpt... надо взять mckFake... из 2.07
а по поводу Rpt вот ответ Кладова из переписки со мной

epn> Здравствуйте Владимир!

epn> По-моему после доработки Rpt(...,Color) оказались пропущены исправления для
epn> файлов mckXxxEditor.inc....
epn> это проявляется при переcборке главного MCK-пакета...

epn> D7

epn> ECM mailto: ecm@ydk.com.ua

да, я уже заметил. Надо самому поправить или в процедуре Rpt сделать
второй параметр по умолчанию = WHITE как раньше, или во всех этих
.inc-файлах добавить , WHITE

Could not compile KOL 2.10 Найти похожие ветки