Compiler warnings about ocvwrapper46

[gidesa]

Hello,
the first warning on mt is clearly ininfluent.
With my compiler (32bit) there is no warning on signed/unsigned.
The others warnings complain from some unitialized variables, but they are always initialized, in instructions sequence out of conditionals, and then always executed. So compiler is too much restrictive.
For example: vecBoxes initialized in vecBoxes:=pCvVectorRect2dCreate(detections.Count);
vecNmsIndex in vecNmsIndex:=pCvVectorintCreate(0);
and so on.

[hafedh-trimeche]

Hi,

vecNmsIndex must be defined outside Try Finally block because if an exception is raised, vecNmsIndex has no value and would generate another exception when calling pCvVectorintDelete(vecNmsIndex);

dptr := dptr + outW*SizeOf(single);
Change to
dptr := dptr + UInt64(outW)*SizeOf(single);

 Assert((nchannels=1) or (nchannels=3),'Channels number must be 1, 2 or 3');
 case nchannels of
  1:  mt:=CV_8UC1;
  2:  mt:=CV_8UC2;
  3:  mt:=CV_8UC3;
 end;

Change to

  Assert((nchannels>=1) and (nchannels<=3),'Channels number must be 1, 2 or 3');
  mt:=CV_8UC1;
  case nchannels of
   2:  mt:=CV_8UC2;
   3:  mt:=CV_8UC3;
  end;

vecBoxes, classIdPoint and vecScores same as vecNmsIndex

img must be set to nil before the Try Finally Block.

Best regards.

[gidesa]

Hello, regarding your good advise to reorganize try ... finally blocks, I released the corrected version of unOpencvDelphi46.pas.
Regards