Re: [Oorexx-devel] RexxQueue class

2008-08-19 Thread Sahananda (Jon) Wolfers
-devel@lists.sourceforge.net> > > > To > > "Open Object Rexx Developer Mailing List" < > oorexx-devel@lists.sourceforge.net> > cc > > > Subject > > Re: [Oorexx-devel] RexxQueue class > > Hi Mark, > > thanks for the long explanation

Re: [Oorexx-devel] RexxQueue class

2008-08-19 Thread David Ashley
cc Subject 08/19/2008 09:15 Re: [Oorexx-devel] RexxQ

Re: [Oorexx-devel] RexxQueue class

2008-08-19 Thread Mark Miesfeld
On Tue, Aug 19, 2008 at 7:03 AM, Sahananda (Jon) Wolfers <[EMAIL PROTECTED]> wrote: > it looks like I didn't properly understand what Rick told me. Jon, I think you pretty much got it. Just a little confusion on the naming maybe. But the code looks okay. I just pasted your changes into my work

Re: [Oorexx-devel] RexxQueue class

2008-08-19 Thread Sahananda (Jon) Wolfers
Hi Mark, thanks for the long explanation. It does look like I made a mistake. I think I also made a mistake in reporting the revision I am at. I right clicked on the repository root and it reports 2940, but when I right click on trunk it reports 3014. I do do an SVN update before each compile,

Re: [Oorexx-devel] RexxQueue class

2008-08-19 Thread Mark Miesfeld
On Tue, Aug 19, 2008 at 12:16 AM, Sahananda (Jon) Wolfers <[EMAIL PROTECTED]> wrote: > > I am at revision 2940. You really need to keep the source tree you are working with up to date. SVN will not overwrite any lines of code you have changed. It will either merge in the updated code with your c

Re: [Oorexx-devel] RexxQueue class

2008-08-19 Thread Sahananda (Jon) Wolfers
Thanks Mark, it looks like I didn't properly understand what Rick told me. Jon 2008/8/19 Mark Miesfeld <[EMAIL PROTECTED]> > On Tue, Aug 19, 2008 at 5:07 AM, Mark Miesfeld <[EMAIL PROTECTED]> wrote: > > > >> + INTERNAL_METHOD(rexx_erase_queue) > > > >> +RexxMethod0(int, rexx_clear_queue) > >

Re: [Oorexx-devel] RexxQueue class

2008-08-19 Thread Mark Miesfeld
On Tue, Aug 19, 2008 at 5:07 AM, Mark Miesfeld <[EMAIL PROTECTED]> wrote: > >> + INTERNAL_METHOD(rexx_erase_queue) > >> +RexxMethod0(int, rexx_clear_queue) > >> +::METHOD empty EXTERNAL 'LIBRARY REXX rexx_clear_queue' > > You can not define the name as rexx_erase_queue in one place and

Re: [Oorexx-devel] RexxQueue class

2008-08-19 Thread Mark Miesfeld
Jon, I haven't looked very closely at this, but you need to use the same name throughout. > + INTERNAL_METHOD(rexx_erase_queue) > +RexxMethod0(int, rexx_clear_queue) > +::METHOD empty EXTERNAL 'LIBRARY REXX rexx_clear_queue' You can not define the name as rexx_erase_queue in one pl

Re: [Oorexx-devel] RexxQueue class

2008-08-18 Thread Mark Miesfeld
On Mon, Aug 18, 2008 at 1:41 PM, Rick McGuire <[EMAIL PROTECTED]> wrote: > This sounds like it might be the same problem that Moritz is seeing, > which is likely a GC problem somewhere. I just went back and looked at that thread and it is the same thing I think. The top of his stack looks the sam

Re: [Oorexx-devel] RexxQueue class

2008-08-18 Thread Rick McGuire
This sounds like it might be the same problem that Moritz is seeing, which is likely a GC problem somewhere. You might want to take a look at the post where I gave Moritz some advice on what to try looking at in terms of break points. The same information would be helpful here. I might have a 64

Re: [Oorexx-devel] RexxQueue class

