tcuseanalyzer.pas 36 KB

12345678910111213141516171819202122232425262728293031323334353637383940414243444546474849505152535455565758596061626364656667686970717273747576777879808182838485868788899091929394959697989910010110210310410510610710810911011111211311411511611711811912012112212312412512612712812913013113213313413513613713813914014114214314414514614714814915015115215315415515615715815916016116216316416516616716816917017117217317417517617717817918018118218318418518618718818919019119219319419519619719819920020120220320420520620720820921021121221321421521621721821922022122222322422522622722822923023123223323423523623723823924024124224324424524624724824925025125225325425525625725825926026126226326426526626726826927027127227327427527627727827928028128228328428528628728828929029129229329429529629729829930030130230330430530630730830931031131231331431531631731831932032132232332432532632732832933033133233333433533633733833934034134234334434534634734834935035135235335435535635735835936036136236336436536636736836937037137237337437537637737837938038138238338438538638738838939039139239339439539639739839940040140240340440540640740840941041141241341441541641741841942042142242342442542642742842943043143243343443543643743843944044144244344444544644744844945045145245345445545645745845946046146246346446546646746846947047147247347447547647747847948048148248348448548648748848949049149249349449549649749849950050150250350450550650750850951051151251351451551651751851952052152252352452552652752852953053153253353453553653753853954054154254354454554654754854955055155255355455555655755855956056156256356456556656756856957057157257357457557657757857958058158258358458558658758858959059159259359459559659759859960060160260360460560660760860961061161261361461561661761861962062162262362462562662762862963063163263363463563663763863964064164264364464564664764864965065165265365465565665765865966066166266366466566666766866967067167267367467567667767867968068168268368468568668768868969069169269369469569669769869970070170270370470570670770870971071171271371471571671771871972072172272372472572672772872973073173273373473573673773873974074174274374474574674774874975075175275375475575675775875976076176276376476576676776876977077177277377477577677777877978078178278378478578678778878979079179279379479579679779879980080180280380480580680780880981081181281381481581681781881982082182282382482582682782882983083183283383483583683783883984084184284384484584684784884985085185285385485585685785885986086186286386486586686786886987087187287387487587687787887988088188288388488588688788888989089189289389489589689789889990090190290390490590690790890991091191291391491591691791891992092192292392492592692792892993093193293393493593693793893994094194294394494594694794894995095195295395495595695795895996096196296396496596696796896997097197297397497597697797897998098198298398498598698798898999099199299399499599699799899910001001100210031004100510061007100810091010101110121013101410151016101710181019102010211022102310241025102610271028102910301031103210331034103510361037103810391040104110421043104410451046104710481049105010511052105310541055105610571058105910601061106210631064106510661067106810691070107110721073107410751076107710781079108010811082108310841085108610871088108910901091109210931094109510961097109810991100110111021103110411051106110711081109111011111112111311141115111611171118111911201121112211231124112511261127112811291130113111321133113411351136113711381139114011411142114311441145114611471148114911501151115211531154115511561157115811591160116111621163116411651166116711681169117011711172117311741175117611771178117911801181118211831184118511861187118811891190119111921193119411951196119711981199120012011202120312041205120612071208120912101211121212131214121512161217121812191220122112221223122412251226122712281229123012311232123312341235123612371238123912401241124212431244124512461247124812491250125112521253125412551256125712581259126012611262126312641265126612671268126912701271127212731274127512761277127812791280128112821283128412851286128712881289129012911292129312941295129612971298129913001301130213031304130513061307130813091310131113121313131413151316131713181319132013211322132313241325132613271328132913301331133213331334133513361337133813391340134113421343134413451346134713481349135013511352135313541355135613571358135913601361136213631364136513661367136813691370137113721373137413751376137713781379138013811382138313841385138613871388138913901391139213931394139513961397139813991400140114021403140414051406140714081409
  1. {
  2. Examples:
  3. ./testpassrc --suite=TTestResolver.TestEmpty
  4. }
  5. unit tcuseanalyzer;
  6. {$mode objfpc}{$H+}
  7. interface
  8. uses
  9. Classes, SysUtils, fpcunit,
  10. PasTree, PScanner, PasResolver,
  11. tcbaseparser, testregistry, strutils, tcresolver, PasUseAnalyzer;
  12. type
  13. { TCustomTestUseAnalyzer }
  14. TCustomTestUseAnalyzer = Class(TCustomTestResolver)
  15. private
  16. FAnalyzer: TPasAnalyzer;
  17. FPAMessages: TFPList; // list of TPAMessage
  18. function GetPAMessages(Index: integer): TPAMessage;
  19. procedure OnAnalyzerMessage(Sender: TObject; Msg: TPAMessage);
  20. protected
  21. procedure SetUp; override;
  22. procedure TearDown; override;
  23. procedure AnalyzeModule; virtual;
  24. procedure AnalyzeProgram; virtual;
  25. procedure AnalyzeUnit; virtual;
  26. procedure AnalyzeWholeProgram; virtual;
  27. procedure CheckUsedMarkers; virtual;
  28. procedure CheckHasHint(MsgType: TMessageType; MsgNumber: integer;
  29. const MsgText: string; Has: boolean = true); virtual;
  30. procedure CheckUnitUsed(const aFilename: string; Used: boolean);
  31. public
  32. property Analyzer: TPasAnalyzer read FAnalyzer;
  33. function PAMessageCount: integer;
  34. property PAMessages[Index: integer]: TPAMessage read GetPAMessages;
  35. end;
  36. { TTestUseAnalyzer }
  37. TTestUseAnalyzer = Class(TCustomTestUseAnalyzer)
  38. published
  39. // single module
  40. procedure TestM_ProgramLocalVar;
  41. procedure TestM_AssignStatement;
  42. procedure TestM_BeginBlock;
  43. procedure TestM_ForLoopStatement;
  44. procedure TestM_AsmStatement;
  45. procedure TestM_CaseOfStatement;
  46. procedure TestM_IfThenElseStatement;
  47. procedure TestM_WhileDoStatement;
  48. procedure TestM_RepeatUntilStatement;
  49. procedure TestM_TryFinallyStatement;
  50. procedure TestM_TypeAlias;
  51. procedure TestM_Unary;
  52. procedure TestM_Const;
  53. procedure TestM_Record;
  54. procedure TestM_Array;
  55. procedure TestM_NestedFuncResult;
  56. procedure TestM_Enums;
  57. procedure TestM_ProcedureType;
  58. procedure TestM_Params;
  59. procedure TestM_Class;
  60. procedure TestM_ClassForward;
  61. procedure TestM_Class_Property;
  62. procedure TestM_Class_PropertyOverride;
  63. procedure TestM_Class_MethodOverride;
  64. procedure TestM_Class_MethodOverride2;
  65. procedure TestM_TryExceptStatement;
  66. // single module hints
  67. procedure TestM_Hint_UnitNotUsed;
  68. procedure TestM_Hint_UnitNotUsed_No_OnlyExternal;
  69. procedure TestM_Hint_ParameterNotUsed;
  70. procedure TestM_Hint_ParameterNotUsed_Abstract;
  71. procedure TestM_Hint_LocalVariableNotUsed;
  72. procedure TestM_Hint_InterfaceUnitVariableUsed;
  73. procedure TestM_Hint_ValueParameterIsAssignedButNeverUsed;
  74. procedure TestM_Hint_LocalVariableIsAssignedButNeverUsed;
  75. procedure TestM_Hint_LocalXYNotUsed;
  76. procedure TestM_Hint_PrivateFieldIsNeverUsed;
  77. procedure TestM_Hint_PrivateFieldIsAssignedButNeverUsed;
  78. procedure TestM_Hint_PrivateMethodIsNeverUsed;
  79. procedure TestM_Hint_PrivateTypeNeverUsed;
  80. procedure TestM_Hint_PrivateConstNeverUsed;
  81. procedure TestM_Hint_PrivatePropertyNeverUsed;
  82. procedure TestM_Hint_LocalClassInProgramNotUsed;
  83. procedure TestM_Hint_LocalMethodInProgramNotUsed;
  84. procedure TestM_Hint_AssemblerParameterIgnored;
  85. procedure TestM_Hint_FunctionResultDoesNotSeemToBeSet;
  86. procedure TestM_Hint_FunctionResultDoesNotSeemToBeSet_Abstract;
  87. procedure TestM_Hint_FunctionResultRecord;
  88. procedure TestM_Hint_FunctionResultPassRecordElement;
  89. procedure TestM_Hint_OutParam_No_AssignedButNeverUsed;
  90. // whole program optimization
  91. procedure TestWP_LocalVar;
  92. procedure TestWP_UnitUsed;
  93. procedure TestWP_UnitNotUsed;
  94. procedure TestWP_UnitInitialization;
  95. procedure TestWP_UnitFinalization;
  96. procedure TestWP_CallInherited;
  97. procedure TestWP_ProgramPublicDeclarations;
  98. procedure TestWP_ClassDefaultProperty;
  99. end;
  100. implementation
  101. { TCustomTestUseAnalyzer }
  102. procedure TCustomTestUseAnalyzer.OnAnalyzerMessage(Sender: TObject;
  103. Msg: TPAMessage);
  104. begin
  105. Msg.AddRef;
  106. FPAMessages.Add(Msg);
  107. end;
  108. function TCustomTestUseAnalyzer.GetPAMessages(Index: integer): TPAMessage;
  109. begin
  110. Result:=TPAMessage(FPAMessages[Index]);
  111. end;
  112. procedure TCustomTestUseAnalyzer.SetUp;
  113. begin
  114. inherited SetUp;
  115. FPAMessages:=TFPList.Create;
  116. FAnalyzer:=TPasAnalyzer.Create;
  117. FAnalyzer.Resolver:=ResolverEngine;
  118. Analyzer.OnMessage:=@OnAnalyzerMessage;
  119. end;
  120. procedure TCustomTestUseAnalyzer.TearDown;
  121. var
  122. i: Integer;
  123. begin
  124. for i:=0 to FPAMessages.Count-1 do
  125. TPAMessage(FPAMessages[i]).Release;
  126. FreeAndNil(FPAMessages);
  127. FreeAndNil(FAnalyzer);
  128. inherited TearDown;
  129. end;
  130. procedure TCustomTestUseAnalyzer.AnalyzeModule;
  131. begin
  132. Analyzer.AnalyzeModule(Module);
  133. Analyzer.EmitModuleHints(Module);
  134. CheckUsedMarkers;
  135. end;
  136. procedure TCustomTestUseAnalyzer.AnalyzeProgram;
  137. begin
  138. ParseProgram;
  139. AnalyzeModule;
  140. end;
  141. procedure TCustomTestUseAnalyzer.AnalyzeUnit;
  142. begin
  143. ParseUnit;
  144. AnalyzeModule;
  145. end;
  146. procedure TCustomTestUseAnalyzer.AnalyzeWholeProgram;
  147. begin
  148. ParseProgram;
  149. Analyzer.AnalyzeWholeProgram(Module as TPasProgram);
  150. CheckUsedMarkers;
  151. end;
  152. procedure TCustomTestUseAnalyzer.CheckUsedMarkers;
  153. var
  154. aMarker: PSrcMarker;
  155. p: SizeInt;
  156. Postfix: String;
  157. Elements: TFPList;
  158. i: Integer;
  159. El: TPasElement;
  160. ExpectedUsed: Boolean;
  161. FoundEl: TPAElement;
  162. begin
  163. aMarker:=FirstSrcMarker;
  164. while aMarker<>nil do
  165. begin
  166. writeln('TCustomTestUseAnalyzer.CheckUsedMarkers ',aMarker^.Identifier,' Line=',aMarker^.Row,' StartCol=',aMarker^.StartCol,' EndCol=',aMarker^.EndCol);
  167. p:=RPos('_',aMarker^.Identifier);
  168. if p>1 then
  169. begin
  170. Postfix:=copy(aMarker^.Identifier,p+1);
  171. if Postfix='used' then
  172. ExpectedUsed:=true
  173. else if Postfix='notused' then
  174. ExpectedUsed:=false
  175. else
  176. RaiseErrorAtSrcMarker('TCustomTestUseAnalyzer.CheckUsedMarkers unknown postfix "'+Postfix+'"',aMarker);
  177. Elements:=FindElementsAt(aMarker);
  178. try
  179. FoundEl:=nil;
  180. for i:=0 to Elements.Count-1 do
  181. begin
  182. El:=TPasElement(Elements[i]);
  183. writeln('TCustomTestUseAnalyzer.CheckUsedMarkers ',aMarker^.Identifier,' ',i,'/',Elements.Count,' El=',GetObjName(El),' ',GetObjName(El.CustomData));
  184. FoundEl:=Analyzer.FindElement(El);
  185. if FoundEl<>nil then break;
  186. end;
  187. if FoundEl<>nil then
  188. begin
  189. if not ExpectedUsed then
  190. RaiseErrorAtSrcMarker('expected element to be *not* used, but it is marked',aMarker);
  191. end
  192. else
  193. begin
  194. if ExpectedUsed then
  195. RaiseErrorAtSrcMarker('expected element to be used, but it is not marked',aMarker);
  196. end;
  197. finally
  198. Elements.Free;
  199. end;
  200. end;
  201. aMarker:=aMarker^.Next;
  202. end;
  203. end;
  204. procedure TCustomTestUseAnalyzer.CheckHasHint(MsgType: TMessageType;
  205. MsgNumber: integer; const MsgText: string; Has: boolean);
  206. var
  207. i: Integer;
  208. Msg: TPAMessage;
  209. s: string;
  210. begin
  211. i:=PAMessageCount-1;
  212. while i>=0 do
  213. begin
  214. Msg:=PAMessages[i];
  215. if (Msg.MsgNumber=MsgNumber) then
  216. begin
  217. if Has then
  218. begin
  219. // must have -> message type and text must match exactly
  220. if (Msg.MsgType=MsgType) and (Msg.MsgText=MsgText) then
  221. exit;
  222. end
  223. else
  224. begin
  225. // must not have -> matching number is enough
  226. break;
  227. end;
  228. end;
  229. dec(i);
  230. end;
  231. if (not Has) and (i<0) then exit;
  232. // mismatch
  233. writeln('TCustomTestUseAnalyzer.CheckHasHint: ');
  234. for i:=0 to PAMessageCount-1 do
  235. begin
  236. Msg:=PAMessages[i];
  237. writeln(' ',i,'/',PAMessageCount,': [',Msg.Id,'] ',Msg.MsgType,': (',Msg.MsgNumber,') {',Msg.MsgText,'}');
  238. end;
  239. s:='';
  240. str(MsgType,s);
  241. Fail('Analyzer Message '+BoolToStr(Has,'not ','')+'found: '+s+': ('+IntToStr(MsgNumber)+') {'+MsgText+'}');
  242. end;
  243. procedure TCustomTestUseAnalyzer.CheckUnitUsed(const aFilename: string;
  244. Used: boolean);
  245. var
  246. aResolver: TTestEnginePasResolver;
  247. PAEl: TPAElement;
  248. begin
  249. aResolver:=FindModuleWithFilename(aFilename);
  250. AssertNotNull('unit not found "'+aFilename+'"',aResolver);
  251. AssertNotNull('unit module not found "'+aFilename+'"',aResolver.Module);
  252. PAEl:=Analyzer.FindElement(aResolver.Module);
  253. if PAEl<>nil then
  254. begin
  255. // unit is used
  256. if not Used then
  257. Fail('expected unit "'+aFilename+'" not used, but it is used');
  258. end
  259. else
  260. begin
  261. // unit is not used
  262. if Used then
  263. Fail('expected unit "'+aFilename+'" used, but it is not used');
  264. end;
  265. end;
  266. function TCustomTestUseAnalyzer.PAMessageCount: integer;
  267. begin
  268. Result:=FPAMessages.Count;
  269. end;
  270. { TTestUseAnalyzer }
  271. procedure TTestUseAnalyzer.TestM_ProgramLocalVar;
  272. begin
  273. StartProgram(false);
  274. Add('procedure {#DoIt_used}DoIt;');
  275. Add('var {#l_notused}l: longint;');
  276. Add('begin');
  277. Add('end;');
  278. Add('begin');
  279. Add(' DoIt;');
  280. AnalyzeProgram;
  281. end;
  282. procedure TTestUseAnalyzer.TestM_AssignStatement;
  283. begin
  284. StartProgram(false);
  285. Add('procedure {#DoIt_used}DoIt;');
  286. Add('var');
  287. Add(' {#a_notused}a: longint;');
  288. Add(' {#b_used}b: longint;');
  289. Add(' {#c_used}c: longint;');
  290. Add('begin');
  291. Add(' b:=c;');
  292. Add('end;');
  293. Add('begin');
  294. Add(' DoIt;');
  295. AnalyzeProgram;
  296. end;
  297. procedure TTestUseAnalyzer.TestM_BeginBlock;
  298. begin
  299. StartProgram(false);
  300. Add('procedure {#DoIt_used}DoIt;');
  301. Add('var');
  302. Add(' {#a_used}a: longint;');
  303. Add('begin');
  304. Add(' begin');
  305. Add(' a:=1;');
  306. Add(' end;');
  307. Add('end;');
  308. Add('begin');
  309. Add(' DoIt;');
  310. AnalyzeProgram;
  311. end;
  312. procedure TTestUseAnalyzer.TestM_ForLoopStatement;
  313. begin
  314. StartProgram(false);
  315. Add('procedure {#DoIt_used}DoIt;');
  316. Add('var');
  317. Add(' {#a_used}a: longint;');
  318. Add(' {#b_used}b: longint;');
  319. Add(' {#c_used}c: longint;');
  320. Add(' {#d_used}d: longint;');
  321. Add('begin');
  322. Add(' for a:=b to c do d:=a;');
  323. Add('end;');
  324. Add('begin');
  325. Add(' DoIt;');
  326. AnalyzeProgram;
  327. end;
  328. procedure TTestUseAnalyzer.TestM_AsmStatement;
  329. begin
  330. StartProgram(false);
  331. Add('procedure {#DoIt_used}DoIt;');
  332. Add('begin');
  333. Add(' asm end;');
  334. Add('end;');
  335. Add('begin');
  336. Add(' DoIt;');
  337. AnalyzeProgram;
  338. end;
  339. procedure TTestUseAnalyzer.TestM_CaseOfStatement;
  340. begin
  341. StartProgram(false);
  342. Add('procedure {#DoIt_used}DoIt;');
  343. Add('const');
  344. Add(' {#a_used}a = 1;');
  345. Add(' {#b_used}b = 2;');
  346. Add('var');
  347. Add(' {#c_used}c: longint;');
  348. Add(' {#d_used}d: longint;');
  349. Add('begin');
  350. Add(' case a of');
  351. Add(' b: c:=1;');
  352. Add(' else');
  353. Add(' d:=2;');
  354. Add(' end;');
  355. Add('end;');
  356. Add('begin');
  357. Add(' DoIt;');
  358. AnalyzeProgram;
  359. end;
  360. procedure TTestUseAnalyzer.TestM_IfThenElseStatement;
  361. begin
  362. StartProgram(false);
  363. Add('procedure {#DoIt_used}DoIt;');
  364. Add('var');
  365. Add(' {#a_used}a: longint;');
  366. Add(' {#b_used}b: longint;');
  367. Add(' {#c_used}c: longint;');
  368. Add('begin');
  369. Add(' if a=0 then b:=1 else c:=2;');
  370. Add('end;');
  371. Add('begin');
  372. Add(' DoIt;');
  373. AnalyzeProgram;
  374. end;
  375. procedure TTestUseAnalyzer.TestM_WhileDoStatement;
  376. begin
  377. StartProgram(false);
  378. Add('procedure {#DoIt_used}DoIt;');
  379. Add('var');
  380. Add(' {#a_used}a: longint;');
  381. Add(' {#b_used}b: longint;');
  382. Add('begin');
  383. Add(' while a>0 do b:=1;');
  384. Add('end;');
  385. Add('begin');
  386. Add(' DoIt;');
  387. AnalyzeProgram;
  388. end;
  389. procedure TTestUseAnalyzer.TestM_RepeatUntilStatement;
  390. begin
  391. StartProgram(false);
  392. Add('procedure {#DoIt_used}DoIt;');
  393. Add('var');
  394. Add(' {#a_used}a: longint;');
  395. Add(' {#b_used}b: longint;');
  396. Add('begin');
  397. Add(' repeat a:=1; until b>1;');
  398. Add('end;');
  399. Add('begin');
  400. Add(' DoIt;');
  401. AnalyzeProgram;
  402. end;
  403. procedure TTestUseAnalyzer.TestM_TryFinallyStatement;
  404. begin
  405. StartProgram(false);
  406. Add('procedure {#DoIt_used}DoIt;');
  407. Add('var');
  408. Add(' {#a_used}a: longint;');
  409. Add(' {#b_used}b: longint;');
  410. Add('begin');
  411. Add(' try');
  412. Add(' a:=1;');
  413. Add(' finally');
  414. Add(' b:=2;');
  415. Add(' end;');
  416. Add('end;');
  417. Add('begin');
  418. Add(' DoIt;');
  419. AnalyzeProgram;
  420. end;
  421. procedure TTestUseAnalyzer.TestM_TypeAlias;
  422. begin
  423. StartProgram(false);
  424. Add('procedure {#DoIt_used}DoIt;');
  425. Add('type');
  426. Add(' {#integer_used}integer = longint;');
  427. Add('var');
  428. Add(' {#a_used}a: integer;');
  429. Add(' {#b_used}b: integer;');
  430. Add(' {#c_notused}c: integer;');
  431. Add('begin');
  432. Add(' a:=b;');
  433. Add('end;');
  434. Add('begin');
  435. Add(' DoIt;');
  436. AnalyzeProgram;
  437. end;
  438. procedure TTestUseAnalyzer.TestM_Unary;
  439. begin
  440. StartProgram(false);
  441. Add('procedure {#DoIt_used}DoIt;');
  442. Add('var');
  443. Add(' {#a_used}a: longint;');
  444. Add(' {#b_used}b: longint;');
  445. Add(' {#c_used}c: longint;');
  446. Add(' {#d_used}d: longint;');
  447. Add('begin');
  448. Add(' a:=+b;');
  449. Add(' a:=c+d;');
  450. Add('end;');
  451. Add('begin');
  452. Add(' DoIt;');
  453. AnalyzeProgram;
  454. end;
  455. procedure TTestUseAnalyzer.TestM_Const;
  456. begin
  457. StartProgram(false);
  458. Add('procedure {#DoIt_used}DoIt;');
  459. Add('var');
  460. Add(' {#a_used}a: longint;');
  461. Add(' {#b_used}b: boolean;');
  462. Add(' {#c_used}c: array of longint;');
  463. Add(' {#d_used}d: string;');
  464. Add('begin');
  465. Add(' a:=+1;');
  466. Add(' b:=true;');
  467. Add(' c:=nil;');
  468. Add(' d:=''foo'';');
  469. Add('end;');
  470. Add('begin');
  471. Add(' DoIt;');
  472. AnalyzeProgram;
  473. end;
  474. procedure TTestUseAnalyzer.TestM_Record;
  475. begin
  476. StartProgram(false);
  477. Add('procedure {#DoIt_used}DoIt;');
  478. Add('type');
  479. Add(' {#integer_used}integer = longint;');
  480. Add(' {#trec_used}TRec = record');
  481. Add(' {#a_used}a: integer;');
  482. Add(' {#b_notused}b: integer;');
  483. Add(' {#c_used}c: integer;');
  484. Add(' end;');
  485. Add('var');
  486. Add(' {#r_used}r: TRec;');
  487. Add('begin');
  488. Add(' r.a:=3;');
  489. Add(' with r do c:=4;');
  490. Add('end;');
  491. Add('begin');
  492. Add(' DoIt;');
  493. AnalyzeProgram;
  494. end;
  495. procedure TTestUseAnalyzer.TestM_Array;
  496. begin
  497. StartProgram(false);
  498. Add('procedure {#DoIt_used}DoIt;');
  499. Add('type');
  500. Add(' {#integer_used}integer = longint;');
  501. Add(' {#tarrayint_used}TArrayInt = array of integer;');
  502. Add('var');
  503. Add(' {#a_used}a: TArrayInt;');
  504. Add(' {#b_used}b: integer;');
  505. Add(' {#c_used}c: TArrayInt;');
  506. Add(' {#d_used}d: integer;');
  507. Add(' {#e_used}e: TArrayInt;');
  508. Add(' {#f_used}f: integer;');
  509. Add(' {#g_used}g: TArrayInt;');
  510. Add(' {#h_used}h: TArrayInt;');
  511. Add(' {#i_used}i: TArrayInt;');
  512. Add('begin');
  513. Add(' a[b]:=c[d];');
  514. Add(' SetLength(e,f)');
  515. Add(' if low(g)=high(h)+length(i) then');
  516. Add('end;');
  517. Add('begin');
  518. Add(' DoIt;');
  519. AnalyzeProgram;
  520. end;
  521. procedure TTestUseAnalyzer.TestM_NestedFuncResult;
  522. begin
  523. StartProgram(false);
  524. Add('procedure {#DoIt_used}DoIt;');
  525. Add('type');
  526. Add(' {#integer_used}integer = longint;');
  527. Add(' {#tarrayint_used}TArrayInt = array of integer;');
  528. Add(' function {#nestedfunc_used}NestedFunc({#b_notused}b: longint): TArrayInt;');
  529. Add(' begin');
  530. Add(' end;');
  531. Add('var');
  532. Add(' {#d_used}d: longint;');
  533. Add('begin');
  534. Add(' NestedFunc(d);');
  535. Add('end;');
  536. Add('begin');
  537. Add(' DoIt;');
  538. AnalyzeProgram;
  539. end;
  540. procedure TTestUseAnalyzer.TestM_Enums;
  541. begin
  542. StartProgram(false);
  543. Add('procedure {#DoIt_used}DoIt(const o);');
  544. Add('type');
  545. Add(' {#TEnum_used}TEnum = (red,blue);');
  546. Add(' {#TEnums_used}TEnums = set of TEnum;');
  547. Add('var');
  548. Add(' {#a_used}a: TEnum;');
  549. Add(' {#b_used}b: TEnums;');
  550. Add(' {#c_used}c: TEnum;');
  551. Add(' {#d_used}d: TEnums;');
  552. Add(' {#e_used}e: TEnums;');
  553. Add(' {#f_used}f: TEnums;');
  554. Add(' {#g_used}g: TEnum;');
  555. Add(' {#h_used}h: TEnum;');
  556. Add('begin');
  557. Add(' b:=[a];');
  558. Add(' if c in d then;');
  559. Add(' if low(e)=high(f) then;');
  560. Add(' if pred(g)=succ(h) then;');
  561. Add('end;');
  562. Add('var {#s_used}s: string;');
  563. Add('begin');
  564. Add(' DoIt(s);');
  565. AnalyzeProgram;
  566. end;
  567. procedure TTestUseAnalyzer.TestM_ProcedureType;
  568. begin
  569. StartProgram(false);
  570. Add('procedure {#DoIt_used}DoIt;');
  571. Add('type');
  572. Add(' {#TProc_used}TProc = procedure;');
  573. Add(' {#TFunc_used}TFunc = function(): longint;');
  574. Add('var');
  575. Add(' {#p_used}p: TProc;');
  576. Add(' {#f_used}f: TFunc;');
  577. Add('begin');
  578. Add(' p:=nil;');
  579. Add(' f:=nil;');
  580. Add('end;');
  581. Add('begin');
  582. Add(' DoIt;');
  583. AnalyzeProgram;
  584. end;
  585. procedure TTestUseAnalyzer.TestM_Params;
  586. begin
  587. StartProgram(false);
  588. Add('procedure {#DoIt_used}DoIt(const o);');
  589. Add('type');
  590. Add(' {#TEnum_used}TEnum = (red,blue);');
  591. Add('var');
  592. Add(' {#a_used}a: longint;');
  593. Add(' {#b_used}b: string;');
  594. Add(' {#c_used}c: longint;');
  595. Add(' {#d_used}d: TEnum;');
  596. Add('begin');
  597. Add(' DoIt(a);');
  598. Add(' DoIt(b[c]);');
  599. Add(' DoIt([d]);');
  600. Add(' DoIt(red);');
  601. Add('end;');
  602. Add('var {#s_used}s: string;');
  603. Add('begin');
  604. Add(' DoIt(s);');
  605. AnalyzeProgram;
  606. end;
  607. procedure TTestUseAnalyzer.TestM_Class;
  608. begin
  609. StartProgram(false);
  610. Add('type');
  611. Add(' {#integer_used}integer = longint;');
  612. Add(' {tobject_used}TObject = class');
  613. Add(' {#a_used}a: integer;');
  614. Add(' end;');
  615. Add('var Obj: TObject;');
  616. Add('begin');
  617. Add(' Obj.a:=3;');
  618. AnalyzeProgram;
  619. end;
  620. procedure TTestUseAnalyzer.TestM_ClassForward;
  621. begin
  622. StartProgram(false);
  623. Add('type');
  624. Add(' {#integer_notused}integer = longint;');
  625. Add(' {#TObject_used}TObject = class end;');
  626. Add(' TFelidae = class;');
  627. Add(' {#TCheetah_used}TCheetah = class');
  628. Add(' public');
  629. Add(' {#i_notused}i: integer;');
  630. Add(' {#f_used}f: TFelidae;');
  631. Add(' end;');
  632. Add(' {TFelidae_used}TFelidae = class');
  633. Add(' end;');
  634. Add('var {#c_used}c: TCheetah;');
  635. Add('begin');
  636. Add(' c.f:=nil;');
  637. AnalyzeProgram;
  638. end;
  639. procedure TTestUseAnalyzer.TestM_Class_Property;
  640. begin
  641. StartProgram(false);
  642. Add('type');
  643. Add(' {#integer_used}integer = longint;');
  644. Add(' {tobject_used}TObject = class');
  645. Add(' {#fa_used}Fa: integer;');
  646. Add(' {#fb_used}Fb: integer;');
  647. Add(' {#fc_used}Fc: integer;');
  648. Add(' {#fd_used}Fd: integer;');
  649. Add(' {#fe_notused}Fe: integer;');
  650. Add(' function {#getfc_used}GetFC: integer;');
  651. Add(' procedure {#setfd_used}SetFD({#setfd_value_used}Value: integer);');
  652. Add(' property {#A_used}A: integer read Fa write Fb;');
  653. Add(' property {#C_used}C: integer read GetFC write SetFD;');
  654. Add(' end;');
  655. Add('function TObject.GetFC: integer;');
  656. Add('begin');
  657. Add(' Result:=Fc;');
  658. Add('end;');
  659. Add('procedure TObject.SetFD({#setfd_value_impl_notused}Value: integer);');
  660. Add('begin');
  661. Add(' Fd:=Value;');
  662. Add('end;');
  663. Add('var Obj: TObject;');
  664. Add('begin');
  665. Add(' Obj.A:=Obj.A;');
  666. Add(' Obj.C:=Obj.C;');
  667. AnalyzeProgram;
  668. end;
  669. procedure TTestUseAnalyzer.TestM_Class_PropertyOverride;
  670. begin
  671. StartProgram(false);
  672. Add('type');
  673. Add(' {#integer_used}integer = longint;');
  674. Add(' {tobject_used}TObject = class');
  675. Add(' {#fa_used}FA: integer;');
  676. Add(' {#fb_notused}FB: integer;');
  677. Add(' property {#obj_a_notused}A: integer read FA write FB;');
  678. Add(' end;');
  679. Add(' {tmobile_used}TMobile = class(TObject)');
  680. Add(' {#fc_used}FC: integer;');
  681. Add(' property {#mob_a_used}A write FC;');
  682. Add(' end;');
  683. Add('var {#m_used}M: TMobile;');
  684. Add('begin');
  685. Add(' M.A:=M.A;');
  686. AnalyzeProgram;
  687. end;
  688. procedure TTestUseAnalyzer.TestM_Class_MethodOverride;
  689. begin
  690. StartProgram(false);
  691. Add('type');
  692. Add(' {tobject_used}TObject = class');
  693. Add(' procedure {#obj_doa_used}DoA; virtual; abstract;');
  694. Add(' procedure {#obj_dob_notused}DoB; virtual; abstract;');
  695. Add(' end;');
  696. Add(' {tmobile_used}TMobile = class(TObject)');
  697. Add(' constructor {#mob_create_used}Create;');
  698. Add(' procedure {#mob_doa_used}DoA; override;');
  699. Add(' procedure {#mob_dob_notused}DoB; override;');
  700. Add(' end;');
  701. Add('constructor TMobile.Create; begin end;');
  702. Add('procedure TMobile.DoA; begin end;');
  703. Add('procedure TMobile.DoB; begin end;');
  704. Add('var {#o_used}o: TObject;');
  705. Add('begin');
  706. Add(' o:=TMobile.Create;'); // use TMobile before o.DoA
  707. Add(' o.DoA;');
  708. AnalyzeProgram;
  709. end;
  710. procedure TTestUseAnalyzer.TestM_Class_MethodOverride2;
  711. begin
  712. StartProgram(false);
  713. Add('type');
  714. Add(' {tobject_used}TObject = class');
  715. Add(' procedure {#obj_doa_used}DoA; virtual; abstract;');
  716. Add(' end;');
  717. Add(' {tmobile_used}TMobile = class(TObject)');
  718. Add(' constructor {#mob_create_used}Create;');
  719. Add(' procedure {#mob_doa_used}DoA; override;');
  720. Add(' end;');
  721. Add('constructor TMobile.Create; begin end;');
  722. Add('procedure TMobile.DoA; begin end;');
  723. Add('var {#o_used}o: TObject;');
  724. Add('begin');
  725. Add(' o.DoA;');
  726. Add(' o:=TMobile.Create;'); // use TMobile after o.DoA
  727. AnalyzeProgram;
  728. end;
  729. procedure TTestUseAnalyzer.TestM_TryExceptStatement;
  730. begin
  731. StartProgram(false);
  732. Add('type');
  733. Add(' {tobject_used}TObject = class');
  734. Add(' constructor Create; external name ''create'';');
  735. Add(' end;');
  736. Add(' {texception_used}Exception = class(TObject);');
  737. Add(' {tdivbyzero_used}EDivByZero = class(Exception);');
  738. Add('procedure {#DoIt_used}DoIt;');
  739. Add('var');
  740. Add(' {#a_used}a: Exception;');
  741. Add(' {#b_used}b: Exception;');
  742. Add(' {#c_used}c: Exception;');
  743. Add(' {#d_used}d: Exception;');
  744. Add(' {#f_used}f: Exception;');
  745. Add('begin');
  746. Add(' try');
  747. Add(' a:=nil;');
  748. Add(' except');
  749. Add(' raise b;');
  750. Add(' end;');
  751. Add(' try');
  752. Add(' if Assigned(c) then ;');
  753. Add(' except');
  754. Add(' on {#e1_used}E1: Exception do raise;');
  755. Add(' on {#e2_notused}E2: EDivByZero do raise d;');
  756. Add(' else f:=nil;');
  757. Add(' end;');
  758. Add('end;');
  759. Add('begin');
  760. Add(' DoIt;');
  761. AnalyzeProgram;
  762. end;
  763. procedure TTestUseAnalyzer.TestM_Hint_UnitNotUsed;
  764. begin
  765. AddModuleWithIntfImplSrc('unit2.pp',
  766. LinesToStr([
  767. 'var i: longint;',
  768. 'procedure DoIt;',
  769. '']),
  770. LinesToStr([
  771. 'procedure DoIt; begin end;']));
  772. StartProgram(true);
  773. Add('uses unit2;');
  774. Add('begin');
  775. AnalyzeProgram;
  776. CheckHasHint(mtHint,nPAUnitNotUsed,'Unit "unit2" not used in afile');
  777. end;
  778. procedure TTestUseAnalyzer.TestM_Hint_UnitNotUsed_No_OnlyExternal;
  779. begin
  780. AddModuleWithIntfImplSrc('unit2.pp',
  781. LinesToStr([
  782. 'var State: longint; external name ''state'';',
  783. 'procedure DoIt; external name ''doing'';',
  784. '']),
  785. LinesToStr([
  786. ]));
  787. StartProgram(true);
  788. Add('uses unit2;');
  789. Add('begin');
  790. Add(' State:=3;');
  791. Add(' DoIt;');
  792. AnalyzeProgram;
  793. // unit hints: no hint, even though no code is actually used
  794. CheckHasHint(mtHint,nPAUnitNotUsed,'Unit "unit2" not used in afile',false);
  795. end;
  796. procedure TTestUseAnalyzer.TestM_Hint_ParameterNotUsed;
  797. begin
  798. StartProgram(true);
  799. Add('procedure DoIt(i: longint);');
  800. Add('begin end;');
  801. Add('begin');
  802. Add(' DoIt(1);');
  803. AnalyzeProgram;
  804. CheckHasHint(mtHint,nPAParameterNotUsed,'Parameter "i" not used');
  805. end;
  806. procedure TTestUseAnalyzer.TestM_Hint_ParameterNotUsed_Abstract;
  807. begin
  808. StartProgram(true);
  809. Add('type');
  810. Add(' TObject = class');
  811. Add(' class procedure DoIt(i: longint); virtual; abstract;');
  812. Add(' end;');
  813. Add('begin');
  814. Add(' TObject.DoIt(3);');
  815. AnalyzeProgram;
  816. CheckHasHint(mtHint,nPAParameterNotUsed,
  817. sPAParameterNotUsed,false);
  818. end;
  819. procedure TTestUseAnalyzer.TestM_Hint_LocalVariableNotUsed;
  820. begin
  821. StartProgram(true);
  822. Add('procedure DoIt;');
  823. Add('const');
  824. Add(' a = 13;');
  825. Add(' b: longint = 14;');
  826. Add('var');
  827. Add(' c: char;');
  828. Add(' d: longint = 15;');
  829. Add('begin end;');
  830. Add('begin');
  831. Add(' DoIt;');
  832. AnalyzeProgram;
  833. CheckHasHint(mtHint,nPALocalXYNotUsed,'Local constant "a" not used');
  834. CheckHasHint(mtHint,nPALocalXYNotUsed,'Local constant "b" not used');
  835. CheckHasHint(mtHint,nPALocalVariableNotUsed,'Local variable "c" not used');
  836. CheckHasHint(mtHint,nPALocalVariableNotUsed,'Local variable "d" not used');
  837. end;
  838. procedure TTestUseAnalyzer.TestM_Hint_InterfaceUnitVariableUsed;
  839. begin
  840. StartUnit(true);
  841. Add('interface');
  842. Add('const {#a_used}a = 1;');
  843. Add('const {#b_used}b: longint = 2;');
  844. Add('var {#c_used}c: longint = 3;');
  845. Add('type');
  846. Add(' {#TColor_used}TColor = longint;');
  847. Add(' {#TFlag_used}TFlag = (red,green);');
  848. Add(' {#TFlags_used}TFlags = set of TFlag;');
  849. Add(' {#TArrInt_used}TArrInt = array of integer;');
  850. Add('implementation');
  851. Add('const {#d_notused}d = 1;');
  852. Add('const {#e_notused}e: longint = 2;');
  853. Add('var {#f_notused}f: longint = 3;');
  854. Add('type');
  855. Add(' {#ImpTColor_notused}ImpTColor = longint;');
  856. Add(' {#ImpTFlag_notused}ImpTFlag = (red,green);');
  857. Add(' {#ImpTFlags_notused}ImpTFlags = set of TFlag;');
  858. Add(' {#ImpTArrInt_notused}ImpTArrInt = array of integer;');
  859. AnalyzeUnit;
  860. CheckHasHint(mtHint,nPALocalVariableIsAssignedButNeverUsed,
  861. 'Local variable "a" is assigned but never used',false);
  862. end;
  863. procedure TTestUseAnalyzer.TestM_Hint_ValueParameterIsAssignedButNeverUsed;
  864. begin
  865. StartProgram(true);
  866. Add('procedure DoIt(i: longint);');
  867. Add('begin');
  868. Add(' i:=3;');
  869. Add('end;');
  870. Add('begin');
  871. Add(' DoIt(1);');
  872. AnalyzeProgram;
  873. CheckHasHint(mtHint,nPAValueParameterIsAssignedButNeverUsed,
  874. 'Value parameter "i" is assigned but never used');
  875. end;
  876. procedure TTestUseAnalyzer.TestM_Hint_LocalVariableIsAssignedButNeverUsed;
  877. begin
  878. StartProgram(true);
  879. Add('procedure DoIt;');
  880. Add('const');
  881. Add(' a: longint = 14;');
  882. Add('var');
  883. Add(' b: char;');
  884. Add(' c: longint = 15;');
  885. Add('begin');
  886. Add(' a:=16;');
  887. Add(' b:=#65;');
  888. Add(' c:=17;');
  889. Add('end;');
  890. Add('begin');
  891. Add(' DoIt;');
  892. AnalyzeProgram;
  893. CheckHasHint(mtHint,nPALocalVariableIsAssignedButNeverUsed,
  894. 'Local variable "a" is assigned but never used');
  895. CheckHasHint(mtHint,nPALocalVariableIsAssignedButNeverUsed,
  896. 'Local variable "b" is assigned but never used');
  897. CheckHasHint(mtHint,nPALocalVariableIsAssignedButNeverUsed,
  898. 'Local variable "c" is assigned but never used');
  899. end;
  900. procedure TTestUseAnalyzer.TestM_Hint_LocalXYNotUsed;
  901. begin
  902. StartProgram(true);
  903. Add('procedure DoIt;');
  904. Add('type');
  905. Add(' TColor = longint;');
  906. Add(' TFlag = (red,green);');
  907. Add(' TFlags = set of TFlag;');
  908. Add(' TArrInt = array of integer;');
  909. Add(' procedure Sub; begin end;');
  910. Add('begin');
  911. Add('end;');
  912. Add('begin');
  913. Add(' DoIt;');
  914. AnalyzeProgram;
  915. CheckHasHint(mtHint,nPALocalXYNotUsed,'Local alias type "TColor" not used');
  916. CheckHasHint(mtHint,nPALocalXYNotUsed,'Local enumeration type "TFlag" not used');
  917. CheckHasHint(mtHint,nPALocalXYNotUsed,'Local set type "TFlags" not used');
  918. CheckHasHint(mtHint,nPALocalXYNotUsed,'Local array type "TArrInt" not used');
  919. CheckHasHint(mtHint,nPALocalXYNotUsed,'Local procedure "Sub" not used');
  920. end;
  921. procedure TTestUseAnalyzer.TestM_Hint_PrivateFieldIsNeverUsed;
  922. begin
  923. StartProgram(true,[supTObject]);
  924. Add('type');
  925. Add(' TMobile = class');
  926. Add(' private');
  927. Add(' a: longint;');
  928. Add(' end;');
  929. Add('var m: TMobile;');
  930. Add('begin');
  931. Add(' m:=nil;');
  932. AnalyzeProgram;
  933. CheckHasHint(mtHint,nPAPrivateFieldIsNeverUsed,'Private field "TMobile.a" is never used');
  934. end;
  935. procedure TTestUseAnalyzer.TestM_Hint_PrivateFieldIsAssignedButNeverUsed;
  936. begin
  937. StartProgram(true,[supTObject]);
  938. Add('type');
  939. Add(' TMobile = class');
  940. Add(' private');
  941. Add(' a: longint;');
  942. Add(' public');
  943. Add(' constructor Create;');
  944. Add(' end;');
  945. Add('constructor TMobile.Create;');
  946. Add('begin');
  947. Add(' a:=3;');
  948. Add('end;');
  949. Add('begin');
  950. Add(' TMobile.Create;');
  951. AnalyzeProgram;
  952. CheckHasHint(mtHint,nPAPrivateFieldIsAssignedButNeverUsed,
  953. 'Private field "TMobile.a" is assigned but never used');
  954. end;
  955. procedure TTestUseAnalyzer.TestM_Hint_PrivateMethodIsNeverUsed;
  956. begin
  957. StartProgram(true,[supTObject]);
  958. Add('type');
  959. Add(' TMobile = class');
  960. Add(' private');
  961. Add(' procedure DoSome; external name ''foo'';');
  962. Add(' public');
  963. Add(' constructor Create;');
  964. Add(' end;');
  965. Add('constructor TMobile.Create;');
  966. Add('begin');
  967. Add('end;');
  968. Add('begin');
  969. Add(' TMobile.Create;');
  970. AnalyzeProgram;
  971. CheckHasHint(mtHint,nPAPrivateMethodIsNeverUsed,
  972. 'Private method "TMobile.DoSome" is never used');
  973. end;
  974. procedure TTestUseAnalyzer.TestM_Hint_PrivateTypeNeverUsed;
  975. begin
  976. StartProgram(true,[supTObject]);
  977. Add('type');
  978. Add(' TMobile = class');
  979. Add(' private');
  980. Add(' type t = longint;');
  981. Add(' public');
  982. Add(' constructor Create;');
  983. Add(' end;');
  984. Add('constructor TMobile.Create;');
  985. Add('begin');
  986. Add('end;');
  987. Add('begin');
  988. Add(' TMobile.Create;');
  989. AnalyzeProgram;
  990. CheckHasHint(mtHint,nPAPrivateTypeXNeverUsed,
  991. 'Private type "TMobile.t" never used');
  992. end;
  993. procedure TTestUseAnalyzer.TestM_Hint_PrivateConstNeverUsed;
  994. begin
  995. StartProgram(true,[supTObject]);
  996. Add('type');
  997. Add(' TMobile = class');
  998. Add(' private');
  999. Add(' const c = 3;');
  1000. Add(' public');
  1001. Add(' constructor Create;');
  1002. Add(' end;');
  1003. Add('constructor TMobile.Create;');
  1004. Add('begin');
  1005. Add('end;');
  1006. Add('begin');
  1007. Add(' TMobile.Create;');
  1008. AnalyzeProgram;
  1009. CheckHasHint(mtHint,nPAPrivateConstXNeverUsed,
  1010. 'Private const "TMobile.c" never used');
  1011. end;
  1012. procedure TTestUseAnalyzer.TestM_Hint_PrivatePropertyNeverUsed;
  1013. begin
  1014. StartProgram(true,[supTObject]);
  1015. Add('type');
  1016. Add(' TMobile = class');
  1017. Add(' private');
  1018. Add(' FA: longint;');
  1019. Add(' property A: longint read FA;');
  1020. Add(' public');
  1021. Add(' constructor Create;');
  1022. Add(' end;');
  1023. Add('constructor TMobile.Create;');
  1024. Add('begin');
  1025. Add('end;');
  1026. Add('begin');
  1027. Add(' TMobile.Create;');
  1028. AnalyzeProgram;
  1029. CheckHasHint(mtHint,nPAPrivatePropertyXNeverUsed,
  1030. 'Private property "TMobile.A" never used');
  1031. end;
  1032. procedure TTestUseAnalyzer.TestM_Hint_LocalClassInProgramNotUsed;
  1033. begin
  1034. StartProgram(true,[supTObject]);
  1035. Add('type');
  1036. Add(' TMobile = class');
  1037. Add(' public');
  1038. Add(' constructor Create;');
  1039. Add(' end;');
  1040. Add('constructor TMobile.Create;');
  1041. Add('begin');
  1042. Add('end;');
  1043. Add('var');
  1044. Add(' m: TMobile;');
  1045. Add('begin');
  1046. AnalyzeProgram;
  1047. CheckHasHint(mtHint,nPALocalXYNotUsed,'Local class "TMobile" not used');
  1048. CheckHasHint(mtHint,nPALocalVariableNotUsed,'Local variable "m" not used');
  1049. end;
  1050. procedure TTestUseAnalyzer.TestM_Hint_LocalMethodInProgramNotUsed;
  1051. begin
  1052. StartProgram(true,[supTObject]);
  1053. Add('type');
  1054. Add(' TMobile = class');
  1055. Add(' public');
  1056. Add(' constructor Create;');
  1057. Add(' end;');
  1058. Add('constructor TMobile.Create;');
  1059. Add('begin');
  1060. Add('end;');
  1061. Add('var');
  1062. Add(' m: TMobile;');
  1063. Add('begin');
  1064. Add(' if m=nil then ;');
  1065. AnalyzeProgram;
  1066. CheckHasHint(mtHint,nPALocalXYNotUsed,'Local constructor "Create" not used');
  1067. end;
  1068. procedure TTestUseAnalyzer.TestM_Hint_AssemblerParameterIgnored;
  1069. begin
  1070. StartProgram(true);
  1071. Add('procedure DoIt(i: longint); assembler;');
  1072. Add('type');
  1073. Add(' {#tcolor_notused}TColor = longint;');
  1074. Add(' {#tflag_notused}TFlag = (red,green);');
  1075. Add(' {#tflags_notused}TFlags = set of TFlag;');
  1076. Add(' {#tarrint_notused}TArrInt = array of integer;');
  1077. Add('const');
  1078. Add(' {#a_notused}a = 13;');
  1079. Add(' {#b_notused}b: longint = 14;');
  1080. Add('var');
  1081. Add(' {#c_notused}c: char;');
  1082. Add(' {#d_notused}d: longint = 15;');
  1083. Add(' procedure {#sub_notused}Sub; begin end;');
  1084. Add('asm end;');
  1085. Add('begin');
  1086. Add(' DoIt(1);');
  1087. AnalyzeProgram;
  1088. CheckHasHint(mtHint,nPAParameterNotUsed,'Parameter "i" not used',false);
  1089. AssertEquals('no hints for assembler proc',0,PAMessageCount);
  1090. end;
  1091. procedure TTestUseAnalyzer.TestM_Hint_FunctionResultDoesNotSeemToBeSet;
  1092. begin
  1093. StartProgram(true);
  1094. Add('function DoIt: longint;');
  1095. Add('begin end;');
  1096. Add('begin');
  1097. Add(' DoIt();');
  1098. AnalyzeProgram;
  1099. CheckHasHint(mtHint,nPAFunctionResultDoesNotSeemToBeSet,
  1100. sPAFunctionResultDoesNotSeemToBeSet);
  1101. end;
  1102. procedure TTestUseAnalyzer.TestM_Hint_FunctionResultDoesNotSeemToBeSet_Abstract;
  1103. begin
  1104. StartProgram(true);
  1105. Add('type');
  1106. Add(' TObject = class');
  1107. Add(' class function DoIt: longint; virtual; abstract;');
  1108. Add(' end;');
  1109. Add('begin');
  1110. Add(' TObject.DoIt;');
  1111. AnalyzeProgram;
  1112. CheckHasHint(mtHint,nPAFunctionResultDoesNotSeemToBeSet,
  1113. sPAFunctionResultDoesNotSeemToBeSet,false);
  1114. end;
  1115. procedure TTestUseAnalyzer.TestM_Hint_FunctionResultRecord;
  1116. begin
  1117. StartProgram(true);
  1118. Add('type');
  1119. Add(' TPoint = record X,Y:longint; end;');
  1120. Add('function Point(Left,Top: longint): TPoint;');
  1121. Add('begin');
  1122. Add(' Result.X:=Left;');
  1123. Add('end;');
  1124. Add('begin');
  1125. Add(' Point(1,2);');
  1126. AnalyzeProgram;
  1127. CheckHasHint(mtHint,nPAFunctionResultDoesNotSeemToBeSet,
  1128. sPAFunctionResultDoesNotSeemToBeSet,false);
  1129. end;
  1130. procedure TTestUseAnalyzer.TestM_Hint_FunctionResultPassRecordElement;
  1131. begin
  1132. StartProgram(true);
  1133. Add('type');
  1134. Add(' TPoint = record X,Y:longint; end;');
  1135. Add('procedure Three(out x: longint);');
  1136. Add('begin');
  1137. Add(' x:=3;');
  1138. Add('end;');
  1139. Add('function Point(Left,Top: longint): TPoint;');
  1140. Add('begin');
  1141. Add(' Three(Result.X)');
  1142. Add('end;');
  1143. Add('begin');
  1144. Add(' Point(1,2);');
  1145. AnalyzeProgram;
  1146. CheckHasHint(mtHint,nPAFunctionResultDoesNotSeemToBeSet,
  1147. sPAFunctionResultDoesNotSeemToBeSet,false);
  1148. end;
  1149. procedure TTestUseAnalyzer.TestM_Hint_OutParam_No_AssignedButNeverUsed;
  1150. begin
  1151. StartProgram(true);
  1152. Add('procedure DoIt(out x: longint);');
  1153. Add('begin');
  1154. Add(' x:=3;');
  1155. Add('end;');
  1156. Add('var i: longint;');
  1157. Add('begin');
  1158. Add(' DoIt(i);');
  1159. AnalyzeProgram;
  1160. CheckHasHint(mtHint,nPAValueParameterIsAssignedButNeverUsed,
  1161. sPAValueParameterIsAssignedButNeverUsed,false);
  1162. end;
  1163. procedure TTestUseAnalyzer.TestWP_LocalVar;
  1164. begin
  1165. StartProgram(false);
  1166. Add('var {#a_notused}a: longint;');
  1167. Add('var {#b_used}b: longint;');
  1168. Add('var {#c_used}c: longint;');
  1169. Add('begin');
  1170. Add(' b:=2;');
  1171. Add(' afile.c:=3;');
  1172. AnalyzeWholeProgram;
  1173. end;
  1174. procedure TTestUseAnalyzer.TestWP_UnitUsed;
  1175. begin
  1176. AddModuleWithIntfImplSrc('unit2.pp',
  1177. LinesToStr([
  1178. 'var i: longint;',
  1179. 'procedure DoIt;',
  1180. '']),
  1181. LinesToStr([
  1182. 'procedure DoIt; begin end;']));
  1183. StartProgram(true);
  1184. Add('uses unit2;');
  1185. Add('begin');
  1186. Add(' i:=3;');
  1187. AnalyzeWholeProgram;
  1188. CheckUnitUsed('unit2.pp',true);
  1189. end;
  1190. procedure TTestUseAnalyzer.TestWP_UnitNotUsed;
  1191. begin
  1192. AddModuleWithIntfImplSrc('unit2.pp',
  1193. LinesToStr([
  1194. 'var i: longint;',
  1195. 'procedure DoIt;',
  1196. '']),
  1197. LinesToStr([
  1198. 'procedure DoIt; begin end;']));
  1199. StartProgram(true);
  1200. Add('uses unit2;');
  1201. Add('begin');
  1202. AnalyzeWholeProgram;
  1203. CheckUnitUsed('unit2.pp',false);
  1204. end;
  1205. procedure TTestUseAnalyzer.TestWP_UnitInitialization;
  1206. begin
  1207. AddModuleWithIntfImplSrc('unit1.pp',
  1208. LinesToStr([
  1209. 'uses unit2;',
  1210. '']),
  1211. LinesToStr([
  1212. 'initialization',
  1213. 'i:=2;']));
  1214. AddModuleWithIntfImplSrc('unit2.pp',
  1215. LinesToStr([
  1216. 'var i: longint;',
  1217. '']),
  1218. LinesToStr([
  1219. '']));
  1220. StartProgram(true);
  1221. Add('uses unit1;');
  1222. Add('begin');
  1223. AnalyzeWholeProgram;
  1224. CheckUnitUsed('unit1.pp',true);
  1225. CheckUnitUsed('unit2.pp',true);
  1226. end;
  1227. procedure TTestUseAnalyzer.TestWP_UnitFinalization;
  1228. begin
  1229. AddModuleWithIntfImplSrc('unit1.pp',
  1230. LinesToStr([
  1231. 'uses unit2;',
  1232. '']),
  1233. LinesToStr([
  1234. 'finalization',
  1235. 'i:=2;']));
  1236. AddModuleWithIntfImplSrc('unit2.pp',
  1237. LinesToStr([
  1238. 'var i: longint;',
  1239. '']),
  1240. LinesToStr([
  1241. '']));
  1242. StartProgram(true);
  1243. Add('uses unit1;');
  1244. Add('begin');
  1245. AnalyzeWholeProgram;
  1246. CheckUnitUsed('unit1.pp',true);
  1247. CheckUnitUsed('unit2.pp',true);
  1248. end;
  1249. procedure TTestUseAnalyzer.TestWP_CallInherited;
  1250. begin
  1251. StartProgram(false);
  1252. Add('type');
  1253. Add(' {#TObject_used}TObject = class');
  1254. Add(' procedure {#TObjectDoA_used}DoA;');
  1255. Add(' procedure {#TObjectDoB_used}DoB;');
  1256. Add(' end;');
  1257. Add(' {#TMobile_used}TMobile = class');
  1258. Add(' procedure {#TMobileDoA_used}DoA;');
  1259. Add(' procedure {#TMobileDoC_used}DoC;');
  1260. Add(' end;');
  1261. Add('procedure TObject.DoA; begin end;');
  1262. Add('procedure TObject.DoB; begin end;');
  1263. Add('procedure TMobile.DoA;');
  1264. Add('begin');
  1265. Add(' inherited;');
  1266. Add('end;');
  1267. Add('procedure TMobile.DoC;');
  1268. Add('begin');
  1269. Add(' inherited DoB;');
  1270. Add('end;');
  1271. Add('var o: TMobile;');
  1272. Add('begin');
  1273. Add(' o.DoA;');
  1274. Add(' o.DoC;');
  1275. AnalyzeWholeProgram;
  1276. end;
  1277. procedure TTestUseAnalyzer.TestWP_ProgramPublicDeclarations;
  1278. begin
  1279. StartProgram(false);
  1280. Add('var');
  1281. Add(' {#vPublic_used}vPublic: longint; public;');
  1282. Add(' {#vPrivate_notused}vPrivate: longint;');
  1283. Add('procedure {#DoPublic_used}DoPublic; public; begin end;');
  1284. Add('procedure {#DoPrivate_notused}DoPrivate; begin end;');
  1285. Add('begin');
  1286. AnalyzeWholeProgram;
  1287. end;
  1288. procedure TTestUseAnalyzer.TestWP_ClassDefaultProperty;
  1289. begin
  1290. StartProgram(false);
  1291. Add('type');
  1292. Add(' {#tobject_used}TObject = class');
  1293. Add(' function {#getitems_notused}Getitems(Index: longint): string;');
  1294. Add(' procedure {#setitems_used}Setitems(Index: longint; Value: String);');
  1295. Add(' property {#items_used}Items[Index: longint]: string read GetItems write SetItems; default;');
  1296. Add(' end;');
  1297. Add('function TObject.Getitems(Index: longint): string; begin end;');
  1298. Add('procedure TObject.Setitems(Index: longint; Value: String); begin end;');
  1299. Add('var');
  1300. Add(' {#l_used}L: TObject;');
  1301. Add('begin');
  1302. Add(' L[0]:=''birdy'';');
  1303. AnalyzeWholeProgram;
  1304. end;
  1305. initialization
  1306. RegisterTests([TTestUseAnalyzer]);
  1307. end.