Selftest reports "Use of uninitialized value" for WriteQuickTime.pl

Started by obetz, June 09, 2021, 02:09:50 PM

Previous topic - Next topic

obetz

Hi Phil,

in my package, the automated selftest outputs t/QuickTime.t ...... 6/17 Use of uninitialized value $format in hash element at lib/Image/ExifTool/WriteQuickTime.pl line 320.
Use of uninitialized value $format in hash element at lib/Image/ExifTool/WriteQuickTime.pl line 320.


Should I care about this?

Oliver

Phil Harvey

Hi Oliver,

Thanks.  I will fix this in the next release.  It shouldn't cause problems.

- Phil

Edit: Hmm.  I can't reproduce this, but I will add a patch to avoid this warning.
...where DIR is the name of a directory/folder containing the images.  On Mac/Linux/PowerShell, use single quotes (') instead of double quotes (") around arguments containing a dollar sign ($).

obetz

Quote from: Phil Harvey on June 09, 2021, 09:10:34 PM
Edit: Hmm.  I can't reproduce this, but I will add a patch to avoid this warning.

could this difference result from the Perl environment I'm using?

Maybe some missing component?

Although I did thorough tests when I assembled the environment, I never can catch all paths through ExifTool so I might miss something.

Phil Harvey

Hi Oliver,

I don't see how the Perl environment could affect this.

It would be caused by a writable QuickTime tag with a Writable type that isn't in my %qtFormat lookup, and that doesn't have an overriding Format value.  This would be my oversight, but I couldn't find a tag like this when I looked quickly.  It will default to 'undef' format when writing, which is correct, and the warning shouldn't be necessary.  I have added a patch which should take care of this warning in 12.28.

- Phil
...where DIR is the name of a directory/folder containing the images.  On Mac/Linux/PowerShell, use single quotes (') instead of double quotes (") around arguments containing a dollar sign ($).

obetz

Quote from: Phil Harvey on June 11, 2021, 09:12:03 AM
I don't see how the Perl environment could affect this.

Thanks for the information, I just wanted to be sure there was no need for action on my part.

Oliver