I am looking at version 1.2 (the latest one). If you are using a different one, please say so.
If it really stops on that return statement, then there is probably a really ugly logic issue where subroutines are not cleanly returned.
There is a limit to how many gosub/return pairs can be left open. I do not know the number, but it definitely has a limit. Maybe this is being reached.
There is a known issue with your character name, where you need to filter out Lord/Lady if your fame reaches that level that has shown up in other scripts. If your character is Lord/Lady... maybe that needs to be changed, but I would expect it to fail earlier.
I would be curious about some variable values when it is looping:
!temp_jindex
!temp_index
%0
%1
%2
%3
%4
#menubutton
There are 2 routines that use TM_AdvJournalScan: gather and movebowls_button. The movebowls_button sub looks pretty clean and will tell you that your container is full, if it runs into an issue dropping your bowls into a secure.
The gather routine is collecting from plants.
So which is it trying to do? move bowls to your secure or gather plant resources?
Gaderian