2008-08-18 Thread Mark Miesfeld
On Mon, Aug 18, 2008 at 1:24 PM, Rick McGuire <[EMAIL PROTECTED]> wrote: > At the moment, I probably can only address windows problems unless > it's something that I can just eyeball. Other than that, I'm just > spending some time trying to implement a number of long sought > enhancements. I'll p

Re: [Oorexx-devel] RexxQueue class

2008-08-18 Thread Rick McGuire
At the moment, I probably can only address windows problems unless it's something that I can just eyeball. Other than that, I'm just spending some time trying to implement a number of long sought enhancements. I'll probably also have much less time to devote to this after today. Rick On Mon, Au

Re: [Oorexx-devel] RexxQueue class

2008-08-18 Thread Mark Miesfeld
On Mon, Aug 18, 2008 at 11:58 AM, Rick McGuire <[EMAIL PROTECTED]> wrote: > I'm having a relaxing vaction today by working on ooRexx code That is the thing that I noticed. Not sure if I should start posting *interesting* problems or wait until your trip is over. -- Mark Miesfeld

Re: [Oorexx-devel] RexxQueue class

2008-08-18 Thread Rick McGuire
On Mon, Aug 18, 2008 at 2:41 PM, Sahananda (Jon) Wolfers <[EMAIL PROTECTED]> wrote: > Thanks Rick, > > I'm not sure I understand that instruction. > The only internalpackage.cpp I have is in main\trunk\interpreter\runtime. > If I don't delete it I get > > link /MAP /NOLOGO /PROFILE /DEBUG -debu

Re: [Oorexx-devel] RexxQueue class

2008-08-18 Thread Sahananda (Jon) Wolfers
Thanks Rick, I'm not sure I understand that instruction. The only internalpackage.cpp I have is in main\trunk\interpreter\runtime. If I don't delete it I get link /MAP /NOLOGO /PROFILE /DEBUG -debugtype:cv /SUBSYSTEM:Windows user32.lib gdi32.lib winspool.lib comdlg32.lib advapi32.lib delayimp

Re: [Oorexx-devel] RexxQueue class

2008-08-17 Thread Rick McGuire
Hi Jon, My hotel has a WiFi connection, so I was able to check email this morning. Your guess was right, adding that INTERNAL_METHOD call is the missing piece needed to make that work. You'll need to force the module InternalPackage.cpp to recompile. Just erasing Win32XXX\InternalPackage.cpp and

Re: [Oorexx-devel] RexxQueue class

2008-08-17 Thread Sahananda (Jon) Wolfers
I don't have time to test this right now, but I think the answer might be that I need to add a line to main\trunk\interpreter\runtime\nativemethods.h reading INTERNAL_METHOD(rexx_clear_queue) I will try to get time to try it out tonight. Jon 2008/8/17 Sahananda (Jon) Wolfers <[EMAIL PROTECT

Re: [Oorexx-devel] RexxQueue class

2008-08-17 Thread Sahananda (Jon) Wolfers
Hi, I am in trouble again. I added Rick's code (as below) to main\trunk\interpreter\classes\RexxQueueMethods.cpp at the end where it seems to belong and added the call to it to the rexxQueue methods in main\trunk\interpreter\RexxClasses\streamClasses.orx. I've updated the SVN branch at trunk. W

Re: [Oorexx-devel] RexxQueue class

2008-08-13 Thread Sahananda (Jon) Wolfers
OK, thanks for your help RIck, I'll get on with this over the next few days. Once again I'm on site, so it may be a while before I get a chance to post. Jon 2008/8/13 Gil Barmwater <[EMAIL PROTECTED]> > See inline comments - > > Rick McGuire wrote: > > On Wed, Aug 13, 2008 at 9:02 AM, Gil Barm

Re: [Oorexx-devel] RexxQueue class

2008-08-13 Thread Gil Barmwater
See inline comments - Rick McGuire wrote: > On Wed, Aug 13, 2008 at 9:02 AM, Gil Barmwater <[EMAIL PROTECTED]> wrote: > >>Just some thoughts on this thread which I have been following with some >>interest. The first is a question: when did the the RexxQueue class >>appear or has it always been t

