Project

General

Profile

Activity

From 02/13/2015 to 03/14/2015

03/13/2015

05:53 PM Database Bug #2524: missing support for NEXT-SIBLING attribute for buffer handles
Eric Faulhaber wrote:
> Code review 0313a:
>
> * The change to @RecordBuffer.getLDBName@ is OK to leave, but it w...
Ovidiu Maxiniuc
04:46 PM Database Bug #2524: missing support for NEXT-SIBLING attribute for buffer handles
Code review 0313a:
* The change to @RecordBuffer.getLDBName@ is OK to leave, but it will necessitate runtime regre...
Eric Faulhaber
04:23 PM Database Bug #2524: missing support for NEXT-SIBLING attribute for buffer handles
The attached update is based on 10810 revision from bzr.
The @interlink@ method specialized in @BufferImpl@ will han...
Ovidiu Maxiniuc
03:25 PM Database Bug #2524: missing support for NEXT-SIBLING attribute for buffer handles
Ovidiu, please merge the 0303c update up to the latest revision of P2J. Eric Faulhaber
05:48 PM Database Bug #2535: enhancing datatype conversion functions
Code review 0313b:
Just thought of one edge case which might be problematic: consider a WHERE clause with a singl...
Eric Faulhaber
05:33 PM Database Bug #2535: enhancing datatype conversion functions
Logic that was incorrectly placed in @index_select.rules@ moved to @where_clause_prep.rules@. Ovidiu Maxiniuc
03:56 PM Database Bug #2535: enhancing datatype conversion functions
Code review 0311a:
> I am not sure I wrote the function call annotation in the right place (@index_selection.rules@) ...
Eric Faulhaber
05:24 PM User Interface Feature #1790: implement menu support
Vadim Gindin wrote:
> The bottom and right borders of sub-menu is not drawn. I've debugged it and found that it beca...
Constantin Asofiei
05:19 PM User Interface Feature #1790: implement menu support
Vadim Gindin wrote:
> The bottom and right borders of sub-menu is not drawn. I've debugged it and found that it beca...
Constantin Asofiei
05:16 PM User Interface Feature #1790: implement menu support
The bottom and right borders of sub-menu is not drawn. I've debugged it and found that it because @ScreenBitmap@ acti... Vadim Gindin
12:26 PM User Interface Feature #1790: implement menu support
> I think it will be not suitable to use DropDown and ComboBox classes for menu goals as a common mechanism.
I agr...
Greg Shah
11:28 AM User Interface Feature #1790: implement menu support
The first question in previous note is solved: wrong implementation of @menu.width()@ Vadim Gindin
09:06 AM User Interface Feature #1790: implement menu support
Separate question about events. Lets look at @Button.processEvent()@ method:... Vadim Gindin
08:53 AM User Interface Feature #1790: implement menu support
The next buffled difficulty. Here is my menu:... Vadim Gindin
03:22 PM User Interface Feature #2480: add support for button and image attributes and options
This update for review(merged with recent code base) is release candidate for @DEFAULT@ button attribute feature. Add... Eugenie Lyzenko
08:21 PM User Interface Feature #2480: add support for button and image attributes and options
This updates is merged with the recent code base. Also it adds conversion/runtime support for standalone set/get @DEF... Eugenie Lyzenko
12:34 PM User Interface Feature #2508: test LIST-ITEMS support in P2J and confirm that it is complete, add or fix feature...
Has passed regression testing and is committed as bzr rev. 10811 Igor Skornyakov
12:33 PM User Interface Bug #2520: Dynamic list controls are not visible
Final version has passed regression testing and is committed as bzr rev. 10811 Igor Skornyakov
12:32 PM User Interface Feature #2522: add attribute/method support for radio-set
Has passed regression testing and is committed as bzr rev. 10811 Igor Skornyakov
12:16 PM User Interface Feature #2522: add attribute/method support for radio-set
Yes, you can check it in. Greg Shah
11:29 AM User Interface Feature #2522: add attribute/method support for radio-set
Please find the final update attached. Igor Skornyakov
11:01 AM User Interface Feature #2522: add attribute/method support for radio-set
Please post the final update here. Greg Shah
04:18 AM User Interface Feature #2522: add attribute/method support for radio-set
All regression tests (except tc_job_002) passed. Can I commit mt changes? Igor Skornyakov
12:22 PM User Interface Feature #2422: add features to BROWSE
Yes, but we have single cell's widget (there are multiple columns) and only for editable browse.
Also we can track ...
Stanislav Lomany
12:06 PM User Interface Feature #2422: add features to BROWSE
Treating the browse column as a real widget is going to add quite a bit of complication. It will have to be ignored ... Greg Shah
11:39 AM User Interface Feature #2422: add features to BROWSE
Guys, about the @SCREEN-VALUE@ of a browse column. This attribute matches the content of the cell of the specific col... Stanislav Lomany

03/12/2015

04:01 PM User Interface Feature #2522: add attribute/method support for radio-set
Constantin Asofiei wrote:
> Igor, I think the update is safe now. But please rename @pairsSetWhenRealized@ to @item...
Igor Skornyakov
03:39 PM User Interface Feature #2522: add attribute/method support for radio-set
Igor, I think the update is safe now. But please rename @pairsSetWhenRealized@ to @itemsSetWhenRealized@ - this flag... Constantin Asofiei
01:16 PM User Interface Feature #2522: add attribute/method support for radio-set
Constantin Asofiei wrote:
> Igor, I'm still trying to understand why the @pairsSetWhenRealized@ flag is needed - ple...
Igor Skornyakov
12:51 PM User Interface Feature #2522: add attribute/method support for radio-set
Igor, I'm still trying to understand why the @pairsSetWhenRealized@ flag is needed - please post a simple, minimum te... Constantin Asofiei
12:37 PM User Interface Feature #2522: add attribute/method support for radio-set
Code Review ias_upd20150312b.zip
I'm OK with the changes.
Greg Shah
12:18 PM User Interface Feature #2522: add attribute/method support for radio-set
Actually Constantin is right: I've not tested all cases. Please find the update attached.
Thank you Constantin.
Igor Skornyakov
11:26 AM User Interface Feature #2522: add attribute/method support for radio-set
Constantin Asofiei wrote:
> About ias_upd20150312a.zip:
> * @ControlSetEntity.pairs@ - the javadoc for @return@ is ...
Igor Skornyakov
10:56 AM User Interface Feature #2522: add attribute/method support for radio-set
About ias_upd20150312a.zip:
* @ControlSetEntity.pairs@ - the javadoc for @return@ is not formatted properly.
* @Con...
Constantin Asofiei
09:07 AM User Interface Feature #2522: add attribute/method support for radio-set
Code Review ias_upd20150312a.zip
I'm fine with the changes.
Constantin: please review.
Greg Shah
07:44 AM User Interface Feature #2522: add attribute/method support for radio-set
Done. The regression test started. The attached archive also contains minor fixed for #2508 and #2520 Igor Skornyakov
09:22 AM User Interface Feature #2480: add support for button and image attributes and options
Code Review evl_upd20150311a.zip
I'm good with the changes.
Greg Shah

03/11/2015

06:47 PM User Interface Feature #2480: add support for button and image attributes and options
The update for review presents suggested approach in base for implementation of the default button feature. The frame... Eugenie Lyzenko
08:37 AM User Interface Feature #2480: add support for button and image attributes and options
My only concern here is that the logic for this should not be in the window widget, but in the frame widget. Doesn't... Greg Shah
09:23 PM User Interface Feature #2480: add support for button and image attributes and options
Looking for best implementation for button @DEFAULT@ attribute handling I see the following opportunity:
1. Pressing...
Eugenie Lyzenko
02:56 PM Database Bug #2535: enhancing datatype conversion functions
With the attached update, queries like:... Ovidiu Maxiniuc
10:45 AM Database Bug #2535: enhancing datatype conversion functions
Yes, this is correct, the new constructors are not really needed.
I checked all datatypes classes and they all have...
Ovidiu Maxiniuc
12:35 PM User Interface Bug #2520: Dynamic list controls are not visible
This looks reasonable. Please add this to the changes for #2522 to reduce testing effort. Also, it will give Consta... Greg Shah
11:55 AM User Interface Bug #2520: Dynamic list controls are not visible
With the following update in Frame... Igor Skornyakov
10:06 AM User Interface Feature #2422: add features to BROWSE
Update merged with the latest sources. Committed to bzr rev 10804. Stanislav Lomany
08:16 AM User Interface Feature #1790: implement menu support
Vadim, some notes about the vig_20150309a.zip update you've sent me:
* Rectangle: the instances of this class need t...
Constantin Asofiei

03/10/2015

