Re-use AllocFileRequest()

5 posts / 0 new

Last post

Sun, 2014-11-09 02:14

mritter0

Offline

Last seen: 2 years 5 months ago

Joined: 2014-04-21 21:15

Re-use AllocFileRequest()

I have created an Asl requester, FileReq=AllocFileRequest(), and use it with AslRequestTags().

All is working just fine. I initialize it once, use it several times, then FreeFileRequest(FileReq); at program quit.

It is working fine, but is this a good idea? Should I allocate it every time before opening the requester and free it right after?

Sun, 2014-11-09 13:54

jabirulo

Offline

Last seen: 2 weeks 5 days ago

Joined: 2013-05-30 00:53

What I do on MIXER (when user

What I do on MIXER (when user chooses 'Change skin...' menu option) is to call this function (and yes, it opens/allocates and frees ASL every time):

BOOL ChangeSkin(void)
{
 STRPTR initdrw = IExec->AllocVecTags(1024, TAG_DONE);
 BOOL resul;
 struct FileRequester *myfilreq = NULL;
 
 IUtility->Strlcpy(initdrw, (STRPTR)ARG("IMAGEPATH"), 1024);
 //STRPTR fileptr = IDOS->PathPart(initdrw);
 //*(fileptr) = '\0';
 
 myfilreq = IAsl->AllocAslRequest(ASL_FileRequest, NULL);
 
 resul = IAsl->AslRequestTags(myfilreq, 
                              ASLFR_TitleText, GetString(&li, MSG_LOAD_TITLE), 
                              ASLFR_DrawersOnly, TRUE, 
                              ASLFR_InitialDrawer, initdrw, 
                              ASLFR_Screen, MixWindow->WScreen, 
                             TAG_END);
 if(resul)
 {
  if(IUtility->Stricmp((STRPTR)ARG("IMAGEPATH"),myfilreq->fr_Drawer) != 0)
   IUtility->Strlcpy( (STRPTR)ARG("IMAGEPATH"), myfilreq->fr_Drawer, 1024 );
  else
   resul = FALSE;
 }
 
 IAsl->FreeAslRequest(myfilreq);
 IExec->FreeVec(initdrw);
 
 return resul;
}

AOS4.1/SAM460ex/PPC460EX-1155MHZ/2048MB/RadeonHD6570/SSD120GB/DVDRW :-P

Sun, 2014-11-09 19:08

salass00

Offline

Last seen: 6 months 1 week ago

Joined: 2011-02-03 11:27

Yes, it is correct and in

Yes, it is correct and in fact recommended to reuse a filereq because that way it will automatically remember what file/directory was selected last.

If you're program supports loading and saving I suggest you use a different one with ASLFR_DoSaveMode set to TRUE for saving. You could reuse the same one setting ASLFR_DoSaveMode in AslRequestTags() to TRUE or FALSE depending on what's needed but it can sometimes be useful to be able to load files from one directory, edit them and then save them to another one without having to change the current directory for the requester every time.

Fri, 2014-11-14 12:04

(Reply to #3) #4

OldFart

Offline

Last seen: 3 months 1 week ago

Joined: 2010-11-30 14:09

@Jabirulo In your code you

@Jabirulo

In your code you make use of functions which return a result, like IExec->AllocVecTags() and IAsl->AllocAslRequest(). You would do well in verifying the result of those functions and only take further actions when the results are 'to your liking' i.e. being non-NULL.

The code as supplied "may" crash, due to f.i. IExec->AllocVecTags() not being able to allocate the required amount of memory, yet you proceed as if it had done so, thereby possibly referencing a NULL-pointer, which will lead to a crash.

OldFart

Sat, 2014-11-15 12:24

(Reply to #4) #5

jabirulo

Offline

Last seen: 2 weeks 5 days ago

Joined: 2013-05-30 00:53

I'm a but lazy/newbie/bad

I'm a but lazy/newbie/bad coder and just thought 1KB to alloc won't fail on today AOS4.1 systems ;-) it's a rodiculous amount of memory (I could make it STRPTR initdrw[1024]; and avoid all those mini allocations).

But anyway THX for suggestions.

AOS4.1/SAM460ex/PPC460EX-1155MHZ/2048MB/RadeonHD6570/SSD120GB/DVDRW :-P

Search form

User login