Re: [Oorexx-devel] RexxQueue class

2008-08-13 Thread Rick McGuire
On Wed, Aug 13, 2008 at 9:02 AM, Gil Barmwater <[EMAIL PROTECTED]> wrote: > Just some thoughts on this thread which I have been following with some > interest. The first is a question: when did the the RexxQueue class > appear or has it always been there and I just didn't notice it? It > appears

Re: [Oorexx-devel] RexxQueue class

2008-08-13 Thread Gil Barmwater
Just some thoughts on this thread which I have been following with some interest. The first is a question: when did the the RexxQueue class appear or has it always been there and I just didn't notice it? It appears to be an OO way - objects and methods - of dealing with the external data queu

Re: [Oorexx-devel] RexxQueue class

2008-08-13 Thread Rick McGuire
My intent for the makearray method was for it to remove the lines from queue, not leave them in. The intent here is to be able to use a DO OVER loop to iterate over the lines in the queue. So remove the line that put it back. I'd also recommend making just a single call to queued with a temporar

Re: [Oorexx-devel] RexxQueue class

2008-08-12 Thread Sahananda (Jon) Wolfers
The approach I am thinking of taking for the makearray is this: /* - */ ::method makearray guarded /* - */ arr = .array~new(self~queued) do i = 1 to self~

Re: [Oorexx-devel] RexxQueue class

2008-08-12 Thread Rick McGuire
Ok, it can be a bit confusing...this is largely because the RexxQueue class serves a dual role. Originally, RexxQueue was the backing implementation for the PUSH and QUEUE instructions and the entity accessed by the RXQUEUE builtin function. In this role, the name of the queue backing the object

Re: [Oorexx-devel] RexxQueue class

2008-08-12 Thread Sahananda (Jon) Wolfers
I have reverted to my former position. There is something I don't understand about the RexxQueue class. It seems counterintuitive to me. Something in the area of the distinction between the name of the Queue and the instance of the class. The new method returns an instance that has a string valu

Re: [Oorexx-devel] RexxQueue class

2008-08-11 Thread Rick McGuire
busted!! :-) Rick On Mon, Aug 11, 2008 at 6:53 AM, Sahananda (Jon) Wolfers <[EMAIL PROTECTED]> wrote: > Thanks Rick, > > I will look into it! This is just a cunning way to show me that > learning c++ would be fun isn't it. > > Jon > > 2008/8/11 Rick McGuire <[EMAIL PROTECTED]>: >> John, >>

Re: [Oorexx-devel] RexxQueue class

2008-08-11 Thread Sahananda (Jon) Wolfers
Thanks Rick, I will look into it! This is just a cunning way to show me that learning c++ would be fun isn't it. Jon 2008/8/11 Rick McGuire <[EMAIL PROTECTED]>: > John, > > Note that in 4.0, the clear method (probably should be "empty" to be > consistent with the other collection classes) would

Re: [Oorexx-devel] RexxQueue class

2008-08-11 Thread Rick McGuire
John, Note that in 4.0, the clear method (probably should be "empty" to be consistent with the other collection classes) would be best implemented by C code using the RexxClearQueue() API I just added. The file in question is interpreter/classes/RexxQueueMethods.cpp. The code would be basically t

Re: [Oorexx-devel] RexxQueue class

2008-08-11 Thread Sahananda (Jon) Wolfers
Please ignore the above. I found the bit of the manual. I can now create my own code samples. thanks Jon 2008/8/11 Sahananda (Jon) Wolfers <[EMAIL PROTECTED]>: > I could take on the rfe's for the RexxQueue class. Although the base > methods are written in C++ it looks like I could add MareArray

[Oorexx-devel] RexxQueue class

2008-08-10 Thread Sahananda (Jon) Wolfers
I could take on the rfe's for the RexxQueue class. Although the base methods are written in C++ it looks like I could add MareArray and Clear in Rexx in STreamClasses.orx. Only drawback is that I don't know what a Rexx External Queue is. I understand Queues (ie: the Queue Collection class) and f