07:17 PM User Interface Feature #1790: implement menu support
> I don't fully understand this statement. MENUBAR draws it's items (menu-items and sub-menus) within its area. Dropp... Greg Shah
06:58 PM User Interface Feature #1790: implement menu support
Greg Shah wrote:
> Some thoughts:
>
> 1. Menus are different from our normal containers like Frames. With menus,...
Vadim Gindin
06:38 PM User Interface Feature #1790: implement menu support
Some thoughts:
1. Menus are different from our normal containers like Frames. With menus, the contained widgets c...
Greg Shah
06:21 PM User Interface Feature #1790: implement menu support
1. I have simple MENUBAR with sub-menu and menu. It is drawn succesfully. At starting moment sub-menu has focus. When... Vadim Gindin
06:06 PM User Interface Feature #1790: implement menu support
If you are stuck, please post specific questions or post more details of the problem so that we can help. Greg Shah
05:45 PM User Interface Feature #2508: test LIST-ITEMS support in P2J and confirm that it is complete, add or fix feature...
Greg Shah wrote:
> Perhaps it makes sense to integrate these changes with your work in #2522 to reduce the amount ...
Igor Skornyakov
05:43 PM User Interface Feature #2508: test LIST-ITEMS support in P2J and confirm that it is complete, add or fix feature...
> And because there will be a separate commit by Constantin I understand a new history entry for the ControlSetEntity... Greg Shah
05:31 PM User Interface Feature #2508: test LIST-ITEMS support in P2J and confirm that it is complete, add or fix feature...
Greg Shah wrote:
> Code Review ias_upd20150310a.zip
>
> I'm fine with the proposed changes, except that the @Cont...
Igor Skornyakov
05:29 PM User Interface Feature #2508: test LIST-ITEMS support in P2J and confirm that it is complete, add or fix feature...
Code Review ias_upd20150310a.zip
I'm fine with the proposed changes, except that the @ControlSetConfig.java@ needs...
Greg Shah
05:24 PM User Interface Feature #2508: test LIST-ITEMS support in P2J and confirm that it is complete, add or fix feature...
Igor Skornyakov wrote:
> Shall I start the regression testing right now?
No, don't put this into testing since Co...
Greg Shah
05:21 PM User Interface Feature #2508: test LIST-ITEMS support in P2J and confirm that it is complete, add or fix feature...
Shall I start the regression testing right now? Igor Skornyakov
05:05 PM User Interface Feature #2508: test LIST-ITEMS support in P2J and confirm that it is complete, add or fix feature...
Please take a look at the augmented version of the Constantin's changes (I've the support for the above mentioned sub... Igor Skornyakov
03:27 PM User Interface Feature #2508: test LIST-ITEMS support in P2J and confirm that it is complete, add or fix feature...
Well, it seems that the situation is the following:
If LIST-ITEMS is assigned before the frame containing the widget...
Igor Skornyakov
03:12 PM User Interface Feature #2508: test LIST-ITEMS support in P2J and confirm that it is complete, add or fix feature...
The situation looks strange. If one puts assignments in Constantin's sample not within "on" block (just two consecuti... Igor Skornyakov
02:51 PM User Interface Feature #2508: test LIST-ITEMS support in P2J and confirm that it is complete, add or fix feature...
Constantin,
I agree with your changes. However I've noticed another non-compatibility with 4GL:
With your sample ru...
Igor Skornyakov
02:11 PM User Interface Feature #2508: test LIST-ITEMS support in P2J and confirm that it is complete, add or fix feature...
Igor, please review and test this update with your standalone tests. It fixes problems in this code:... Constantin Asofiei
11:03 AM User Interface Feature #2508: test LIST-ITEMS support in P2J and confirm that it is complete, add or fix feature...
The updated zip is attached. Greg Shah
10:51 AM User Interface Feature #2508: test LIST-ITEMS support in P2J and confirm that it is complete, add or fix feature...
setListItems() method fixed.
Has passed conversion regression testing and is committed as bzr rev. 10802.
Igor Skornyakov
10:36 AM User Interface Feature #2508: test LIST-ITEMS support in P2J and confirm that it is complete, add or fix feature...
Greg Shah wrote:
> What is the status of this? Also, please post the updated zip.
Regression test just passed (i...
Igor Skornyakov
10:05 AM User Interface Feature #2508: test LIST-ITEMS support in P2J and confirm that it is complete, add or fix feature...
What is the status of this? Also, please post the updated zip. Greg Shah
05:10 PM User Interface Bug #2520: Dynamic list controls are not visible
The updating of @savedWidgets@ doesn't work. Continue the investigation. Igor Skornyakov
06:26 AM User Interface Bug #2520: Dynamic list controls are not visible
I see. Thank you Constantin. Igor Skornyakov
06:21 AM User Interface Bug #2520: Dynamic list controls are not visible
Igor, see this test:... Constantin Asofiei
05:56 AM User Interface Bug #2520: Dynamic list controls are not visible
Constantin,
If I correctly understand your scenario my change retains the 4GL compatibility: after the frame is enab...
Igor Skornyakov
05:05 PM User Interface Feature #2480: add support for button and image attributes and options
The investigation shows the default button feature works only in GUI mode. The character mode just ignores this attri... Eugenie Lyzenko
03:28 PM Database Bug #2535: enhancing datatype conversion functions
Ovidiu Maxiniuc wrote:
> I guess we should keep the new constructor and replace the return line like:
> ...
Eric Faulhaber
02:53 PM Database Bug #2535: enhancing datatype conversion functions
The generated @evaluate@ method for @WhereExpression@ @whereExpr0@ looks currently like this:... Ovidiu Maxiniuc
02:39 PM Database Bug #2535: enhancing datatype conversion functions
You are right: we don't want to mask this error always, because there may be times when it represents a legitimate er... Eric Faulhaber
01:35 PM Database Bug #2535: enhancing datatype conversion functions
Eric,
I have an issue regarding the implementation of these methods. At first I implemented as simple as this:
<...
Ovidiu Maxiniuc
11:57 AM Database Bug #2535: enhancing datatype conversion functions
I just discovered that we don't cover all the conversions between datatypes. We need to implement additional construc... Ovidiu Maxiniuc
11:34 AM Database Bug #2535 (Closed): enhancing datatype conversion functions
Ovidiu Maxiniuc
11:20 AM User Interface Feature #2422: add features to BROWSE
The @QueryWrapper@ changes in 0310a look OK to me. Eric Faulhaber
09:59 AM User Interface Feature #2422: add features to BROWSE
Code Review svl_upd20150310a.zip
I'm fine with the changes.
Eric: please review the @QueryWrapper@ change.
B...
Greg Shah
05:53 AM User Interface Feature #2422: add features to BROWSE
Update for review. Relatively to the previous version of the update (0305a) only @QueryWrapper@ has changed. Stanislav Lomany
05:50 AM User Interface Feature #2422: add features to BROWSE
About the recet bug. Here is the code that repositions browse to an invalid id:... Stanislav Lomany

03/09/2015

07:24 PM User Interface Feature #2480: add support for button and image attributes and options
@evl_upd20150307a.zip@ has been committed as 10800. Eugenie Lyzenko
06:28 PM User Interface Feature #2480: add support for button and image attributes and options
OK, please commit it. Greg Shah
06:13 PM User Interface Feature #2480: add support for button and image attributes and options
Conversion testing completed. No difference in generated code. So the update @evl_upd20150307a.zip@ is ready to be co... Eugenie Lyzenko
03:23 PM User Interface Feature #2480: add support for button and image attributes and options
The new runtime test results: @10797_5c96d7c_20150309_evl.zip@ for update based on 10797. No regression found.
Sta...
Eugenie Lyzenko
06:53 PM User Interface Feature #2487: implement READ-FILE() and SAVE-FILE() methods in the editor widget
Please note this text from the Progress docs for @EDITOR Phrase@:... Greg Shah
04:43 PM User Interface Bug #2520: Dynamic list controls are not visible
Constantin Asofiei wrote:
> Igor, I think this is part of a larger defect in P2J, related to dynamic widgets/frames....
Igor Skornyakov
03:33 PM User Interface Bug #2520: Dynamic list controls are not visible
Constantin Asofiei wrote:
> The urgency now is to determine if your Frame.addWidget change does not break any existin...
Igor Skornyakov
03:30 PM User Interface Bug #2520: Dynamic list controls are not visible
Igor Skornyakov wrote:
> Constantin Asofiei wrote:
>
> > Yes, don't complicate things for now. Just confirm the ...
Constantin Asofiei
03:21 PM User Interface Bug #2520: Dynamic list controls are not visible
Constantin Asofiei wrote:
> Yes, don't complicate things for now. Just confirm the implications of your change an...
Igor Skornyakov
03:14 PM User Interface Bug #2520: Dynamic list controls are not visible
Igor Skornyakov wrote:
> Constantin Asofiei wrote:
>
> > In any case, this is not currently supported by P2J. Th...
Constantin Asofiei
03:10 PM User Interface Bug #2520: Dynamic list controls are not visible
Constantin Asofiei wrote:
> In any case, this is not currently supported by P2J. The APIs are a no-op now.
I s...
Igor Skornyakov
02:57 PM User Interface Bug #2520: Dynamic list controls are not visible
Igor Skornyakov wrote:
> Constantin Asofiei wrote:
>
> > As I see from dvref.pdf, AUTO-RESIZE is for GUI only.
>...
Constantin Asofiei
02:48 PM User Interface Bug #2520: Dynamic list controls are not visible
Constantin Asofiei wrote:
> As I see from dvref.pdf, AUTO-RESIZE is for GUI only.
I know. However in my tests t...
Igor Skornyakov
02:44 PM User Interface Bug #2520: Dynamic list controls are not visible
Igor Skornyakov wrote:
> Constantin Asofiei wrote:
> > Igor, I think this is part of a larger defect in P2J, relate...
Constantin Asofiei
02:39 PM User Interface Bug #2520: Dynamic list controls are not visible
Constantin Asofiei wrote:
> Igor, I think this is part of a larger defect in P2J, related to dynamic widgets/frames....
Igor Skornyakov
02:25 PM User Interface Bug #2520: Dynamic list controls are not visible
Igor, I think this is part of a larger defect in P2J, related to dynamic widgets/frames.
Something I think shoul...
Constantin Asofiei
10:52 AM User Interface Bug #2520: Dynamic list controls are not visible
Constantin Asofiei wrote:
> So, Igor: what issue were you trying to solve? Do you have a standalone test for it?
...
Igor Skornyakov
10:35 AM User Interface Bug #2520: Dynamic list controls are not visible
Greg Shah wrote:
> Constantin: please review the change (bzr rev 10797). Is the modification safe?
It's hard to tel...
Constantin Asofiei
09:12 AM User Interface Bug #2520: Dynamic list controls are not visible
Greg Shah wrote:
> Igor: in the future, please do not check anything in before you are explicitly approved to do so....
Igor Skornyakov
09:08 AM User Interface Bug #2520: Dynamic list controls are not visible
Igor: in the future, please do not check anything in before you are explicitly approved to do so. Such approval is a... Greg Shah
09:47 AM User Interface Feature #2508: test LIST-ITEMS support in P2J and confirm that it is complete, add or fix feature...
Code Review ias_upd20150309a.zip
The change is good. Please add a history entry, post the updated 0309b zip and g...
Greg Shah
09:19 AM User Interface Feature #2508: test LIST-ITEMS support in P2J and confirm that it is complete, add or fix feature...
Fixed regression reported by Constantin (see #2408 notes 135/136/137). Restarted regression test Igor Skornyakov
08:54 AM User Interface Feature #2508 (Closed): test LIST-ITEMS support in P2J and confirm that it is complete, add or fi...
Greg Shah
09:11 AM User Interface Feature #2498: implement ADD-LAST() method
Constantin Asofiei wrote:
> Have you tested both attributes with both widgets?
The method is the same for both widg...
Igor Skornyakov
08:22 AM User Interface Feature #2498: implement ADD-LAST() method
Igor Skornyakov wrote:
> Constantin Asofiei wrote:
> > Igor, you have a regression when LIST-ITEMS and LIST-ITEM-PA...
Constantin Asofiei
07:55 AM User Interface Feature #2498: implement ADD-LAST() method
Constantin Asofiei wrote:
> Igor, you have a regression when LIST-ITEMS and LIST-ITEM-PAIRS are set via the widget a...
Igor Skornyakov
07:25 AM User Interface Feature #2498: implement ADD-LAST() method
Igor, you have a regression when LIST-ITEMS and LIST-ITEM-PAIRS are set via the widget attribute, as in:... Constantin Asofiei

03/08/2015

05:20 AM User Interface Bug #2520: Dynamic list controls are not visible
Has passed conversion regression testing and is committed as bzr rev. 10797. Igor Skornyakov
07:40 PM User Interface Feature #2480: add support for button and image attributes and options
The testing completed, no regressions. The results: @10794_5c96d7c_20150307_evl.zip@.
Looks like I have to re-run ...
Eugenie Lyzenko

03/07/2015

01:51 PM User Interface Feature #2480: add support for button and image attributes and options
Merge with 10796 code base. Eugenie Lyzenko
08:46 PM User Interface Feature #2480: add support for button and image attributes and options
The conversion testing completed. Generated codes are identical, no regressions. Starting runtime testing. Eugenie Lyzenko
03:51 AM User Interface Bug #2520: Dynamic list controls are not visible
Fixed. Started regression test Igor Skornyakov
03:36 AM User Interface Feature #2508: test LIST-ITEMS support in P2J and confirm that it is complete, add or fix feature...
Passed runtime regression testing and checked in as bzr rev 10796. Igor Skornyakov

03/06/2015

05:56 PM Database Bug #2488: replace runtime compilation of dynamic queries
Code review 0306a:
Looks good. I'll run tests with this update and let you know how it goes.
Eric Faulhaber
01:31 PM Database Bug #2488: replace runtime compilation of dynamic queries
The strings literals are generated as ready-to-be-written to java source files and the escape sequences must be proce... Ovidiu Maxiniuc
04:46 PM User Interface Feature #2480: add support for button and image attributes and options
Eugenie Lyzenko wrote:
> >Eugenie, do you know where the clipping rectangle is first calculated wrong? If not, can y...
Hynek Cihlar
04:15 PM User Interface Feature #2480: add support for button and image attributes and options
>I have no objections to this.
OK. I'm starting the conversion and runtime testing.
Eugenie Lyzenko
04:01 PM User Interface Feature #2480: add support for button and image attributes and options
>Eugenie, do you know where the clipping rectangle is first calculated wrong? If not, can you trace it down? Also, is... Eugenie Lyzenko
02:20 PM User Interface Feature #2480: add support for button and image attributes and options
Greg Shah wrote:
> > One consideration about clipping. The feature is now working for button widget but I think it h...
Hynek Cihlar
02:05 PM User Interface Feature #2480: add support for button and image attributes and options
> One consideration about clipping. The feature is now working for button widget but I think it has an issue. If usin... Greg Shah
02:04 PM User Interface Feature #2480: add support for button and image attributes and options
Code Review evl_upd20150306a.zip
I'm fine with the changes.
Greg Shah
12:32 PM User Interface Feature #2480: add support for button and image attributes and options
This update for review has completed runtime implementation for @NO-FOCUS@/@FLAT-BUTTON@ client part plus fixed newly... Eugenie Lyzenko
03:45 PM User Interface Feature #2508: test LIST-ITEMS support in P2J and confirm that it is complete, add or fix feature...
After 3 runs the following 2 tests failed (in addition to the expected tc_job_002):... Igor Skornyakov
03:00 PM User Interface Feature #1790: implement menu support
Code Review vig_upd20150306a.zip
This is a good step forward.
1. It seems like there should be shared code betw...
Greg Shah
02:16 PM User Interface Feature #1790: implement menu support
> When I'm pressing '->' in MENUBAR first menu-item (sub-menu) is blended when loosing focus. There are several calls... Greg Shah
10:37 AM User Interface Feature #1790: implement menu support
It seems I will not make working version until today. Here is my current update with the following changes.
* Focus ...
Vadim Gindin
02:40 PM User Interface Feature #2422: add features to BROWSE
One regression test has failed: @P2JQuery.bufferHandle()@ returns invalid handle. Looking into it. Stanislav Lomany
08:46 AM User Interface Feature #2422: add features to BROWSE
Code Review svl_upd20150305a.zip
I'm OK with the changes.
Eric: please review the @QueryWrapper@ change.
Greg Shah
02:59 AM User Interface Feature #2422: add features to BROWSE
Update for review. Fixes minor issues. Stanislav Lomany
11:44 AM User Interface Feature #2534: methods/attrs support for EDITOR, SELECTION-LIST and COMBO-BOX
NUM-ITEMS runtime support (combo-box and selection-list)
INNER-LINES runtime setter attribute unknown behavior shoul...
Greg Shah
09:38 AM User Interface Feature #2534 (Closed): methods/attrs support for EDITOR, SELECTION-LIST and COMBO-BOX
Greg Shah
08:46 AM User Interface Bug #2520: Dynamic list controls are not visible
It seems that I've resolved the issue (with one-line fix). Will start regression testing after the one for #2508 will... Igor Skornyakov

03/05/2015

03:50 PM User Interface Feature #2508: test LIST-ITEMS support in P2J and confirm that it is complete, add or fix feature...
Code Review ias_upd20150305a.zip
I'm good with the changes.
Greg Shah
03:30 PM User Interface Feature #2508: test LIST-ITEMS support in P2J and confirm that it is complete, add or fix feature...
NPE was a result of my bug. To fix the cursor issue it was sufficient to hide cursor on SELECTION-LIST focus gain (an... Igor Skornyakov
09:02 AM User Interface Feature #2508: test LIST-ITEMS support in P2J and confirm that it is complete, add or fix feature...
OK, please do fix these as part of this task. Greg Shah
09:00 AM User Interface Feature #2508: test LIST-ITEMS support in P2J and confirm that it is complete, add or fix feature...
Greg Shah wrote:
> Are these bugs specific to your change or are they already existing in P2J?
These bugs are not...
Igor Skornyakov
08:57 AM User Interface Feature #2508: test LIST-ITEMS support in P2J and confirm that it is complete, add or fix feature...
Are these bugs specific to your change or are they already existing in P2J? Greg Shah
08:56 AM User Interface Feature #2508: test LIST-ITEMS support in P2J and confirm that it is complete, add or fix feature...
Found at least two bugs.
1. When there are several widgets in a frame the cursor position doesn't change when switch...
Igor Skornyakov

03/04/2015

04:26 PM User Interface Feature #2487: implement READ-FILE() and SAVE-FILE() methods in the editor widget
> 1) take big file (much more then limit). And time to load this file will be much bigger then time to read file just... Greg Shah
04:24 PM User Interface Feature #2487: implement READ-FILE() and SAVE-FILE() methods in the editor widget
Code Review evk_upd20150301a.zip
1. Please name @LogicalTerminal.writeFile()@ as @LogicalTerminal.saveFile()@ to b...
Greg Shah
03:58 PM User Interface Feature #2487: implement READ-FILE() and SAVE-FILE() methods in the editor widget
> Also I need to set modified flag. But didn't find good place to do it. Where I can have access to this flag ?
Se...
Greg Shah
08:08 PM User Interface Feature #2487: implement READ-FILE() and SAVE-FILE() methods in the editor widget
1) Added error handling and file stream close
4GL writes data into file till IO error. But for read operation 4gl re...
Evgeny Kiselev
03:17 PM User Interface Feature #2508: test LIST-ITEMS support in P2J and confirm that it is complete, add or fix feature...
Merged with revno 10793. Tested. The selection list behavior on gain/loose focus has to be fixed. Igor Skornyakov
11:43 AM User Interface Feature #2498 (Closed): implement ADD-LAST() method
Greg Shah
11:08 AM User Interface Feature #2498: implement ADD-LAST() method
Passed runtime regression testing and checked in as bzr rev 10793. Igor Skornyakov
11:08 AM User Interface Feature #2498: implement ADD-LAST() method
After updating majic_baseline @tc_po_item_003@ and @tc_po_item_004@. Majic rebuild passed OK.
Igor Skornyakov
08:49 AM User Interface Feature #2498: implement ADD-LAST() method
Greg Shah wrote:
> Don't worry about the ctrl-c tests. The 3-way tests are known to fail and so long as all of them...
Igor Skornyakov
08:41 AM User Interface Feature #2498: implement ADD-LAST() method
Don't worry about the ctrl-c tests. The 3-way tests are known to fail and so long as all of them don't fail on all r... Greg Shah
05:24 AM User Interface Feature #2498: implement ADD-LAST() method
Constantin Asofiei wrote:
> With rev 10787 there were baseline changes. You should reconfigure your setup or do a ...
Igor Skornyakov
05:17 AM User Interface Feature #2498: implement ADD-LAST() method
Igor Skornyakov wrote:
> @gso_ctrlc_3way_tests@ restarted. I've extracted @tc_po_item_003@ and @tc_po_item_004@ and ...
Constantin Asofiei
05:08 AM User Interface Feature #2498: implement ADD-LAST() method
In a third run all @gso_ctrlc_3way_tests@ failed. @tc_job_clock_004@ passed. I've also noticed that the following te... Igor Skornyakov
06:45 AM Database Bug #2524: missing support for NEXT-SIBLING attribute for buffer handles
Ovidiu Maxiniuc wrote:
> Constantin Asofiei wrote:
> > Ovidiu, about HandleChain changes: the buffer-specific code ...
Constantin Asofiei
05:43 AM Database Bug #2524: missing support for NEXT-SIBLING attribute for buffer handles
Constantin Asofiei wrote:
> Ovidiu, about HandleChain changes: the buffer-specific code I think should reside in ove...
Ovidiu Maxiniuc
05:37 AM Database Bug #2524: missing support for NEXT-SIBLING attribute for buffer handles
Ovidiu, about HandleChain changes: the buffer-specific code I think should reside in overridden methods in the Buffer... Constantin Asofiei
05:17 AM Database Bug #2524: missing support for NEXT-SIBLING attribute for buffer handles
Eric Faulhaber wrote:
> Ovidiu: please rename the testcase to something more descriptive (you can keep "2524" as par...
Ovidiu Maxiniuc
09:33 PM User Interface Feature #2480: add support for button and image attributes and options
This update for review contains runtime implementation for @NO-FOCUS@/@FLAT-BUTTON@. It is required to do more testin... Eugenie Lyzenko

03/03/2015

05:28 PM Database Bug #2524: missing support for NEXT-SIBLING attribute for buffer handles
One more thing: please make the @BufferImpl@ constructor you added @protected@ instead of @public@. As an implementat... Eric Faulhaber
03:29 PM Database Bug #2524: missing support for NEXT-SIBLING attribute for buffer handles
Code review 0303c:
You added a call to @release()@ at @TemporaryBuffer@:3411, which seems unrelated to the remaini...
Eric Faulhaber
12:26 PM Database Bug #2524: missing support for NEXT-SIBLING attribute for buffer handles
The attached update fixes the management of dynamic buffer handles. Ovidiu Maxiniuc
05:18 PM User Interface Feature #2498: implement ADD-LAST() method
The following tests failed identically in two runs:... Igor Skornyakov
09:43 AM User Interface Feature #2498: implement ADD-LAST() method
Greg Shah wrote:
> I think @unwrapControlEntity()@ should be removed from @handle@. Rebuilding MAJIC will confirm...
Igor Skornyakov
09:37 AM User Interface Feature #2498: implement ADD-LAST() method
Code Review ias_upd20150303c.zip
I think @unwrapControlEntity()@ should be removed from @handle@. Rebuilding MAJI...
Greg Shah
07:30 AM User Interface Feature #2498: implement ADD-LAST() method
The problem was with @ScrollConatiner.ensureVisibility()@: I've reimplemented it to achieve the correct behavior for ... Igor Skornyakov
02:50 PM User Interface Feature #2422: add features to BROWSE
svl_upd20150302a was committed to bzr revision 10792. Stanislav Lomany
02:25 PM User Interface Feature #2422: add features to BROWSE
OK, my bad. I misread your post. The last record to the top of the browse IS CORRECT. In this case my update works pr... Stanislav Lomany
01:28 PM User Interface Feature #2422: add features to BROWSE
Correct or not, it's what Progress 9.1 did with the AOG Browse screen. It was quite strange. I've never recreated it ... Eric Faulhaber
01:23 PM User Interface Feature #2422: add features to BROWSE
Eric, are you sure that scrolling to the last row (and making it the scroll row) is not a correct behavior? Stanislav Lomany
09:50 AM User Interface Feature #2422: add features to BROWSE
AOG Browse fails. Reverting the code in AdaptiveQuery doesn't fix the issue. I'll look into that. Stanislav Lomany
09:31 AM User Interface Feature #2422: add features to BROWSE
@Cursor@:297 looks a little different to me. Have you confirmed the described behavior with the AOG Browse has not be... Eric Faulhaber
03:47 AM User Interface Feature #2422: add features to BROWSE
Eric, in this file there is also updated history entry which says ... Stanislav Lomany
09:29 PM User Interface Feature #2422: add features to BROWSE
Greg Shah wrote:
> Code Review svl_upd20150302a.zip
>
> I'm OK with the changes.
>
> Eric: please review the @...
Eric Faulhaber
11:55 AM Database Bug #2533 (WIP): management of dynamic buffer handles
Ovidiu Maxiniuc

03/02/2015

05:11 PM User Interface Feature #2422: add features to BROWSE
Code Review svl_upd20150302a.zip
I'm OK with the changes.
Eric: please review the @AdaptiveQuery@ change.
Greg Shah
05:05 PM User Interface Feature #2422: add features to BROWSE
Passed regression testing, so I can check it in just after review. Stanislav Lomany
03:47 PM User Interface Feature #2422: add features to BROWSE
Update for review with fixes for RFQ tests. Regression testing is running. Stanislav Lomany
04:44 PM User Interface Feature #2498: implement ADD-LAST() method
Just a note. If the screen size is ~ 132x43 then the failing screen with and without my changes look identical (manua... Igor Skornyakov
11:21 AM User Interface Feature #2498: implement ADD-LAST() method
Greg Shah wrote:
> Remove your update and manually test again to determine if it is your update that causes this. I...
Igor Skornyakov
11:15 AM User Interface Feature #2498: implement ADD-LAST() method
Remove your update and manually test again to determine if it is your update that causes this. I guess it is so, unl... Greg Shah
11:14 AM User Interface Feature #2498: implement ADD-LAST() method
Constantin Asofiei wrote:
> Igor, is your terminal set to 80 cols by 24 rows? MAJIC layout will not be correct if th...
Igor Skornyakov
11:08 AM User Interface Feature #2498: implement ADD-LAST() method
Igor, is your terminal set to 80 cols by 24 rows? MAJIC layout will not be correct if the terminal is not set like this. Constantin Asofiei
11:07 AM User Interface Feature #2498: implement ADD-LAST() method
Hynek Cihlar wrote:
> Igor if haven't already, try to run the test manually. That is, start the client, login, and f...
Igor Skornyakov
08:34 AM User Interface Feature #2498: implement ADD-LAST() method
Igor Skornyakov wrote:
> I've added a delay to gso_301 (more presisely, to @common/find_rfq_so_8000356.xml@), but th...
Hynek Cihlar
08:27 AM User Interface Feature #2498: implement ADD-LAST() method
Constantin Asofiei wrote:
> Greg Shah wrote:
> > Has anyone seen this kind of failure in the RFQ tests? Could we h...
Igor Skornyakov
02:24 PM User Interface Feature #2334 (Closed): implement the GUI message line widget
Greg Shah
11:30 AM User Interface Bug #2528: finish investigation and implemention of INPUT, SCREEN-VALUE and INPUT-VALUE
This task is meant to complete the implementation of the INPUT, SCREEN-VALUE and INPUT-VALUE.
Some initial finding...
Constantin Asofiei
11:22 AM User Interface Bug #2528 (New): finish investigation and implemention of INPUT, SCREEN-VALUE and INPUT-VALUE
Constantin Asofiei
11:15 AM Base Language Bug #2527: after ENDKEY (F4) the server-side screen buffer needs to be updated
An ENDKEY condition generated by i.e. F4 while an update is in progress will update the SCREEN-VALUE and INPUT functi... Constantin Asofiei
11:09 AM Base Language Bug #2527 (New): after ENDKEY (F4) the server-side screen buffer needs to be updated
Constantin Asofiei
09:20 PM User Interface Feature #2487: implement READ-FILE() and SAVE-FILE() methods in the editor widget
Interesting. These sound pretty easy for us to implement.
> maximum stored data depends on @INNER-CHARS@ paramete...
Greg Shah
07:11 PM User Interface Feature #2487: implement READ-FILE() and SAVE-FILE() methods in the editor widget
1) If some IO error while reading/writing, then it read or wrote data can be used, but status of operation will be "n... Evgeny Kiselev

02/27/2015

05:18 PM User Interface Feature #2480: add support for button and image attributes and options
The update contains conversion support and runtime start point starting point for @NO-FOCUS@ and @FLAT-BUTTON@ option... Eugenie Lyzenko
02:21 PM User Interface Feature #2498: implement ADD-LAST() method
No, let it finish. Greg Shah
01:41 PM User Interface Feature #2498: implement ADD-LAST() method
Greg Shah wrote:
> Igor: do you have any other problems to solve or is this it?
No. I do not have any new problem...
Igor Skornyakov
01:37 PM User Interface Feature #2498: implement ADD-LAST() method
Igor: do you have any other problems to solve or is this it?
Constantin, Hynek, Eric and I all have quite a few ch...
Greg Shah
01:11 PM User Interface Feature #2498: implement ADD-LAST() method
Greg Shah wrote:
> Has anyone seen this kind of failure in the RFQ tests? Could we have checked something in that ha...
Constantin Asofiei
01:03 PM User Interface Feature #2498: implement ADD-LAST() method
Has anyone seen this kind of failure in the RFQ tests? Could we have checked something in that has broken RFQ? Greg Shah
11:45 AM User Interface Feature #2498: implement ADD-LAST() method
Both gsO301 and gso_308 failed at the same screen. As far as I understand the code for this screen is in @syman/so/se... Igor Skornyakov
07:18 AM User Interface Feature #2498: implement ADD-LAST() method
After fixes only the following tests failed in two runs:
3 gso_301: failure in step 6: 'timeout before the spec...
Igor Skornyakov
08:37 AM Database Bug #2526: Records preserving its position in the sorting index for adaptive query
As for initial analysis: currently we use record snapshots as the base to find the next record in the index using an ... Stanislav Lomany
08:28 AM Database Bug #2526: Records preserving its position in the sorting index for adaptive query
This problem was noticed in an editable browse which easily exposes this bug. Assigning a value to a field of the ind... Stanislav Lomany
08:17 AM Database Bug #2526 (New): Records preserving its position in the sorting index for adaptive query
Stanislav Lomany
07:41 AM User Interface Feature #2525: Runtime support of frame positional attributes
The objective of this issue is to implement support of missing parts of frame positional attributes. Also to make sur... Hynek Cihlar
07:23 AM User Interface Feature #2525 (New): Runtime support of frame positional attributes
Hynek Cihlar

02/26/2015

04:07 PM User Interface Feature #2479 (Closed): DEFINE IMAGE and image phrase support for both button and image
Greg Shah
03:38 PM User Interface Feature #2479: DEFINE IMAGE and image phrase support for both button and image
The update @evl_upd20150225b.zip@ has been committed in bzr as 10780. Eugenie Lyzenko
03:20 PM User Interface Feature #2479: DEFINE IMAGE and image phrase support for both button and image
Yes, please check it in. Greg Shah
03:14 PM User Interface Feature #2479: DEFINE IMAGE and image phrase support for both button and image
Testing completed. The results file: @10778_5c96d7c_20150226_evl.zip@.
So the update is ready to be committed and ...
Eugenie Lyzenko
11:47 AM User Interface Feature #2479: DEFINE IMAGE and image phrase support for both button and image
The main part of the regression tests completed OK. Waiting for CTRL-C to be done. Eugenie Lyzenko
07:48 PM User Interface Feature #2479: DEFINE IMAGE and image phrase support for both button and image
The conversion testing completed. Codes are identical, no regression. Continue with the runtime testing. Eugenie Lyzenko
12:49 PM Database Bug #2488: replace runtime compilation of dynamic queries
I found a test case that fails to convert properly.... Ovidiu Maxiniuc
12:25 PM User Interface Feature #2422: add features to BROWSE
Stanislav Lomany wrote:
> After my recent minor changes RFQ tests started to fail. I'm looking into that.
I would...
Eric Faulhaber
12:15 PM User Interface Feature #2422: add features to BROWSE
After my recent minor changes RFQ tests started to fail. I'm looking into that. Stanislav Lomany
12:09 PM User Interface Feature #2422: add features to BROWSE
The persistence changes in 0225a look OK to me. Eric Faulhaber
09:38 AM User Interface Feature #2422: add features to BROWSE
Code Review svl_upd20150225a.zip
I'm fine with the changes. I looked at everything pretty carefully, except @Brow...
Greg Shah
10:18 AM User Interface Feature #1790: implement menu support
Yes, that was the reason.. Sorry for trouble. Vadim Gindin
10:03 AM User Interface Feature #1790: implement menu support
Mistake..
Here is the place in @ThinClient.waitFor@ where immedExit is set to true:...
Vadim Gindin
09:59 AM User Interface Feature #1790: implement menu support
I found that during processing of @ThinClient.waitFor@ the flag @immedExit@ is set to true. It probably a reason.. It... Vadim Gindin
09:51 AM User Interface Feature #1790: implement menu support
I'm stuck... I can't make menu events working. I don't fully understand how events queue works. Here are the followin... Vadim Gindin
06:30 AM User Interface Feature #2508: test LIST-ITEMS support in P2J and confirm that it is complete, add or fix feature...
Code Review ias_upd20150225a.zip
1. The update should only contain files that were modified for this task. As far...
Greg Shah

02/25/2015

03:58 PM User Interface Feature #2479: DEFINE IMAGE and image phrase support for both button and image
> Eugenie: please confirm that it is not needed. If so, please remove it. Then go ahead with both conversion and ru... Eugenie Lyzenko
03:52 PM User Interface Feature #2479: DEFINE IMAGE and image phrase support for both button and image
> the call to @setPhysicalLocation@ in @IGI.initialize@ should be unnecessary as you already set the physical locatio... Greg Shah
03:36 PM User Interface Feature #2479: DEFINE IMAGE and image phrase support for both button and image
Greg Shah wrote:
> Code Review evl_upd20150225a.zip
>
> I'm fine with the changes.
>
> Hynek: please look at t...
Hynek Cihlar
02:11 PM User Interface Feature #2479: DEFINE IMAGE and image phrase support for both button and image
Code Review evl_upd20150225a.zip
I'm fine with the changes.
Hynek: please look at the @BaseConfig@ change and c...
Greg Shah
12:07 PM User Interface Feature #2479: DEFINE IMAGE and image phrase support for both button and image
This update for review has general fix for dynamic widget location issue. All processing shifted to @BaseConfig.apply... Eugenie Lyzenko
09:33 AM User Interface Feature #2479: DEFINE IMAGE and image phrase support for both button and image
Greg Shah wrote:
> Code Review evl_upd20150224a.zip
>
> Are @ButtonGuiImpl.draw()@ and @ImageGuiImpl.draw()@ real...
Hynek Cihlar
09:17 AM User Interface Feature #2479: DEFINE IMAGE and image phrase support for both button and image
Eugenie Lyzenko wrote:
> One more possible TODO change is rework for image phrase setters for button related images ...
Greg Shah
09:16 AM User Interface Feature #2479: DEFINE IMAGE and image phrase support for both button and image
One more possible TODO change is rework for image phrase setters for button related images we have discussed previous... Eugenie Lyzenko
09:05 AM User Interface Feature #2479: DEFINE IMAGE and image phrase support for both button and image
Code Review evl_upd20150224a.zip
Are @ButtonGuiImpl.draw()@ and @ImageGuiImpl.draw()@ really the proper locations ...
Greg Shah
08:53 AM User Interface Feature #2479: DEFINE IMAGE and image phrase support for both button and image
Eugenie Lyzenko wrote:
> The only thing I see is to unify @CONVERT-3D-COLORS@ attribute for button and image widgets...
Greg Shah
08:51 AM User Interface Feature #2479: DEFINE IMAGE and image phrase support for both button and image
The only thing I see is to unify @CONVERT-3D-COLORS@ attribute for button and image widgets. Currently button has @NO... Eugenie Lyzenko
03:53 PM Database Bug #2524: missing support for NEXT-SIBLING attribute for buffer handles
The reference pdf states that the SESSION:FIRST-BUFFER is the head of the linked list of _"handles for the dynamic bu... Ovidiu Maxiniuc
03:09 PM User Interface Feature #2508: test LIST-ITEMS support in P2J and confirm that it is complete, add or fix feature...
Finished adding DELIMITER attribute support and fixing LIST-ITEMS and LIST-ITEM-PAIRS logic. Testing is still require... Igor Skornyakov
02:32 PM User Interface Feature #2508: test LIST-ITEMS support in P2J and confirm that it is complete, add or fix feature...
Greg Shah wrote:
> Progress is full of these insane "quirks". In these cases we try to leave careful comments and j...
Igor Skornyakov
02:19 PM User Interface Feature #2508: test LIST-ITEMS support in P2J and confirm that it is complete, add or fix feature...
Progress is full of these insane "quirks". In these cases we try to leave careful comments and javadoc to explain th... Greg Shah
02:15 PM User Interface Feature #2508: test LIST-ITEMS support in P2J and confirm that it is complete, add or fix feature...
Greg Shah wrote:
> Are you saying that the 4GL reacts differently in these two situations?
Yes, it does.
> If so...
Igor Skornyakov
02:02 PM User Interface Feature #2508: test LIST-ITEMS support in P2J and confirm that it is complete, add or fix feature...
Are you saying that the 4GL reacts differently in these two situations?
Unknown literal:...
Greg Shah
12:47 PM User Interface Feature #2508: test LIST-ITEMS support in P2J and confirm that it is complete, add or fix feature...
I've discovered a strange behavior: for the COMBO-BOX and SELECTION-LIST @ADD-LAST(..., ?)@ is rejected but the code
...
Igor Skornyakov
01:58 PM User Interface Feature #2498: implement ADD-LAST() method
Greg Shah wrote:
> OK, this makes sense. As you move forward, do keep in mind that you will likely need to merge yo...
Igor Skornyakov
11:58 AM User Interface Feature #2498: implement ADD-LAST() method
OK, this makes sense. As you move forward, do keep in mind that you will likely need to merge your code up to newer ... Greg Shah
11:35 AM User Interface Feature #2498: implement ADD-LAST() method
Adding separate height/width fields doesn't help. Starting a detailed investigation. In particular I can see the the ... Igor Skornyakov
09:08 AM User Interface Feature #2478 (Closed): button image support
Greg Shah
09:07 AM User Interface Feature #2476 (Closed): window icon support improvements
Greg Shah
08:55 AM User Interface Feature #2480: add support for button and image attributes and options
The following attributes/options are left to b implemented/verified as working if already implemented:
+Button opt...
Eugenie Lyzenko
05:53 AM User Interface Feature #2422: add features to BROWSE
Major browse update for review. Changed caching logic in order to match 4GL. Stanislav Lomany

02/24/2015

06:33 PM User Interface Feature #2479: DEFINE IMAGE and image phrase support for both button and image
This update for review merges the recent code base plus fixes the location issue for dynamic images and buttons. The ... Eugenie Lyzenko
03:14 PM Database Bug #2524 (WIP): missing support for NEXT-SIBLING attribute for buffer handles
Ovidiu Maxiniuc
03:11 PM Database Bug #2524: missing support for NEXT-SIBLING attribute for buffer handles
I noticed the following message printed in the app-server 'client' log after each executed request.... Ovidiu Maxiniuc
02:44 PM Database Bug #2524 (Closed): missing support for NEXT-SIBLING attribute for buffer handles
Ovidiu Maxiniuc

02/23/2015

03:08 PM User Interface Feature #2508: test LIST-ITEMS support in P2J and confirm that it is complete, add or fix feature...
The version on lindev01 is 10.2B and has not been changed since it was installed in early 2013. I access it via /usr... Greg Shah
02:34 PM User Interface Feature #2508: test LIST-ITEMS support in P2J and confirm that it is complete, add or fix feature...
RADIO-BUTTONS attribute for the RADIO-SET widget behaves like LIST-ITEM-PAIRS for the SELECTION-LIST.
Conclusion:
...
Igor Skornyakov
10:38 AM User Interface Feature #2508: test LIST-ITEMS support in P2J and confirm that it is complete, add or fix feature...
1. We need to add support for the DELIMITER attribute
2. DELIMITER attribute value cannot be "unknown", but can be "...
Igor Skornyakov
02:41 PM User Interface Feature #2498: implement ADD-LAST() method
Greg Shah wrote:
> Although it seems unlikely that the problems are caused by your update, it is very unusual for th...
Igor Skornyakov
11:30 AM User Interface Feature #2498: implement ADD-LAST() method
Although it seems unlikely that the problems are caused by your update, it is very unusual for the same list of tests... Greg Shah
09:14 AM User Interface Feature #2498: implement ADD-LAST() method
Greg,
Can I commit my changes?
Thank you.
Igor Skornyakov
01:29 PM User Interface Feature #2479: DEFINE IMAGE and image phrase support for both button and image
> Normally, I would ask you to get these tested, but Igor has overlapping changes being tested in #2498 and he has ha... Eugenie Lyzenko
01:07 PM User Interface Feature #2479: DEFINE IMAGE and image phrase support for both button and image
Code Review evl_upd20150223a.zip
The changes look good.
Normally, I would ask you to get these tested, but Igor...
Greg Shah
12:39 PM User Interface Feature #2479: DEFINE IMAGE and image phrase support for both button and image
Sorry the update @0223a@ was inconsistent(has not final versions). Please see the new drop. Eugenie Lyzenko
12:20 PM User Interface Feature #2479: DEFINE IMAGE and image phrase support for both button and image
This update for review adds boolean attributes setters/getters outside of the @DEFINE IMAGE@ or @DEFINE BUTTON@ state... Eugenie Lyzenko
07:19 AM User Interface Feature #2479: DEFINE IMAGE and image phrase support for both button and image
The remaining thing I'm going to implement is having getters/setters for all attributes in image and button widgets t... Eugenie Lyzenko
12:02 PM User Interface Feature #1790: implement menu support
>> At this point are ChUI menus fully functional?
>
> No. I can't finish client-side part because of event-processi...
Greg Shah
10:07 AM User Interface Feature #1790: implement menu support
You're right. Some of noted points are not implemented. See below
Greg Shah wrote:
> At this point are ChUI menus...
Vadim Gindin

02/22/2015

04:05 PM User Interface Feature #2479: DEFINE IMAGE and image phrase support for both button and image
Greg Shah wrote:
> Yes, go ahead and commit it.
Committed in bzr as 10771.
Eugenie Lyzenko
03:32 PM User Interface Feature #2479: DEFINE IMAGE and image phrase support for both button and image
Yes, go ahead and commit it. Greg Shah
02:53 PM User Interface Feature #2479: DEFINE IMAGE and image phrase support for both button and image
The conversion testing completed, generated codes are identical. No regressions.
So the update is ready to be comm...
Eugenie Lyzenko
07:03 AM User Interface Feature #2479: DEFINE IMAGE and image phrase support for both button and image
The update fixes the size/location regression for images used to draw the buttons. Plus slightly enhanced icon detect... Eugenie Lyzenko
08:01 AM User Interface Feature #2498: implement ADD-LAST() method
Attached detailed reports for failed tests. Igor Skornyakov
05:44 AM User Interface Feature #2498: implement ADD-LAST() method
There where 3 runs of the regression test.
The following test failed in in all 3 runs:
gso_tests
57 gso_187: fa...
Igor Skornyakov

02/20/2015

06:10 PM User Interface Feature #2513: implement GUI runtime support for TOGGLE-BOX
Here is the next update containing basic GUI implementation.
Questions:
1. When I'm going between checkboxes I've...
Vadim Gindin
05:25 PM User Interface Feature #2523: implement GUI radio-set widget
The core widget implementation for GUI must be created including drawing and event processing. Mouse support must wo... Greg Shah
05:23 PM User Interface Feature #2523 (Closed): implement GUI radio-set widget
Greg Shah
05:22 PM User Interface Feature #2522: add attribute/method support for radio-set
Implement conversion and runtime support for the following:
+Attributes+
NUM-BUTTONS
+Methods+
DISABLE()
...
Greg Shah
05:21 PM User Interface Feature #2522 (Closed): add attribute/method support for radio-set
Greg Shah
05:21 PM User Interface Feature #2479: DEFINE IMAGE and image phrase support for both button and image
>Please do fix both of these issues (character unit mismatches and the regression for IMAGE-SIZE options) as part of ... Eugenie Lyzenko
05:00 PM User Interface Feature #2479: DEFINE IMAGE and image phrase support for both button and image
Eugenie Lyzenko wrote:
> >Have you run your testcases in both windev01 and in P2J? Does our implementation match exa...
Greg Shah
04:59 PM User Interface Feature #2479: DEFINE IMAGE and image phrase support for both button and image
> I've changed the rule file and want to make sure there will be no regression her. Is it OK?
Good point. Yes, c...
Greg Shah
04:57 PM User Interface Feature #2479: DEFINE IMAGE and image phrase support for both button and image
>Have you run your testcases in both windev01 and in P2J? Does our implementation match exactly?
Yes, I'm comparin...
Eugenie Lyzenko
04:41 PM User Interface Feature #2479: DEFINE IMAGE and image phrase support for both button and image
>OK. I don't think this needs full runtime testing. The GUI stuff won't get loaded in MAJIC testing. Even the server-... Eugenie Lyzenko
04:34 PM User Interface Feature #2479: DEFINE IMAGE and image phrase support for both button and image
Have you run your testcases in both windev01 and in P2J? Does our implementation match exactly? Greg Shah
04:32 PM User Interface Feature #2479: DEFINE IMAGE and image phrase support for both button and image
OK. I don't think this needs full runtime testing. The GUI stuff won't get loaded in MAJIC testing. Even the serve... Greg Shah
04:06 PM User Interface Feature #2479: DEFINE IMAGE and image phrase support for both button and image
>1. In Convert3DColorsFilter, are the input colors to be converted really hard coded in the 4GL?
I think so, from ...
Eugenie Lyzenko
03:52 PM User Interface Feature #2479: DEFINE IMAGE and image phrase support for both button and image
Code Review evl_upd20150220a.zip
1. In @Convert3DColorsFilter@, are the input colors to be converted really hard c...
Greg Shah
03:40 PM User Interface Feature #2479: DEFINE IMAGE and image phrase support for both button and image
This updates for review includes runtime implementation of the @CONVERT-3D-COLORS@ option for image and button widget... Eugenie Lyzenko
02:22 PM User Interface Feature #2479: DEFINE IMAGE and image phrase support for both button and image
Code Review evl_upd20150219a.zip
It is good.
Greg Shah
08:01 PM User Interface Feature #2479: DEFINE IMAGE and image phrase support for both button and image
This update adds conversion rule change to support @RETAIN-SHAPE@ sub-option and runtime implementation for @STRETCH-... Eugenie Lyzenko
05:20 PM User Interface Feature #2521 (Closed): enhance/implement radio-set for GUI
Greg Shah
02:31 PM User Interface Feature #2508: test LIST-ITEMS support in P2J and confirm that it is complete, add or fix feature...
Yes, I think you are right. Please test and fix any issues related to those.
Another thing: please test these bot...
Greg Shah
02:26 PM User Interface Feature #2508: test LIST-ITEMS support in P2J and confirm that it is complete, add or fix feature...
I understand the LIST-ITEM-PAIRS and RADIO-BUTTONS attributes are in the scope of this task as well. Is that correct?
Igor Skornyakov
02:16 PM User Interface Feature #1790: implement menu support
At this point are ChUI menus fully functional?
Is there any error handling, event or other missing functionality? ...
Greg Shah
01:56 PM User Interface Feature #1790: implement menu support
Javadoc fix.
Remaining features needing to be implemented is empty at this moment. Probably some features will be ...
Vadim Gindin
01:22 PM User Interface Feature #1790: implement menu support
Please post a list of the remaining features needing to be implemented, enhanced and/or bugs needing fixing. You can... Greg Shah
01:21 PM User Interface Feature #1790: implement menu support
In your next update, please resolve these:... Greg Shah
01:02 PM User Interface Feature #1790: implement menu support
Current update has passed full testing and committed to rev #10766. Vadim Gindin
12:26 PM User Interface Feature #1790: implement menu support
OK, you can commit vig_upd20150220a.zip. Greg Shah
12:12 PM User Interface Feature #1790: implement menu support
Here is merged current update with the latest revision. Vadim Gindin
12:06 PM User Interface Feature #1790: implement menu support
> I suspect that this error is not related with my update. Don't you know why it happens?
We don't know for sure. ...
Greg Shah
11:02 AM User Interface Feature #1790: implement menu support
I've ran regression testing 3rd time. And the test @tc_codes_employees_20@ has passed. So, we can assume that update ... Vadim Gindin
03:28 AM User Interface Feature #1790: implement menu support
Current update has passed conversion testing. I ran regression testing twice. Only one test has failed during both ru... Vadim Gindin
02:00 PM User Interface Feature #2498: implement ADD-LAST() method
Code Review ias_upd20150220c.zip
It looks good.
Greg Shah
01:50 PM User Interface Feature #2498: implement ADD-LAST() method
Greg Shah wrote:
> There is about to be a check in from #1790. When that occurs, please merge up to the latest bzr ...
Igor Skornyakov
12:47 PM User Interface Feature #2498: implement ADD-LAST() method
There is about to be a check in from #1790. When that occurs, please merge up to the latest bzr revision and put tha... Greg Shah
12:42 PM User Interface Feature #2498: implement ADD-LAST() method
The following tests failed in both runs:
57 gso_187 GSO 187 - Initial selection on Contact Master returns to main...
Igor Skornyakov
12:00 PM User Interface Feature #2498: implement ADD-LAST() method
Code Review ias_upd20150220a.zip
The @ScrollDirection@ constructor still needs Javadoc. Obviously, this doesn't i...
Greg Shah
11:57 AM User Interface Feature #2498: implement ADD-LAST() method
> The ChuiOutputManager sets the value of the cursorInvalid in the at method and doesn't reset it in the append(Strin... Greg Shah
03:57 AM User Interface Feature #2498: implement ADD-LAST() method
Greg Shah wrote:
> Igor: in regard to the scrolling problems, I'd like for you to finish the current task without ad...
Igor Skornyakov
03:41 AM User Interface Feature #2498: implement ADD-LAST() method
Constantin Asofiei wrote:
> Please add Keyboard.SE_NEXT_FRAME and Keyboard.SE_PREV_FRAME cases to AbstractWidget.ign...
Igor Skornyakov
03:39 AM User Interface Feature #2498: implement ADD-LAST() method
Greg Shah wrote:
> Code Review ias_upd20150219a.zip
>
> 1. In @RadioButtonImpl.draw()@, why iterate through chara...
Igor Skornyakov
02:34 AM User Interface Feature #2498: implement ADD-LAST() method
About the NPE in note 73 and the w3.p test. A widget can be determine as no longer alive (thus can be deleted) by bo... Constantin Asofiei
02:20 AM User Interface Feature #2498: implement ADD-LAST() method
Igor Skornyakov wrote:
> 3. After some recent changes (as far as I understand - introducing @manageSystemActions()@ ...
Constantin Asofiei
07:39 PM User Interface Feature #2498: implement ADD-LAST() method
Code Review ias_upd20150219a.zip
1. In @RadioButtonImpl.draw()@, why iterate through character by character instea...
Greg Shah
07:08 PM User Interface Feature #2498: implement ADD-LAST() method
Igor: in regard to the scrolling problems, I'd like for you to finish the current task without adding more work to it... Greg Shah
07:03 PM User Interface Feature #2498: implement ADD-LAST() method
Constantin: please comment on note 72 and item 3 in note 85. Greg Shah
01:47 PM User Interface Bug #2520: Dynamic list controls are not visible
Greg Shah wrote:
> Is this still a problem on bzr rev 10766?
Yes, it is.
Igor Skornyakov
01:38 PM User Interface Bug #2520: Dynamic list controls are not visible
Is this still a problem on bzr rev 10766? Greg Shah
01:37 PM User Interface Bug #2520: Dynamic list controls are not visible
The dynamic RADIO-SET and COMBO-BOX are not visible, but are "working" - one can make a "blind" selection with the RA... Greg Shah
07:47 AM User Interface Bug #2520 (Closed): Dynamic list controls are not visible
Igor Skornyakov
01:40 PM User Interface Bug #2518: NPE on exit from testcase with mixed usage of static and dynamic widgets
From CA:
About the NPE in note 73 and the w3.p test. A widget can be determine as no longer alive (thus can be del...
Greg Shah
01:40 PM User Interface Bug #2518: NPE on exit from testcase with mixed usage of static and dynamic widgets
The attached sample after conversion throws NPE on exit:
[02/17/2015 19:25:51 MSK] (StandardServer.invoke:SEVERE) ...
Greg Shah
07:41 AM User Interface Bug #2518 (New): NPE on exit from testcase with mixed usage of static and dynamic widgets
Igor Skornyakov
01:39 PM User Interface Bug #2519: suspicious looking scrolling code
1. The @ScrollBar.scroll()@ method looks to contain a typo: in line 253 @limit.x@ seems to be instead of @limit.y@
2...
Greg Shah
07:43 AM User Interface Bug #2519 (New): suspicious looking scrolling code
Igor Skornyakov
01:36 PM User Interface Feature #2487: implement READ-FILE() and SAVE-FILE() methods in the editor widget
Code Review evk_upd20150218a.zip
It is a good step.
1. @EditorWidget.read/saveFile(character)@ need @null@ and ...
Greg Shah
08:02 PM User Interface Feature #2487: implement READ-FILE() and SAVE-FILE() methods in the editor widget
Added runtime support. I'm not finished testing at the moment, but common functionality is working. Evgeny Kiselev
11:25 AM User Interface Feature #2422: add features to BROWSE
Technically that's not a browse problem, but because of record retrieval logic browse uses, an editable browse easily... Stanislav Lomany

02/19/2015

03:04 PM Database Bug #2488: replace runtime compilation of dynamic queries
Ovidiu Maxiniuc wrote:
> The generated code is unchanged for majic. The attached update passed the conversion test.
...
Eric Faulhaber
12:01 PM Database Bug #2488: replace runtime compilation of dynamic queries
The generated code is unchanged for majic. The attached update passed the conversion test. Ovidiu Maxiniuc
09:44 AM Database Bug #2488: replace runtime compilation of dynamic queries
Eric Faulhaber wrote:
> Where did the need to improve the predicate prep come from? What is the rationale behind the...
Ovidiu Maxiniuc
11:18 AM User Interface Feature #2498: implement ADD-LAST() method
I've finished with scrolling changes. My changes do not affect the @ComboBox@ behavior as this widget doesn't rely on... Igor Skornyakov
09:45 AM User Interface Feature #2479: DEFINE IMAGE and image phrase support for both button and image
Code Review evl_upd20150218a.zip
It looks good.
Greg Shah

02/18/2015

06:06 PM User Interface Feature #2479: DEFINE IMAGE and image phrase support for both button and image
This update for your review adds transparency runtime implementation for image widget. Also it prepares other boolean... Eugenie Lyzenko
05:49 PM User Interface Feature #1790: implement menu support
Code Review vig_upd20150219a.zip
I ma OK with the changes. Please get this update conversion and runtime regressi...
Greg Shah
05:37 PM User Interface Feature #1790: implement menu support
The next update, containing remarks corrections and overrides bug fix Vadim Gindin
04:01 PM User Interface Feature #1790: implement menu support
> Don't you remember where that conversion logic?
See the @process_frame@ function in @annotations/frame_scoping.r...
Greg Shah
03:46 PM User Interface Feature #1790: implement menu support
Greg Shah wrote:
> > Wait a minute. It seems that it is a frames naming (class naming) problem.
>
> This makes se...
Vadim Gindin
02:52 PM User Interface Feature #1790: implement menu support
> Wait a minute. It seems that it is a frames naming (class naming) problem.
This makes sense. Please put some pr...
Greg Shah
02:36 PM User Interface Feature #1790: implement menu support
Greg Shah wrote:
> In the 4GL I suspect that your @DISPLAY@ statements in the triggers are targeting the the default...
Vadim Gindin
02:21 PM User Interface Feature #1790: implement menu support
In the 4GL I suspect that your @DISPLAY@ statements in the triggers are targeting the the default "unnamed" frame. S... Greg Shah
02:00 PM User Interface Feature #1790: implement menu support
I'm debugging difficult bug:... Vadim Gindin
05:41 PM User Interface Feature #2487: implement READ-FILE() and SAVE-FILE() methods in the editor widget
Is there a bug in our implementation of the 4GL built-in function @SEARCH()@ that makes it work differently with file... Greg Shah
05:38 PM User Interface Feature #2487: implement READ-FILE() and SAVE-FILE() methods in the editor widget
Greg Shah wrote:
> I don't think you need to do anything like that. The client-side code would call @FileSystemDaem...
Evgeny Kiselev
05:34 PM User Interface Feature #2487: implement READ-FILE() and SAVE-FILE() methods in the editor widget
I don't think you need to do anything like that. The client-side code would call @FileSystemDaemon.searchPath()@, wh... Greg Shah
05:31 PM User Interface Feature #2487: implement READ-FILE() and SAVE-FILE() methods in the editor widget
Can you suggest what is common "looking" directories in 4gl ? at least common work dir I can take from @Configuration... Evgeny Kiselev
03:40 PM Database Bug #2488: replace runtime compilation of dynamic queries
Code review 0218a:
Where did the need to improve the predicate prep come from? What is the rationale behind the cu...
Eric Faulhaber
02:40 PM Database Bug #2488: replace runtime compilation of dynamic queries
Added support for @LogicalExpression@ anon c'tor.
Refactored some method names to better fit their implementation.
Ovidiu Maxiniuc
02:41 PM Database Bug #2514: incorrect return values for get{First|Next|Last|Previous} methods
This update should fix the return values for respective methods. Ovidiu Maxiniuc
07:07 AM User Interface Feature #2498: implement ADD-LAST() method
Hynek Cihlar wrote:
> I think it won't hurt if you start fixing the immediate pains - the unclear model of scroll ...
Igor Skornyakov
07:04 AM User Interface Feature #2498: implement ADD-LAST() method
Igor Skornyakov wrote:
> I'm not sure that I understand this completely. However it seems that it means that a widge...
Hynek Cihlar
05:03 AM User Interface Feature #2498: implement ADD-LAST() method
Hynek Cihlar wrote:
> Personally I would rather see the scroll event initiated by the widget receiving focus itsel...
Igor Skornyakov

02/17/2015

04:28 PM User Interface Feature #2498: implement ADD-LAST() method
Greg Shah wrote:
> Igor Skornyakov wrote:
> > I see. May I make a suggestion?
> > First of all I think that it is ...
Hynek Cihlar
04:12 PM User Interface Feature #2498: implement ADD-LAST() method
Igor Skornyakov wrote:
> Greg Shah wrote:
>
> > Please provide more specifics about what needs to be different.
...
Hynek Cihlar
04:07 PM User Interface Feature #2498: implement ADD-LAST() method
Igor Skornyakov wrote:
> I see. May I make a suggestion?
> First of all I think that it is important for the scroll...
Greg Shah
02:29 PM User Interface Feature #2498: implement ADD-LAST() method
I see. May I make a suggestion?
First of all I think that it is important for the scroll to be well-defined and idemp...
Igor Skornyakov
02:03 PM User Interface Feature #2498: implement ADD-LAST() method
Igor Skornyakov wrote:
> Greg Shah wrote:
>
> > Please provide more specifics about what needs to be different.
...
Greg Shah
12:59 PM User Interface Feature #2498: implement ADD-LAST() method
Greg Shah wrote:
> Please provide more specifics about what needs to be different.
Well, first of all the whole...
Igor Skornyakov
12:42 PM User Interface Feature #2498: implement ADD-LAST() method
Greg Shah wrote:
> Generally, no it is not OK.
>
> I do see that @RadioButtonImpl.drawButton()@ does not honor th...
Igor Skornyakov
12:40 PM User Interface Feature #2498: implement ADD-LAST() method
Igor Skornyakov wrote:
> Another note. The look of the RadioSet widget at runtime is different in 4GL and Java: in 4...
Greg Shah
12:34 PM User Interface Feature #2498: implement ADD-LAST() method
Igor Skornyakov wrote:
> It seems that the @ScrollContainer.ensureVisibility()@ method's implementation is not suita...
Greg Shah
12:33 PM User Interface Feature #2498: implement ADD-LAST() method
In regard to the NPE, I don't understand why we would be pushing screen definitions to the client during the processi... Greg Shah
11:57 AM User Interface Feature #2498: implement ADD-LAST() method
Another note. The look of the RadioSet widget at runtime is different in 4GL and Java: in 4GL the focused item is hig... Igor Skornyakov
11:34 AM User Interface Feature #2498: implement ADD-LAST() method
It seems that the @ScrollContainer.ensureVisibility()@ method's implementation is not suitable for at least a @RadioS... Igor Skornyakov
11:29 AM User Interface Feature #2498: implement ADD-LAST() method
The sample code is a result of my test evolution - I've added more code while investigating the widget behavior. Plea... Igor Skornyakov
10:02 AM User Interface Feature #2498: implement ADD-LAST() method
The code seems very complicated, with many more frames than I would have expected to be used or needed. Did you get ... Greg Shah
04:13 AM User Interface Feature #2498: implement ADD-LAST() method
Greg Shah wrote:
> 3. About the NPE in @LogicalTerminal.pushScreenDefinition()@, can you show the 4GL code that can ...
Igor Skornyakov
10:44 PM User Interface Feature #2498: implement ADD-LAST() method
Code Review ias_upd20150216a.zip
1. The @RadioSetInterface@ header should be above the import and package statemen...
Greg Shah
03:34 PM Database Bug #2488: replace runtime compilation of dynamic queries
Oops, I forgot the update. here it is. Ovidiu Maxiniuc
03:33 PM Database Bug #2488: replace runtime compilation of dynamic queries
Changes since 0210a:
* Improved @preparePredicate()@ for OPEN QUERY predicates to
- allow one level nested quotes
- a...
Ovidiu Maxiniuc
10:21 PM Database Bug #2488: replace runtime compilation of dynamic queries
Code review 0210a:
The version of @DynamicQueryHelper@ in the update shows no difference from the latest version i...
Eric Faulhaber
08:55 AM User Interface Feature #2412: Implement widget browser
Code Review hc_upd20150217b.zip
It looks good. Please check it in.
Greg Shah
04:54 AM User Interface Feature #2412: Implement widget browser
Missing history entry added and converted tabs to spaces in the whole PropertyTable.java. build.xml removed as this s... Hynek Cihlar
10:48 PM User Interface Feature #2412: Implement widget browser
Code Review hc_upd20150217a.zip
1. @PropertyTable@ needs a history entry. It also needs all of its hard-tab chars...
Greg Shah
08:15 PM User Interface Feature #2412: Implement widget browser
The attached update extends Widget browser with the possibility to copy widget properties to clipboard. This is usefu... Hynek Cihlar
02:06 AM User Interface Feature #2422: add features to BROWSE
> > But I can handle browsing part because BrowseColumnWidget.setDColor(Color) does the job.
>
> I don't understan...
Stanislav Lomany
10:31 PM User Interface Feature #2422: add features to BROWSE
> while BrowseColumnWidget.setDColor(NumberType) doesn't actually set a color because ColorSpec.convert() always retu... Greg Shah

02/16/2015

03:49 PM User Interface Feature #2498: implement ADD-LAST() method
Added support for the HORIZONTAL and EXPAND attributes. Still need to fix scrolling behavior. Igor Skornyakov
03:32 PM User Interface Feature #2422: add features to BROWSE
FYI, Majic alters browse column "colors" in two places. E.g.:... Stanislav Lomany
09:51 AM User Interface Feature #2479: DEFINE IMAGE and image phrase support for both button and image
Committed in bzr as 10757. Eugenie Lyzenko
09:38 AM User Interface Feature #2479: DEFINE IMAGE and image phrase support for both button and image
Check it in. Greg Shah
08:51 AM User Interface Feature #2479: DEFINE IMAGE and image phrase support for both button and image
The runtime testing completed without regressions(separate run for CTRL-C 3 way tests). The results are in @10754_5c9... Eugenie Lyzenko

02/13/2015

02:32 PM Database Bug #2514: incorrect return values for get{First|Next|Last|Previous} methods
Alternative working form for ... Ovidiu Maxiniuc
07:46 AM Database Bug #2514: incorrect return values for get{First|Next|Last|Previous} methods
The set of get{First|Next|Last|Previous} methods should return:
* @?@ (unknown) if the query object handle is invali...
Ovidiu Maxiniuc
07:38 AM Database Bug #2514 (Closed): incorrect return values for get{First|Next|Last|Previous} methods
Ovidiu Maxiniuc
12:38 PM User Interface Feature #2479: DEFINE IMAGE and image phrase support for both button and image
Greg Shah wrote:
> Code Review evl_upd20150212b.zip
>
> It looks good, get it both conversion and runtime regress...
Eugenie Lyzenko
09:22 AM User Interface Feature #2479: DEFINE IMAGE and image phrase support for both button and image
Code Review evl_upd20150212b.zip
It looks good, get it both conversion and runtime regression tested.
Greg Shah
10:53 AM User Interface Feature #1790: implement menu support
Code Review vig_upd20150210a.zip
This is very close.
1. There is a build directory included in your update zip ...
Greg Shah
09:55 AM User Interface Feature #2498: implement ADD-LAST() method
Code Review ias_upd20150212a.zip
It is very good. Some minor things:
1. Add @RadioSetInterface@ to @HandleComm...
Greg Shah
 

Also available in: Atom