View Issue Details

IDProjectCategoryView StatusLast Update
0002709MMW v4Properties/Auto-Toolspublic2006-12-22 19:11
Reporterjiri Assigned To 
PriorityimmediateSeverityminorReproducibilityalways
Status closedResolutionfixed 
Product Version2.5.4 
Fixed in Version2.5.5 
Summary0002709: .fla extension isn't supported
DescriptionSometimes FLAC uses .fla extension, but it isn't supported in MM. We should add it to f_flac.dll. However, make sure that it doesn't fail on non-FLAC files with this extension.
TagsNo tags attached.
Fixed in build

Activities

peke

2006-12-01 00:51

developer   ~0008212

Last edited: 2006-12-01 00:53

Added support and tested accordingly to description and tested by renaming various type of files in either .FLAC or .FLA and then scanning them in MediaMonkey.

jiri

2006-12-01 14:34

administrator   ~0008213

I don't understand how could it possibly work? I see that you just added another 'FLAC' line to sources:

  OUR_FORMAT2 : TFORMAT_INFO = (extension:'FLAC'; description:'FLAC Audio'; flags:0);

How do you let know MM that '.FLA' is supported?

peke

2006-12-02 00:31

developer   ~0008220

Ok It must be Typo, I have Reposted and F_FLAC.DPR and saved then again so that SVN catches it as modified.

It should stay OUR_FORMAT2 : TFORMAT_INFO = (extension:'FLA'; description:'FLAC Audio'; flags:0);

jiri

2006-12-02 16:20

administrator   ~0008221

I still don't think it's ok, what about FORMAT_CanHandle() function, it should support both extensions too.

peke

2006-12-02 22:18

developer   ~0008222

Mine Line 1177 Contains:
If (Not (S = Our_Format.extension)) And (Not (S = Our_Format2.extension)) Then

With both formats/extension.

It looks that I have some problems in SVN Cache.
I've resynchronized it to be 100 sure and indeed File is modified.
It's OK now.

rusty

2006-12-21 22:33

administrator   ~0008337

I created .fla files but there's no way to scan them into MM. i.e. the scan folders dialog doesn't have an option to select the .fla extension.

rusty

2006-12-21 22:33

administrator   ~0008338

Setting as urgent because this was supposed to be included with MM 2.5.5, however, if there's risk, we can defer.

peke

2006-12-22 00:41

developer   ~0008339

.FLA is not supported as input plugin for FLAC do not have in list of supported
General->File Types. Internally F_FLAC Support it but MM do not have Input plugin allocated to .FLA.
I have changed in_flac.c to include .FLA in listing.

Assigning to Jiri to check if I change it correctly.

jiri

2006-12-22 10:37

administrator   ~0008342

Ok, included.

rusty

2006-12-22 19:11

administrator   ~0008345

Verified 2.5.5.988.