123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228229230231232233234235236237238239240241242243244245246247248249250251252253254255256257258259260261262263264265266267268269270271272273274275276277278279280281282283284285286287288289290291292293294295296297298299300301302303304305306307308309310311312313314315316317318319320321322323324325326327328329330331332333334335336337338339340341342343344345346347348349350351352353354355356357358359360361362363364365366367368369370371372373374375376377378379380381382383384385386387388389390391392393394395396397398399400401402403404405406407408409410411412413414415416417418419420421422423424425426427428429430431432433434435436437438439440441442443444445446447448449450451452453454455456457458459460461462463464465466467468469470471472473474475476477478479480481482483484485486487488489490491492493494495496497498499500501502503504505506507508509510511512513514515516517518519520521522523524525526527528529530531532533534535536537538539540541542543544545546547548549550551552553554555556557558559560561562563564565566567568569570571572573574575576577578579580581582583584585586587588589590591592593594595596597598599600601602603604605606607608609610611612613614615616617618619620621622623624625626627628629630631632633634635636637638639640641642643644645646647648649650651652653654655656657658659660661662663664665666667668669670671672673674675676677678679680681682683684685686687688689690691692693694695696697698699700701702703704705706707708709710711712713714715716717718719720721722723724725726727728729730731732733734735736737738739740741742743744745746747748749750751752753754755756757758759760761762763764765766767768769770771772773774775776777778779780781782783784785786787788789790791792793794795796797798799800801802803804805806807808809810811812813814815816817818819820821822823824825826827828829830831832833834835836837838839840841842843844845846847848849850851852853854855856857858859860861862863864865866867868869870871872873874875876877878879880881882883884885886887888889890891892893894895896897898899900901902903904905906907908909910911912913914915916917918919920921922923924925926927928929930931932933934935936937938939940941942943944945946947948949950951952953954955956957958959960961962963964965966967968969970971972973974975976977978979980981982983984985986987988989990991992993994995996997998999100010011002100310041005100610071008100910101011101210131014101510161017101810191020102110221023102410251026102710281029103010311032103310341035103610371038103910401041104210431044104510461047104810491050105110521053105410551056105710581059106010611062106310641065106610671068106910701071107210731074107510761077107810791080108110821083108410851086108710881089109010911092109310941095109610971098109911001101110211031104110511061107110811091110111111121113111411151116111711181119112011211122112311241125112611271128112911301131113211331134113511361137113811391140114111421143114411451146114711481149115011511152115311541155115611571158115911601161116211631164116511661167116811691170117111721173117411751176117711781179118011811182118311841185118611871188118911901191119211931194119511961197119811991200120112021203120412051206120712081209121012111212121312141215121612171218121912201221122212231224122512261227122812291230123112321233123412351236123712381239124012411242124312441245124612471248124912501251125212531254125512561257125812591260126112621263126412651266126712681269127012711272127312741275127612771278127912801281128212831284128512861287128812891290129112921293129412951296129712981299130013011302130313041305130613071308130913101311131213131314131513161317131813191320132113221323132413251326132713281329133013311332133313341335133613371338133913401341134213431344134513461347134813491350135113521353135413551356135713581359136013611362136313641365136613671368136913701371137213731374137513761377137813791380138113821383138413851386138713881389139013911392139313941395139613971398139914001401140214031404140514061407140814091410141114121413141414151416141714181419142014211422142314241425142614271428142914301431143214331434143514361437143814391440144114421443144414451446144714481449145014511452145314541455145614571458145914601461146214631464146514661467146814691470147114721473147414751476147714781479148014811482148314841485148614871488148914901491149214931494149514961497149814991500150115021503150415051506150715081509151015111512151315141515151615171518151915201521152215231524152515261527152815291530153115321533153415351536153715381539154015411542154315441545154615471548154915501551155215531554155515561557155815591560156115621563156415651566156715681569157015711572157315741575157615771578157915801581158215831584158515861587158815891590159115921593159415951596159715981599160016011602160316041605160616071608160916101611161216131614161516161617161816191620162116221623162416251626162716281629163016311632163316341635163616371638163916401641 |
- {
- Examples:
- ./testpassrc --suite=TTestResolver.TestEmpty
- }
- unit tcuseanalyzer;
- {$mode objfpc}{$H+}
- interface
- uses
- Classes, SysUtils, fpcunit,
- PasTree, PScanner, PasResolver,
- tcbaseparser, testregistry, strutils, tcresolver, PasUseAnalyzer;
- type
- { TCustomTestUseAnalyzer }
- TCustomTestUseAnalyzer = Class(TCustomTestResolver)
- private
- FAnalyzer: TPasAnalyzer;
- FPAMessages: TFPList; // list of TPAMessage
- FPAGoodMessages: TFPList;
- function GetPAMessages(Index: integer): TPAMessage;
- procedure OnAnalyzerMessage(Sender: TObject; Msg: TPAMessage);
- protected
- procedure SetUp; override;
- procedure TearDown; override;
- procedure AnalyzeModule; virtual;
- procedure AnalyzeProgram; virtual;
- procedure AnalyzeUnit; virtual;
- procedure AnalyzeWholeProgram; virtual;
- procedure CheckUsedMarkers; virtual;
- procedure CheckHasHint(MsgType: TMessageType; MsgNumber: integer;
- const MsgText: string); virtual;
- procedure CheckUnexpectedMessages; virtual;
- procedure CheckUnitUsed(const aFilename: string; Used: boolean); virtual;
- public
- property Analyzer: TPasAnalyzer read FAnalyzer;
- function PAMessageCount: integer;
- property PAMessages[Index: integer]: TPAMessage read GetPAMessages;
- end;
- { TTestUseAnalyzer }
- TTestUseAnalyzer = Class(TCustomTestUseAnalyzer)
- published
- // single module
- procedure TestM_ProgramLocalVar;
- procedure TestM_AssignStatement;
- procedure TestM_BeginBlock;
- procedure TestM_ForLoopStatement;
- procedure TestM_AsmStatement;
- procedure TestM_CaseOfStatement;
- procedure TestM_IfThenElseStatement;
- procedure TestM_WhileDoStatement;
- procedure TestM_RepeatUntilStatement;
- procedure TestM_TryFinallyStatement;
- procedure TestM_TypeAlias;
- procedure TestM_Unary;
- procedure TestM_Const;
- procedure TestM_Record;
- procedure TestM_Array;
- procedure TestM_NestedFuncResult;
- procedure TestM_Enums;
- procedure TestM_ProcedureType;
- procedure TestM_Params;
- procedure TestM_Class;
- procedure TestM_ClassForward;
- procedure TestM_Class_Property;
- procedure TestM_Class_PropertyOverride;
- procedure TestM_Class_MethodOverride;
- procedure TestM_Class_MethodOverride2;
- procedure TestM_TryExceptStatement;
- // single module hints
- procedure TestM_Hint_UnitNotUsed;
- procedure TestM_Hint_UnitNotUsed_No_OnlyExternal;
- procedure TestM_Hint_ParameterNotUsed;
- procedure TestM_Hint_ParameterNotUsed_Abstract;
- procedure TestM_Hint_ParameterNotUsedTypecast;
- procedure TestM_Hint_LocalVariableNotUsed;
- procedure TestM_Hint_InterfaceUnitVariableUsed;
- procedure TestM_Hint_ValueParameterIsAssignedButNeverUsed;
- procedure TestM_Hint_LocalVariableIsAssignedButNeverUsed;
- procedure TestM_Hint_LocalXYNotUsed;
- procedure TestM_Hint_PrivateFieldIsNeverUsed;
- procedure TestM_Hint_PrivateFieldIsAssignedButNeverUsed;
- procedure TestM_Hint_PrivateMethodIsNeverUsed;
- procedure TestM_Hint_LocalDestructor_No_IsNeverUsed;
- procedure TestM_Hint_PrivateTypeNeverUsed;
- procedure TestM_Hint_PrivateConstNeverUsed;
- procedure TestM_Hint_PrivatePropertyNeverUsed;
- procedure TestM_Hint_LocalClassInProgramNotUsed;
- procedure TestM_Hint_LocalMethodInProgramNotUsed;
- procedure TestM_Hint_AssemblerParameterIgnored;
- procedure TestM_Hint_FunctionResultDoesNotSeemToBeSet;
- procedure TestM_Hint_FunctionResultDoesNotSeemToBeSet_Abstract;
- procedure TestM_Hint_FunctionResultRecord;
- procedure TestM_Hint_FunctionResultPassRecordElement;
- procedure TestM_Hint_OutParam_No_AssignedButNeverUsed;
- // whole program optimization
- procedure TestWP_LocalVar;
- procedure TestWP_UnitUsed;
- procedure TestWP_UnitNotUsed;
- procedure TestWP_UnitInitialization;
- procedure TestWP_UnitFinalization;
- procedure TestWP_CallInherited;
- procedure TestWP_ProgramPublicDeclarations;
- procedure TestWP_ClassDefaultProperty;
- procedure TestWP_Published;
- procedure TestWP_PublishedSetType;
- procedure TestWP_PublishedArrayType;
- procedure TestWP_PublishedClassOfType;
- procedure TestWP_PublishedRecordType;
- procedure TestWP_PublishedProcType;
- procedure TestWP_PublishedProperty;
- end;
- implementation
- { TCustomTestUseAnalyzer }
- procedure TCustomTestUseAnalyzer.OnAnalyzerMessage(Sender: TObject;
- Msg: TPAMessage);
- begin
- Msg.AddRef;
- FPAMessages.Add(Msg);
- end;
- function TCustomTestUseAnalyzer.GetPAMessages(Index: integer): TPAMessage;
- begin
- Result:=TPAMessage(FPAMessages[Index]);
- end;
- procedure TCustomTestUseAnalyzer.SetUp;
- begin
- inherited SetUp;
- FPAMessages:=TFPList.Create;
- FPAGoodMessages:=TFPList.Create;
- FAnalyzer:=TPasAnalyzer.Create;
- FAnalyzer.Resolver:=ResolverEngine;
- Analyzer.OnMessage:=@OnAnalyzerMessage;
- end;
- procedure TCustomTestUseAnalyzer.TearDown;
- var
- i: Integer;
- begin
- FreeAndNil(FPAGoodMessages);
- for i:=0 to FPAMessages.Count-1 do
- TPAMessage(FPAMessages[i]).Release;
- FreeAndNil(FPAMessages);
- FreeAndNil(FAnalyzer);
- inherited TearDown;
- end;
- procedure TCustomTestUseAnalyzer.AnalyzeModule;
- begin
- Analyzer.AnalyzeModule(Module);
- Analyzer.EmitModuleHints(Module);
- CheckUsedMarkers;
- end;
- procedure TCustomTestUseAnalyzer.AnalyzeProgram;
- begin
- ParseProgram;
- AnalyzeModule;
- end;
- procedure TCustomTestUseAnalyzer.AnalyzeUnit;
- begin
- ParseUnit;
- AnalyzeModule;
- end;
- procedure TCustomTestUseAnalyzer.AnalyzeWholeProgram;
- begin
- ParseProgram;
- Analyzer.AnalyzeWholeProgram(Module as TPasProgram);
- CheckUsedMarkers;
- end;
- procedure TCustomTestUseAnalyzer.CheckUsedMarkers;
- var
- aMarker: PSrcMarker;
- p: SizeInt;
- Postfix: String;
- Elements: TFPList;
- i: Integer;
- El: TPasElement;
- ExpectedUsed: Boolean;
- FoundEl: TPAElement;
- begin
- aMarker:=FirstSrcMarker;
- while aMarker<>nil do
- begin
- writeln('TCustomTestUseAnalyzer.CheckUsedMarkers ',aMarker^.Identifier,' Line=',aMarker^.Row,' StartCol=',aMarker^.StartCol,' EndCol=',aMarker^.EndCol);
- p:=RPos('_',aMarker^.Identifier);
- if p>1 then
- begin
- Postfix:=copy(aMarker^.Identifier,p+1);
- if Postfix='used' then
- ExpectedUsed:=true
- else if Postfix='notused' then
- ExpectedUsed:=false
- else
- RaiseErrorAtSrcMarker('TCustomTestUseAnalyzer.CheckUsedMarkers unknown postfix "'+Postfix+'"',aMarker);
- Elements:=FindElementsAt(aMarker);
- try
- FoundEl:=nil;
- for i:=0 to Elements.Count-1 do
- begin
- El:=TPasElement(Elements[i]);
- writeln('TCustomTestUseAnalyzer.CheckUsedMarkers ',aMarker^.Identifier,' ',i,'/',Elements.Count,' El=',GetObjName(El),' ',GetObjName(El.CustomData));
- FoundEl:=Analyzer.FindElement(El);
- if FoundEl<>nil then break;
- end;
- if FoundEl<>nil then
- begin
- if not ExpectedUsed then
- RaiseErrorAtSrcMarker('expected element to be *not* used, but it is marked',aMarker);
- end
- else
- begin
- if ExpectedUsed then
- RaiseErrorAtSrcMarker('expected element to be used, but it is not marked',aMarker);
- end;
- finally
- Elements.Free;
- end;
- end;
- aMarker:=aMarker^.Next;
- end;
- end;
- procedure TCustomTestUseAnalyzer.CheckHasHint(MsgType: TMessageType;
- MsgNumber: integer; const MsgText: string);
- var
- i: Integer;
- Msg: TPAMessage;
- s: string;
- begin
- i:=PAMessageCount-1;
- while i>=0 do
- begin
- Msg:=PAMessages[i];
- if (Msg.MsgNumber=MsgNumber) then
- begin
- if (Msg.MsgType=MsgType) and (Msg.MsgText=MsgText) then
- begin
- FPAGoodMessages.Add(Msg);
- exit;
- end;
- end;
- dec(i);
- end;
- // mismatch
- writeln('TCustomTestUseAnalyzer.CheckHasHint: ');
- for i:=0 to PAMessageCount-1 do
- begin
- Msg:=PAMessages[i];
- writeln(' ',i,'/',PAMessageCount,': [',Msg.Id,'] ',Msg.MsgType,': (',Msg.MsgNumber,') {',Msg.MsgText,'}');
- end;
- s:='';
- str(MsgType,s);
- Fail('Analyzer Message not found: '+s+': ('+IntToStr(MsgNumber)+') {'+MsgText+'}');
- end;
- procedure TCustomTestUseAnalyzer.CheckUnexpectedMessages;
- var
- i: Integer;
- Msg: TPAMessage;
- s: String;
- begin
- for i:=0 to PAMessageCount-1 do
- begin
- Msg:=PAMessages[i];
- if FPAGoodMessages.IndexOf(Msg)>=0 then continue;
- s:='';
- str(Msg.MsgType,s);
- Fail('Analyzer Message found ['+IntToStr(Msg.Id)+'] '+s+': ('+IntToStr(Msg.MsgNumber)+') {'+Msg.MsgText+'}');
- end;
- end;
- procedure TCustomTestUseAnalyzer.CheckUnitUsed(const aFilename: string;
- Used: boolean);
- var
- aResolver: TTestEnginePasResolver;
- PAEl: TPAElement;
- begin
- aResolver:=FindModuleWithFilename(aFilename);
- AssertNotNull('unit not found "'+aFilename+'"',aResolver);
- AssertNotNull('unit module not found "'+aFilename+'"',aResolver.Module);
- PAEl:=Analyzer.FindElement(aResolver.Module);
- if PAEl<>nil then
- begin
- // unit is used
- if not Used then
- Fail('expected unit "'+aFilename+'" not used, but it is used');
- end
- else
- begin
- // unit is not used
- if Used then
- Fail('expected unit "'+aFilename+'" used, but it is not used');
- end;
- end;
- function TCustomTestUseAnalyzer.PAMessageCount: integer;
- begin
- Result:=FPAMessages.Count;
- end;
- { TTestUseAnalyzer }
- procedure TTestUseAnalyzer.TestM_ProgramLocalVar;
- begin
- StartProgram(false);
- Add('procedure {#DoIt_used}DoIt;');
- Add('var {#l_notused}l: longint;');
- Add('begin');
- Add('end;');
- Add('begin');
- Add(' DoIt;');
- AnalyzeProgram;
- end;
- procedure TTestUseAnalyzer.TestM_AssignStatement;
- begin
- StartProgram(false);
- Add('procedure {#DoIt_used}DoIt;');
- Add('var');
- Add(' {#a_notused}a: longint;');
- Add(' {#b_used}b: longint;');
- Add(' {#c_used}c: longint;');
- Add('begin');
- Add(' b:=c;');
- Add('end;');
- Add('begin');
- Add(' DoIt;');
- AnalyzeProgram;
- end;
- procedure TTestUseAnalyzer.TestM_BeginBlock;
- begin
- StartProgram(false);
- Add('procedure {#DoIt_used}DoIt;');
- Add('var');
- Add(' {#a_used}a: longint;');
- Add('begin');
- Add(' begin');
- Add(' a:=1;');
- Add(' end;');
- Add('end;');
- Add('begin');
- Add(' DoIt;');
- AnalyzeProgram;
- end;
- procedure TTestUseAnalyzer.TestM_ForLoopStatement;
- begin
- StartProgram(false);
- Add('procedure {#DoIt_used}DoIt;');
- Add('var');
- Add(' {#a_used}a: longint;');
- Add(' {#b_used}b: longint;');
- Add(' {#c_used}c: longint;');
- Add(' {#d_used}d: longint;');
- Add('begin');
- Add(' for a:=b to c do d:=a;');
- Add('end;');
- Add('begin');
- Add(' DoIt;');
- AnalyzeProgram;
- end;
- procedure TTestUseAnalyzer.TestM_AsmStatement;
- begin
- StartProgram(false);
- Add('procedure {#DoIt_used}DoIt;');
- Add('begin');
- Add(' asm end;');
- Add('end;');
- Add('begin');
- Add(' DoIt;');
- AnalyzeProgram;
- end;
- procedure TTestUseAnalyzer.TestM_CaseOfStatement;
- begin
- StartProgram(false);
- Add('procedure {#DoIt_used}DoIt;');
- Add('const');
- Add(' {#a_used}a = 1;');
- Add(' {#b_used}b = 2;');
- Add('var');
- Add(' {#c_used}c: longint;');
- Add(' {#d_used}d: longint;');
- Add('begin');
- Add(' case a of');
- Add(' b: c:=1;');
- Add(' else');
- Add(' d:=2;');
- Add(' end;');
- Add('end;');
- Add('begin');
- Add(' DoIt;');
- AnalyzeProgram;
- end;
- procedure TTestUseAnalyzer.TestM_IfThenElseStatement;
- begin
- StartProgram(false);
- Add('procedure {#DoIt_used}DoIt;');
- Add('var');
- Add(' {#a_used}a: longint;');
- Add(' {#b_used}b: longint;');
- Add(' {#c_used}c: longint;');
- Add('begin');
- Add(' if a=0 then b:=1 else c:=2;');
- Add('end;');
- Add('begin');
- Add(' DoIt;');
- AnalyzeProgram;
- end;
- procedure TTestUseAnalyzer.TestM_WhileDoStatement;
- begin
- StartProgram(false);
- Add('procedure {#DoIt_used}DoIt;');
- Add('var');
- Add(' {#a_used}a: longint;');
- Add(' {#b_used}b: longint;');
- Add('begin');
- Add(' while a>0 do b:=1;');
- Add('end;');
- Add('begin');
- Add(' DoIt;');
- AnalyzeProgram;
- end;
- procedure TTestUseAnalyzer.TestM_RepeatUntilStatement;
- begin
- StartProgram(false);
- Add('procedure {#DoIt_used}DoIt;');
- Add('var');
- Add(' {#a_used}a: longint;');
- Add(' {#b_used}b: longint;');
- Add('begin');
- Add(' repeat a:=1; until b>1;');
- Add('end;');
- Add('begin');
- Add(' DoIt;');
- AnalyzeProgram;
- end;
- procedure TTestUseAnalyzer.TestM_TryFinallyStatement;
- begin
- StartProgram(false);
- Add('procedure {#DoIt_used}DoIt;');
- Add('var');
- Add(' {#a_used}a: longint;');
- Add(' {#b_used}b: longint;');
- Add('begin');
- Add(' try');
- Add(' a:=1;');
- Add(' finally');
- Add(' b:=2;');
- Add(' end;');
- Add('end;');
- Add('begin');
- Add(' DoIt;');
- AnalyzeProgram;
- end;
- procedure TTestUseAnalyzer.TestM_TypeAlias;
- begin
- StartProgram(false);
- Add('procedure {#DoIt_used}DoIt;');
- Add('type');
- Add(' {#integer_used}integer = longint;');
- Add('var');
- Add(' {#a_used}a: integer;');
- Add(' {#b_used}b: integer;');
- Add(' {#c_notused}c: integer;');
- Add('begin');
- Add(' a:=b;');
- Add('end;');
- Add('begin');
- Add(' DoIt;');
- AnalyzeProgram;
- end;
- procedure TTestUseAnalyzer.TestM_Unary;
- begin
- StartProgram(false);
- Add('procedure {#DoIt_used}DoIt;');
- Add('var');
- Add(' {#a_used}a: longint;');
- Add(' {#b_used}b: longint;');
- Add(' {#c_used}c: longint;');
- Add(' {#d_used}d: longint;');
- Add('begin');
- Add(' a:=+b;');
- Add(' a:=c+d;');
- Add('end;');
- Add('begin');
- Add(' DoIt;');
- AnalyzeProgram;
- end;
- procedure TTestUseAnalyzer.TestM_Const;
- begin
- StartProgram(false);
- Add('procedure {#DoIt_used}DoIt;');
- Add('var');
- Add(' {#a_used}a: longint;');
- Add(' {#b_used}b: boolean;');
- Add(' {#c_used}c: array of longint;');
- Add(' {#d_used}d: string;');
- Add('begin');
- Add(' a:=+1;');
- Add(' b:=true;');
- Add(' c:=nil;');
- Add(' d:=''foo'';');
- Add('end;');
- Add('begin');
- Add(' DoIt;');
- AnalyzeProgram;
- end;
- procedure TTestUseAnalyzer.TestM_Record;
- begin
- StartProgram(false);
- Add('procedure {#DoIt_used}DoIt;');
- Add('type');
- Add(' {#integer_used}integer = longint;');
- Add(' {#trec_used}TRec = record');
- Add(' {#a_used}a: integer;');
- Add(' {#b_notused}b: integer;');
- Add(' {#c_used}c: integer;');
- Add(' end;');
- Add('var');
- Add(' {#r_used}r: TRec;');
- Add('begin');
- Add(' r.a:=3;');
- Add(' with r do c:=4;');
- Add('end;');
- Add('begin');
- Add(' DoIt;');
- AnalyzeProgram;
- end;
- procedure TTestUseAnalyzer.TestM_Array;
- begin
- StartProgram(false);
- Add('procedure {#DoIt_used}DoIt;');
- Add('type');
- Add(' {#integer_used}integer = longint;');
- Add(' {#tarrayint_used}TArrayInt = array of integer;');
- Add('var');
- Add(' {#a_used}a: TArrayInt;');
- Add(' {#b_used}b: integer;');
- Add(' {#c_used}c: TArrayInt;');
- Add(' {#d_used}d: integer;');
- Add(' {#e_used}e: TArrayInt;');
- Add(' {#f_used}f: integer;');
- Add(' {#g_used}g: TArrayInt;');
- Add(' {#h_used}h: TArrayInt;');
- Add(' {#i_used}i: TArrayInt;');
- Add('begin');
- Add(' a[b]:=c[d];');
- Add(' SetLength(e,f)');
- Add(' if low(g)=high(h)+length(i) then');
- Add('end;');
- Add('begin');
- Add(' DoIt;');
- AnalyzeProgram;
- end;
- procedure TTestUseAnalyzer.TestM_NestedFuncResult;
- begin
- StartProgram(false);
- Add('procedure {#DoIt_used}DoIt;');
- Add('type');
- Add(' {#integer_used}integer = longint;');
- Add(' {#tarrayint_used}TArrayInt = array of integer;');
- Add(' function {#nestedfunc_used}NestedFunc({#b_notused}b: longint): TArrayInt;');
- Add(' begin');
- Add(' end;');
- Add('var');
- Add(' {#d_used}d: longint;');
- Add('begin');
- Add(' NestedFunc(d);');
- Add('end;');
- Add('begin');
- Add(' DoIt;');
- AnalyzeProgram;
- end;
- procedure TTestUseAnalyzer.TestM_Enums;
- begin
- StartProgram(false);
- Add('procedure {#DoIt_used}DoIt(const o);');
- Add('type');
- Add(' {#TEnum_used}TEnum = (red,blue);');
- Add(' {#TEnums_used}TEnums = set of TEnum;');
- Add('var');
- Add(' {#a_used}a: TEnum;');
- Add(' {#b_used}b: TEnums;');
- Add(' {#c_used}c: TEnum;');
- Add(' {#d_used}d: TEnums;');
- Add(' {#e_used}e: TEnums;');
- Add(' {#f_used}f: TEnums;');
- Add(' {#g_used}g: TEnum;');
- Add(' {#h_used}h: TEnum;');
- Add('begin');
- Add(' b:=[a];');
- Add(' if c in d then;');
- Add(' if low(e)=high(f) then;');
- Add(' if pred(g)=succ(h) then;');
- Add('end;');
- Add('var {#s_used}s: string;');
- Add('begin');
- Add(' DoIt(s);');
- AnalyzeProgram;
- end;
- procedure TTestUseAnalyzer.TestM_ProcedureType;
- begin
- StartProgram(false);
- Add('procedure {#DoIt_used}DoIt;');
- Add('type');
- Add(' {#TProc_used}TProc = procedure;');
- Add(' {#TFunc_used}TFunc = function(): longint;');
- Add('var');
- Add(' {#p_used}p: TProc;');
- Add(' {#f_used}f: TFunc;');
- Add('begin');
- Add(' p:=nil;');
- Add(' f:=nil;');
- Add('end;');
- Add('begin');
- Add(' DoIt;');
- AnalyzeProgram;
- end;
- procedure TTestUseAnalyzer.TestM_Params;
- begin
- StartProgram(false);
- Add('procedure {#DoIt_used}DoIt(const o);');
- Add('type');
- Add(' {#TEnum_used}TEnum = (red,blue);');
- Add('var');
- Add(' {#a_used}a: longint;');
- Add(' {#b_used}b: string;');
- Add(' {#c_used}c: longint;');
- Add(' {#d_used}d: TEnum;');
- Add('begin');
- Add(' DoIt(a);');
- Add(' DoIt(b[c]);');
- Add(' DoIt([d]);');
- Add(' DoIt(red);');
- Add('end;');
- Add('var {#s_used}s: string;');
- Add('begin');
- Add(' DoIt(s);');
- AnalyzeProgram;
- end;
- procedure TTestUseAnalyzer.TestM_Class;
- begin
- StartProgram(false);
- Add('type');
- Add(' {#integer_used}integer = longint;');
- Add(' {tobject_used}TObject = class');
- Add(' {#a_used}a: integer;');
- Add(' end;');
- Add('var Obj: TObject;');
- Add('begin');
- Add(' Obj.a:=3;');
- AnalyzeProgram;
- end;
- procedure TTestUseAnalyzer.TestM_ClassForward;
- begin
- StartProgram(false);
- Add('type');
- Add(' {#integer_notused}integer = longint;');
- Add(' {#TObject_used}TObject = class end;');
- Add(' TFelidae = class;');
- Add(' {#TCheetah_used}TCheetah = class');
- Add(' public');
- Add(' {#i_notused}i: integer;');
- Add(' {#f_used}f: TFelidae;');
- Add(' end;');
- Add(' {TFelidae_used}TFelidae = class');
- Add(' end;');
- Add('var {#c_used}c: TCheetah;');
- Add('begin');
- Add(' c.f:=nil;');
- AnalyzeProgram;
- end;
- procedure TTestUseAnalyzer.TestM_Class_Property;
- begin
- StartProgram(false);
- Add('type');
- Add(' {#integer_used}integer = longint;');
- Add(' {tobject_used}TObject = class');
- Add(' {#fa_used}Fa: integer;');
- Add(' {#fb_used}Fb: integer;');
- Add(' {#fc_used}Fc: integer;');
- Add(' {#fd_used}Fd: integer;');
- Add(' {#fe_notused}Fe: integer;');
- Add(' function {#getfc_used}GetFC: integer;');
- Add(' procedure {#setfd_used}SetFD({#setfd_value_used}Value: integer);');
- Add(' property {#A_used}A: integer read Fa write Fb;');
- Add(' property {#C_used}C: integer read GetFC write SetFD;');
- Add(' end;');
- Add('function TObject.GetFC: integer;');
- Add('begin');
- Add(' Result:=Fc;');
- Add('end;');
- Add('procedure TObject.SetFD({#setfd_value_impl_notused}Value: integer);');
- Add('begin');
- Add(' Fd:=Value;');
- Add('end;');
- Add('var Obj: TObject;');
- Add('begin');
- Add(' Obj.A:=Obj.A;');
- Add(' Obj.C:=Obj.C;');
- AnalyzeProgram;
- end;
- procedure TTestUseAnalyzer.TestM_Class_PropertyOverride;
- begin
- StartProgram(false);
- Add('type');
- Add(' {#integer_used}integer = longint;');
- Add(' {tobject_used}TObject = class');
- Add(' {#fa_used}FA: integer;');
- Add(' {#fb_notused}FB: integer;');
- Add(' property {#obj_a_notused}A: integer read FA write FB;');
- Add(' end;');
- Add(' {tmobile_used}TMobile = class(TObject)');
- Add(' {#fc_used}FC: integer;');
- Add(' property {#mob_a_used}A write FC;');
- Add(' end;');
- Add('var {#m_used}M: TMobile;');
- Add('begin');
- Add(' M.A:=M.A;');
- AnalyzeProgram;
- end;
- procedure TTestUseAnalyzer.TestM_Class_MethodOverride;
- begin
- StartProgram(false);
- Add('type');
- Add(' {tobject_used}TObject = class');
- Add(' procedure {#obj_doa_used}DoA; virtual; abstract;');
- Add(' procedure {#obj_dob_notused}DoB; virtual; abstract;');
- Add(' end;');
- Add(' {tmobile_used}TMobile = class(TObject)');
- Add(' constructor {#mob_create_used}Create;');
- Add(' procedure {#mob_doa_used}DoA; override;');
- Add(' procedure {#mob_dob_used}DoB; override;');
- Add(' end;');
- Add('constructor TMobile.Create; begin end;');
- Add('procedure TMobile.DoA; begin end;');
- Add('procedure TMobile.DoB; begin end;');
- Add('var {#o_used}o: TObject;');
- Add('begin');
- Add(' o:=TMobile.Create;'); // use TMobile before o.DoA
- Add(' o.DoA;');
- AnalyzeProgram;
- end;
- procedure TTestUseAnalyzer.TestM_Class_MethodOverride2;
- begin
- StartProgram(false);
- Add('type');
- Add(' {tobject_used}TObject = class');
- Add(' procedure {#obj_doa_used}DoA; virtual; abstract;');
- Add(' end;');
- Add(' {tmobile_used}TMobile = class(TObject)');
- Add(' constructor {#mob_create_used}Create;');
- Add(' procedure {#mob_doa_used}DoA; override;');
- Add(' end;');
- Add('constructor TMobile.Create; begin end;');
- Add('procedure TMobile.DoA; begin end;');
- Add('var {#o_used}o: TObject;');
- Add('begin');
- Add(' o.DoA;');
- Add(' o:=TMobile.Create;'); // use TMobile after o.DoA
- AnalyzeProgram;
- end;
- procedure TTestUseAnalyzer.TestM_TryExceptStatement;
- begin
- StartProgram(false);
- Add('type');
- Add(' {tobject_used}TObject = class');
- Add(' constructor Create; external name ''create'';');
- Add(' end;');
- Add(' {texception_used}Exception = class(TObject);');
- Add(' {tdivbyzero_used}EDivByZero = class(Exception);');
- Add('procedure {#DoIt_used}DoIt;');
- Add('var');
- Add(' {#a_used}a: Exception;');
- Add(' {#b_used}b: Exception;');
- Add(' {#c_used}c: Exception;');
- Add(' {#d_used}d: Exception;');
- Add(' {#f_used}f: Exception;');
- Add('begin');
- Add(' try');
- Add(' a:=nil;');
- Add(' except');
- Add(' raise b;');
- Add(' end;');
- Add(' try');
- Add(' if Assigned(c) then ;');
- Add(' except');
- Add(' on {#e1_used}E1: Exception do raise;');
- Add(' on {#e2_notused}E2: EDivByZero do raise d;');
- Add(' else f:=nil;');
- Add(' end;');
- Add('end;');
- Add('begin');
- Add(' DoIt;');
- AnalyzeProgram;
- end;
- procedure TTestUseAnalyzer.TestM_Hint_UnitNotUsed;
- begin
- AddModuleWithIntfImplSrc('unit2.pp',
- LinesToStr([
- 'var i: longint;',
- 'procedure DoIt;',
- '']),
- LinesToStr([
- 'procedure DoIt; begin end;']));
- StartProgram(true);
- Add('uses unit2;');
- Add('begin');
- AnalyzeProgram;
- CheckHasHint(mtHint,nPAUnitNotUsed,'Unit "unit2" not used in afile');
- CheckUnexpectedMessages;
- end;
- procedure TTestUseAnalyzer.TestM_Hint_UnitNotUsed_No_OnlyExternal;
- begin
- AddModuleWithIntfImplSrc('unit2.pp',
- LinesToStr([
- 'var State: longint; external name ''state'';',
- 'procedure DoIt; external name ''doing'';',
- '']),
- LinesToStr([
- ]));
- StartProgram(true);
- Add('uses unit2;');
- Add('begin');
- Add(' State:=3;');
- Add(' DoIt;');
- AnalyzeProgram;
- // unit hints: no hint, even though no code is actually used
- CheckUnexpectedMessages;
- end;
- procedure TTestUseAnalyzer.TestM_Hint_ParameterNotUsed;
- begin
- StartProgram(true);
- Add('procedure DoIt(i: longint);');
- Add('begin end;');
- Add('begin');
- Add(' DoIt(1);');
- AnalyzeProgram;
- CheckHasHint(mtHint,nPAParameterNotUsed,'Parameter "i" not used');
- CheckUnexpectedMessages;
- end;
- procedure TTestUseAnalyzer.TestM_Hint_ParameterNotUsed_Abstract;
- begin
- StartProgram(true);
- Add('type');
- Add(' TObject = class');
- Add(' class procedure DoIt(i: longint); virtual; abstract;');
- Add(' end;');
- Add('begin');
- Add(' TObject.DoIt(3);');
- AnalyzeProgram;
- CheckUnexpectedMessages;
- end;
- procedure TTestUseAnalyzer.TestM_Hint_ParameterNotUsedTypecast;
- begin
- StartProgram(true);
- Add('type');
- Add(' TObject = class end;');
- Add(' TSortCompare = function(a,b: Pointer): integer;');
- Add(' TObjCompare = function(a,b: TObject): integer;');
- Add('procedure Sort(const Compare: TSortCompare);');
- Add('begin');
- Add(' Compare(nil,nil);');
- Add('end;');
- Add('procedure DoIt(const Compare: TObjCompare);');
- Add('begin');
- Add(' Sort(TSortCompare(Compare));');
- Add('end;');
- Add('begin');
- Add(' DoIt(nil);');
- AnalyzeProgram;
- CheckUnexpectedMessages;
- end;
- procedure TTestUseAnalyzer.TestM_Hint_LocalVariableNotUsed;
- begin
- StartProgram(true);
- Add('procedure DoIt;');
- Add('const');
- Add(' a = 13;');
- Add(' b: longint = 14;');
- Add('var');
- Add(' c: char;');
- Add(' d: longint = 15;');
- Add('begin end;');
- Add('begin');
- Add(' DoIt;');
- AnalyzeProgram;
- CheckHasHint(mtHint,nPALocalXYNotUsed,'Local constant "a" not used');
- CheckHasHint(mtHint,nPALocalXYNotUsed,'Local constant "b" not used');
- CheckHasHint(mtHint,nPALocalVariableNotUsed,'Local variable "c" not used');
- CheckHasHint(mtHint,nPALocalVariableNotUsed,'Local variable "d" not used');
- CheckUnexpectedMessages;
- end;
- procedure TTestUseAnalyzer.TestM_Hint_InterfaceUnitVariableUsed;
- begin
- StartUnit(true);
- Add('interface');
- Add('const {#a_used}a = 1;');
- Add('const {#b_used}b: longint = 2;');
- Add('var {#c_used}c: longint = 3;');
- Add('type');
- Add(' {#TColor_used}TColor = longint;');
- Add(' {#TFlag_used}TFlag = (red,green);');
- Add(' {#TFlags_used}TFlags = set of TFlag;');
- Add(' {#TArrInt_used}TArrInt = array of integer;');
- Add('implementation');
- Add('const {#d_notused}d = 1;');
- Add('const {#e_notused}e: longint = 2;');
- Add('var {#f_notused}f: longint = 3;');
- Add('type');
- Add(' {#ImpTColor_notused}ImpTColor = longint;');
- Add(' {#ImpTFlag_notused}ImpTFlag = (red,green);');
- Add(' {#ImpTFlags_notused}ImpTFlags = set of TFlag;');
- Add(' {#ImpTArrInt_notused}ImpTArrInt = array of integer;');
- AnalyzeUnit;
- CheckHasHint(mtHint,nPALocalXYNotUsed,'Local constant "d" not used');
- CheckHasHint(mtHint,nPALocalXYNotUsed,'Local constant "e" not used');
- CheckHasHint(mtHint,nPALocalVariableNotUsed,'Local variable "f" not used');
- CheckHasHint(mtHint,nPALocalXYNotUsed,'Local alias type "ImpTColor" not used');
- CheckHasHint(mtHint,nPALocalXYNotUsed,'Local enumeration type "ImpTFlag" not used');
- CheckHasHint(mtHint,nPALocalXYNotUsed,'Local set type "ImpTFlags" not used');
- CheckHasHint(mtHint,nPALocalXYNotUsed,'Local array type "ImpTArrInt" not used');
- CheckUnexpectedMessages;
- end;
- procedure TTestUseAnalyzer.TestM_Hint_ValueParameterIsAssignedButNeverUsed;
- begin
- StartProgram(true);
- Add('procedure DoIt(i: longint);');
- Add('begin');
- Add(' i:=3;');
- Add('end;');
- Add('begin');
- Add(' DoIt(1);');
- AnalyzeProgram;
- CheckHasHint(mtHint,nPAValueParameterIsAssignedButNeverUsed,
- 'Value parameter "i" is assigned but never used');
- CheckUnexpectedMessages;
- end;
- procedure TTestUseAnalyzer.TestM_Hint_LocalVariableIsAssignedButNeverUsed;
- begin
- StartProgram(true);
- Add('procedure DoIt;');
- Add('const');
- Add(' a: longint = 14;');
- Add('var');
- Add(' b: char;');
- Add(' c: longint = 15;');
- Add('begin');
- Add(' a:=16;');
- Add(' b:=#65;');
- Add(' c:=17;');
- Add('end;');
- Add('begin');
- Add(' DoIt;');
- AnalyzeProgram;
- CheckHasHint(mtHint,nPALocalVariableIsAssignedButNeverUsed,
- 'Local variable "a" is assigned but never used');
- CheckHasHint(mtHint,nPALocalVariableIsAssignedButNeverUsed,
- 'Local variable "b" is assigned but never used');
- CheckHasHint(mtHint,nPALocalVariableIsAssignedButNeverUsed,
- 'Local variable "c" is assigned but never used');
- CheckUnexpectedMessages;
- end;
- procedure TTestUseAnalyzer.TestM_Hint_LocalXYNotUsed;
- begin
- StartProgram(true);
- Add('procedure DoIt;');
- Add('type');
- Add(' TColor = longint;');
- Add(' TFlag = (red,green);');
- Add(' TFlags = set of TFlag;');
- Add(' TArrInt = array of integer;');
- Add(' procedure Sub; begin end;');
- Add('begin');
- Add('end;');
- Add('begin');
- Add(' DoIt;');
- AnalyzeProgram;
- CheckHasHint(mtHint,nPALocalXYNotUsed,'Local alias type "TColor" not used');
- CheckHasHint(mtHint,nPALocalXYNotUsed,'Local enumeration type "TFlag" not used');
- CheckHasHint(mtHint,nPALocalXYNotUsed,'Local set type "TFlags" not used');
- CheckHasHint(mtHint,nPALocalXYNotUsed,'Local array type "TArrInt" not used');
- CheckHasHint(mtHint,nPALocalXYNotUsed,'Local procedure "Sub" not used');
- CheckUnexpectedMessages;
- end;
- procedure TTestUseAnalyzer.TestM_Hint_PrivateFieldIsNeverUsed;
- begin
- StartProgram(true,[supTObject]);
- Add('type');
- Add(' TMobile = class');
- Add(' private');
- Add(' a: longint;');
- Add(' end;');
- Add('var m: TMobile;');
- Add('begin');
- Add(' m:=nil;');
- AnalyzeProgram;
- CheckHasHint(mtHint,nPAPrivateFieldIsNeverUsed,
- 'Private field "TMobile.a" is never used');
- CheckHasHint(mtHint,nPALocalVariableIsAssignedButNeverUsed,
- 'Local variable "m" is assigned but never used');
- CheckUnexpectedMessages;
- end;
- procedure TTestUseAnalyzer.TestM_Hint_PrivateFieldIsAssignedButNeverUsed;
- begin
- StartProgram(true,[supTObject]);
- Add('type');
- Add(' TMobile = class');
- Add(' private');
- Add(' a: longint;');
- Add(' public');
- Add(' constructor Create;');
- Add(' end;');
- Add('constructor TMobile.Create;');
- Add('begin');
- Add(' a:=3;');
- Add('end;');
- Add('begin');
- Add(' TMobile.Create;');
- AnalyzeProgram;
- CheckHasHint(mtHint,nPAPrivateFieldIsAssignedButNeverUsed,
- 'Private field "TMobile.a" is assigned but never used');
- CheckUnexpectedMessages;
- end;
- procedure TTestUseAnalyzer.TestM_Hint_PrivateMethodIsNeverUsed;
- begin
- StartProgram(true,[supTObject]);
- Add('type');
- Add(' TMobile = class');
- Add(' private');
- Add(' procedure DoSome; external name ''foo'';');
- Add(' public');
- Add(' constructor Create;');
- Add(' end;');
- Add('constructor TMobile.Create;');
- Add('begin');
- Add('end;');
- Add('begin');
- Add(' TMobile.Create;');
- AnalyzeProgram;
- CheckHasHint(mtHint,nPAPrivateMethodIsNeverUsed,
- 'Private method "TMobile.DoSome" is never used');
- CheckUnexpectedMessages;
- end;
- procedure TTestUseAnalyzer.TestM_Hint_LocalDestructor_No_IsNeverUsed;
- begin
- StartProgram(true,[supTObject]);
- Add('type');
- Add(' TMobile = class');
- Add(' private');
- Add(' public');
- Add(' constructor Create;');
- Add(' destructor Destroy; override;');
- Add(' end;');
- Add('var DestroyCount: longint = 0;');
- Add('constructor TMobile.Create;');
- Add('begin');
- Add('end;');
- Add('destructor TMobile.Destroy;');
- Add('begin');
- Add(' inc(DestroyCount);');
- Add(' inherited;');
- Add('end;');
- Add('var o: TObject;');
- Add('begin');
- Add(' o:=TMobile.Create;');
- Add(' o.Destroy;');
- AnalyzeProgram;
- CheckUnexpectedMessages;
- end;
- procedure TTestUseAnalyzer.TestM_Hint_PrivateTypeNeverUsed;
- begin
- StartProgram(true,[supTObject]);
- Add('type');
- Add(' TMobile = class');
- Add(' private');
- Add(' type t = longint;');
- Add(' public');
- Add(' constructor Create;');
- Add(' end;');
- Add('constructor TMobile.Create;');
- Add('begin');
- Add('end;');
- Add('begin');
- Add(' TMobile.Create;');
- AnalyzeProgram;
- CheckHasHint(mtHint,nPAPrivateTypeXNeverUsed,
- 'Private type "TMobile.t" never used');
- CheckUnexpectedMessages;
- end;
- procedure TTestUseAnalyzer.TestM_Hint_PrivateConstNeverUsed;
- begin
- StartProgram(true,[supTObject]);
- Add('type');
- Add(' TMobile = class');
- Add(' private');
- Add(' const c = 3;');
- Add(' public');
- Add(' constructor Create;');
- Add(' end;');
- Add('constructor TMobile.Create;');
- Add('begin');
- Add('end;');
- Add('begin');
- Add(' TMobile.Create;');
- AnalyzeProgram;
- CheckHasHint(mtHint,nPAPrivateConstXNeverUsed,
- 'Private const "TMobile.c" never used');
- CheckUnexpectedMessages;
- end;
- procedure TTestUseAnalyzer.TestM_Hint_PrivatePropertyNeverUsed;
- begin
- StartProgram(true,[supTObject]);
- Add('type');
- Add(' TMobile = class');
- Add(' private');
- Add(' FA: longint;');
- Add(' property A: longint read FA;');
- Add(' public');
- Add(' constructor Create;');
- Add(' end;');
- Add('constructor TMobile.Create;');
- Add('begin');
- Add('end;');
- Add('begin');
- Add(' TMobile.Create;');
- AnalyzeProgram;
- CheckHasHint(mtHint,nPAPrivatePropertyXNeverUsed,
- 'Private property "TMobile.A" never used');
- CheckHasHint(mtHint,nPAPrivateFieldIsNeverUsed,
- 'Private field "TMobile.FA" is never used');
- CheckUnexpectedMessages;
- end;
- procedure TTestUseAnalyzer.TestM_Hint_LocalClassInProgramNotUsed;
- begin
- StartProgram(true,[supTObject]);
- Add('type');
- Add(' TMobile = class');
- Add(' public');
- Add(' constructor Create;');
- Add(' end;');
- Add('constructor TMobile.Create;');
- Add('begin');
- Add('end;');
- Add('var');
- Add(' m: TMobile;');
- Add('begin');
- AnalyzeProgram;
- CheckHasHint(mtHint,nPALocalXYNotUsed,'Local class "TMobile" not used');
- CheckHasHint(mtHint,nPALocalVariableNotUsed,'Local variable "m" not used');
- CheckUnexpectedMessages;
- end;
- procedure TTestUseAnalyzer.TestM_Hint_LocalMethodInProgramNotUsed;
- begin
- StartProgram(true,[supTObject]);
- Add('type');
- Add(' TMobile = class');
- Add(' public');
- Add(' constructor Create;');
- Add(' end;');
- Add('constructor TMobile.Create;');
- Add('begin');
- Add('end;');
- Add('var');
- Add(' m: TMobile;');
- Add('begin');
- Add(' if m=nil then ;');
- AnalyzeProgram;
- CheckHasHint(mtHint,nPALocalXYNotUsed,'Local constructor "Create" not used');
- CheckUnexpectedMessages;
- end;
- procedure TTestUseAnalyzer.TestM_Hint_AssemblerParameterIgnored;
- begin
- StartProgram(true);
- Add('procedure DoIt(i: longint); assembler;');
- Add('type');
- Add(' {#tcolor_notused}TColor = longint;');
- Add(' {#tflag_notused}TFlag = (red,green);');
- Add(' {#tflags_notused}TFlags = set of TFlag;');
- Add(' {#tarrint_notused}TArrInt = array of integer;');
- Add('const');
- Add(' {#a_notused}a = 13;');
- Add(' {#b_notused}b: longint = 14;');
- Add('var');
- Add(' {#c_notused}c: char;');
- Add(' {#d_notused}d: longint = 15;');
- Add(' procedure {#sub_notused}Sub; begin end;');
- Add('asm end;');
- Add('begin');
- Add(' DoIt(1);');
- AnalyzeProgram;
- CheckUnexpectedMessages;
- end;
- procedure TTestUseAnalyzer.TestM_Hint_FunctionResultDoesNotSeemToBeSet;
- begin
- StartProgram(true);
- Add('function DoIt: longint;');
- Add('begin end;');
- Add('begin');
- Add(' DoIt();');
- AnalyzeProgram;
- CheckHasHint(mtHint,nPAFunctionResultDoesNotSeemToBeSet,
- sPAFunctionResultDoesNotSeemToBeSet);
- CheckUnexpectedMessages;
- end;
- procedure TTestUseAnalyzer.TestM_Hint_FunctionResultDoesNotSeemToBeSet_Abstract;
- begin
- StartProgram(true);
- Add('type');
- Add(' TObject = class');
- Add(' class function DoIt: longint; virtual; abstract;');
- Add(' end;');
- Add('begin');
- Add(' TObject.DoIt;');
- AnalyzeProgram;
- CheckUnexpectedMessages;
- end;
- procedure TTestUseAnalyzer.TestM_Hint_FunctionResultRecord;
- begin
- StartProgram(true);
- Add('type');
- Add(' TPoint = record X,Y:longint; end;');
- Add('function Point(Left: longint): TPoint;');
- Add('begin');
- Add(' Result.X:=Left;');
- Add('end;');
- Add('begin');
- Add(' Point(1);');
- AnalyzeProgram;
- CheckHasHint(mtHint,nPALocalVariableIsAssignedButNeverUsed,
- 'Local variable "X" is assigned but never used');
- CheckHasHint(mtHint,nPALocalVariableNotUsed,'Local variable "Y" not used');
- CheckUnexpectedMessages;
- end;
- procedure TTestUseAnalyzer.TestM_Hint_FunctionResultPassRecordElement;
- begin
- StartProgram(true);
- Add('type');
- Add(' TPoint = record X,Y:longint; end;');
- Add('procedure Three(out x: longint);');
- Add('begin');
- Add(' x:=3;');
- Add('end;');
- Add('function Point(): TPoint;');
- Add('begin');
- Add(' Three(Result.X)');
- Add('end;');
- Add('begin');
- Add(' Point();');
- AnalyzeProgram;
- CheckHasHint(mtHint,nPALocalVariableNotUsed,'Local variable "Y" not used');
- CheckUnexpectedMessages;
- end;
- procedure TTestUseAnalyzer.TestM_Hint_OutParam_No_AssignedButNeverUsed;
- begin
- StartProgram(true);
- Add('procedure DoIt(out x: longint);');
- Add('begin');
- Add(' x:=3;');
- Add('end;');
- Add('var i: longint;');
- Add('begin');
- Add(' DoIt(i);');
- AnalyzeProgram;
- CheckUnexpectedMessages;
- end;
- procedure TTestUseAnalyzer.TestWP_LocalVar;
- begin
- StartProgram(false);
- Add('var {#a_notused}a: longint;');
- Add('var {#b_used}b: longint;');
- Add('var {#c_used}c: longint;');
- Add('begin');
- Add(' b:=2;');
- Add(' afile.c:=3;');
- AnalyzeWholeProgram;
- end;
- procedure TTestUseAnalyzer.TestWP_UnitUsed;
- begin
- AddModuleWithIntfImplSrc('unit2.pp',
- LinesToStr([
- 'var i: longint;',
- 'procedure DoIt;',
- '']),
- LinesToStr([
- 'procedure DoIt; begin end;']));
- StartProgram(true);
- Add('uses unit2;');
- Add('begin');
- Add(' i:=3;');
- AnalyzeWholeProgram;
- CheckUnitUsed('unit2.pp',true);
- end;
- procedure TTestUseAnalyzer.TestWP_UnitNotUsed;
- begin
- AddModuleWithIntfImplSrc('unit2.pp',
- LinesToStr([
- 'var i: longint;',
- 'procedure DoIt;',
- '']),
- LinesToStr([
- 'procedure DoIt; begin end;']));
- StartProgram(true);
- Add('uses unit2;');
- Add('begin');
- AnalyzeWholeProgram;
- CheckUnitUsed('unit2.pp',false);
- end;
- procedure TTestUseAnalyzer.TestWP_UnitInitialization;
- begin
- AddModuleWithIntfImplSrc('unit1.pp',
- LinesToStr([
- 'uses unit2;',
- '']),
- LinesToStr([
- 'initialization',
- 'i:=2;']));
- AddModuleWithIntfImplSrc('unit2.pp',
- LinesToStr([
- 'var i: longint;',
- '']),
- LinesToStr([
- '']));
- StartProgram(true);
- Add('uses unit1;');
- Add('begin');
- AnalyzeWholeProgram;
- CheckUnitUsed('unit1.pp',true);
- CheckUnitUsed('unit2.pp',true);
- end;
- procedure TTestUseAnalyzer.TestWP_UnitFinalization;
- begin
- AddModuleWithIntfImplSrc('unit1.pp',
- LinesToStr([
- 'uses unit2;',
- '']),
- LinesToStr([
- 'finalization',
- 'i:=2;']));
- AddModuleWithIntfImplSrc('unit2.pp',
- LinesToStr([
- 'var i: longint;',
- '']),
- LinesToStr([
- '']));
- StartProgram(true);
- Add('uses unit1;');
- Add('begin');
- AnalyzeWholeProgram;
- CheckUnitUsed('unit1.pp',true);
- CheckUnitUsed('unit2.pp',true);
- end;
- procedure TTestUseAnalyzer.TestWP_CallInherited;
- begin
- StartProgram(false);
- Add('type');
- Add(' {#TObject_used}TObject = class');
- Add(' procedure {#TObjectDoA_used}DoA;');
- Add(' procedure {#TObjectDoB_used}DoB;');
- Add(' end;');
- Add(' {#TMobile_used}TMobile = class');
- Add(' procedure {#TMobileDoA_used}DoA;');
- Add(' procedure {#TMobileDoC_used}DoC;');
- Add(' end;');
- Add('procedure TObject.DoA; begin end;');
- Add('procedure TObject.DoB; begin end;');
- Add('procedure TMobile.DoA;');
- Add('begin');
- Add(' inherited;');
- Add('end;');
- Add('procedure TMobile.DoC;');
- Add('begin');
- Add(' inherited DoB;');
- Add('end;');
- Add('var o: TMobile;');
- Add('begin');
- Add(' o.DoA;');
- Add(' o.DoC;');
- AnalyzeWholeProgram;
- end;
- procedure TTestUseAnalyzer.TestWP_ProgramPublicDeclarations;
- begin
- StartProgram(false);
- Add('var');
- Add(' {#vPublic_used}vPublic: longint; public;');
- Add(' {#vPrivate_notused}vPrivate: longint;');
- Add('procedure {#DoPublic_used}DoPublic; public; begin end;');
- Add('procedure {#DoPrivate_notused}DoPrivate; begin end;');
- Add('begin');
- AnalyzeWholeProgram;
- end;
- procedure TTestUseAnalyzer.TestWP_ClassDefaultProperty;
- begin
- StartProgram(false);
- Add('type');
- Add(' {#tobject_used}TObject = class');
- Add(' function {#getitems_notused}Getitems(Index: longint): string;');
- Add(' procedure {#setitems_used}Setitems(Index: longint; Value: String);');
- Add(' property {#items_used}Items[Index: longint]: string read GetItems write SetItems; default;');
- Add(' end;');
- Add('function TObject.Getitems(Index: longint): string; begin end;');
- Add('procedure TObject.Setitems(Index: longint; Value: String); begin end;');
- Add('var');
- Add(' {#l_used}L: TObject;');
- Add('begin');
- Add(' L[0]:=''birdy'';');
- AnalyzeWholeProgram;
- end;
- procedure TTestUseAnalyzer.TestWP_Published;
- begin
- StartProgram(false);
- Add('type');
- Add(' {#tobject_used}TObject = class');
- Add(' private');
- Add(' {#fcol_used}FCol: string;');
- Add(' {#fbird_notused}FBird: string;');
- Add(' published');
- Add(' {#fielda_used}FieldA: longint;');
- Add(' procedure {#doit_used}ProcA; virtual; abstract;');
- Add(' property {#col_used}Col: string read FCol;');
- Add(' end;');
- Add('var');
- Add(' {#o_used}o: TObject;');
- Add('begin');
- Add(' o:=nil;');
- AnalyzeWholeProgram;
- end;
- procedure TTestUseAnalyzer.TestWP_PublishedSetType;
- begin
- StartProgram(false);
- Add('type');
- Add(' {#tflag_used}TFlag = (red, green);');
- Add(' {#tflags_used}TFlags = set of TFlag;');
- Add(' {#tobject_used}TObject = class');
- Add(' published');
- Add(' {#fielda_used}FieldA: TFlag;');
- Add(' {#fieldb_used}FieldB: TFlags;');
- Add(' end;');
- Add('var');
- Add(' {#o_used}o: TObject;');
- Add('begin');
- Add(' o:=nil;');
- AnalyzeWholeProgram;
- end;
- procedure TTestUseAnalyzer.TestWP_PublishedArrayType;
- begin
- StartProgram(false);
- Add('type');
- Add(' {#tdynarr_used}TDynArr = array of longint;');
- Add(' {#tstatarr_used}TStatArr = array[boolean] of longint;');
- Add(' {#tobject_used}TObject = class');
- Add(' published');
- Add(' {#fielda_used}FieldA: TDynArr;');
- Add(' {#fieldb_used}FieldB: TStatArr;');
- Add(' end;');
- Add('var');
- Add(' {#o_used}o: TObject;');
- Add('begin');
- Add(' o:=nil;');
- AnalyzeWholeProgram;
- end;
- procedure TTestUseAnalyzer.TestWP_PublishedClassOfType;
- begin
- StartProgram(false);
- Add('type');
- Add(' {#tobjectclass_used}TObjectClass = class of TObject;');
- Add(' {#tobject_used}TObject = class');
- Add(' published');
- Add(' {#fielda_used}FieldA: TObjectClass;');
- Add(' end;');
- Add(' {#tclass_used}TClass = class of TObject;');
- Add('var');
- Add(' {#c_used}c: TClass;');
- Add('begin');
- Add(' c:=nil;');
- AnalyzeWholeProgram;
- end;
- procedure TTestUseAnalyzer.TestWP_PublishedRecordType;
- begin
- StartProgram(false);
- Add('type');
- Add(' {#trec_used}TRec = record');
- Add(' {treci_used}i: longint;');
- Add(' end;');
- Add(' {#tobject_used}TObject = class');
- Add(' published');
- Add(' {#fielda_used}FieldA: TRec;');
- Add(' end;');
- Add('var');
- Add(' {#o_used}o: TObject;');
- Add('begin');
- Add(' o:=nil;');
- AnalyzeWholeProgram;
- end;
- procedure TTestUseAnalyzer.TestWP_PublishedProcType;
- begin
- StartProgram(false);
- Add('type');
- Add(' {#ta_used}ta = array of longint;');
- Add(' {#tb_used}tb = array of longint;');
- Add(' {#tproca_used}TProcA = procedure;');
- Add(' {#tfunca_used}TFuncA = function: ta;');
- Add(' {#tprocb_used}TProcB = procedure(a: tb);');
- Add(' {#tobject_used}TObject = class');
- Add(' published');
- Add(' {#fielda_used}FieldA: TProcA;');
- Add(' {#fieldb_used}FieldB: TFuncA;');
- Add(' {#fieldc_used}FieldC: TProcB;');
- Add(' end;');
- Add('var');
- Add(' {#o_used}o: TObject;');
- Add('begin');
- Add(' o:=nil;');
- AnalyzeWholeProgram;
- end;
- procedure TTestUseAnalyzer.TestWP_PublishedProperty;
- begin
- StartProgram(false);
- Add('const');
- Add(' {#defcol_used}DefCol = 3;');
- Add(' {#defsize_notused}DefSize = 43;');
- Add('type');
- Add(' {#tobject_used}TObject = class');
- Add(' private');
- Add(' {#fcol_used}FCol: longint;');
- Add(' {#fsize_used}FSize: longint;');
- Add(' {#fbird_notused}FBird: string;');
- Add(' {#fcolstored_used}FColStored: boolean;');
- Add(' {#fsizestored_notused}FSizeStored: boolean;');
- Add(' public');
- Add(' property {#size_used}Size: longint read FSize stored FSizeStored default DefSize;');
- Add(' published');
- Add(' property {#col_used}Col: longint read FCol stored FColStored default DefCol;');
- Add(' end;');
- Add('var');
- Add(' {#o_used}o: TObject;');
- Add('begin');
- Add(' if o.Size=13 then ;');
- AnalyzeWholeProgram;
- end;
- initialization
- RegisterTests([TTestUseAnalyzer]);
- end.
|