FWD » Core Development » Base Language

11/06/2013

09:15 PM Feature #1641: implement SAX XML support

from note 55
1. Fixed
2. Added special wrapper for File/Stream/memptr/longchar. In 4gl method @set-input-source@ is... Evgeny Kiselev

11/02/2013

05:24 PM Bug #2183: fix resource save/restoration when used with string/handle functions

> > (1) "Zero" handles
>
> As a general rule, we always implement any language feature (documented or not) that ca... Hynek Cihlar

11/01/2013

07:32 AM Bug #2183: fix resource save/restoration when used with string/handle functions

> > (2) Resource ID deallocation
>
> Yes, we need to solve this for sure. Constantin is working on something rela... Constantin Asofiei

07:06 AM Bug #2183: fix resource save/restoration when used with string/handle functions

Overall, this is very good work.
+Answers+
> (1) "Zero" handles
As a general rule, we always implement any l... Greg Shah

09:54 PM Bug #2183 (Review): fix resource save/restoration when used with string/handle functions

Attached are the changes to review. The changes passed regression tests.
The source code includes several TODO comme... Hynek Cihlar

10/31/2013

03:48 AM Feature #2196: implement proper resource deletion of resources created in a procedure ran persistent

Greg Shah wrote:
> I suspect the buffer is actually referencing a record whose lock has been downgraded to share-loc... Constantin Asofiei

03:39 AM Feature #1641: implement SAX XML support

# @handle.isHandleTypeOf@
- please do not use an existing class name for variables/parameters. It is confusing. I'm ... Constantin Asofiei

10:01 PM Feature #1641 (WIP): implement SAX XML support

1) I have question about procedure handle:
- in handle I have @ExternalProgramWrapper@ where I can get real instance... Evgeny Kiselev

10/30/2013

12:41 PM Feature #2196: implement proper resource deletion of resources created in a procedure ran persistent

> Thus, record locks do survive, and we can end up having access to buffer referencing an exclusively-locked record o... Greg Shah

12:35 PM Feature #2196: implement proper resource deletion of resources created in a procedure ran persistent

Again, something else I've missed. I've been checking handles for static widgets which were only included in a FORM s... Constantin Asofiei

12:14 PM Feature #2196: implement proper resource deletion of resources created in a procedure ran persistent

> 1. In the persistent procedure case, do 4GL transactions (and buffer scopes/record locks) end at the end of the ext... Constantin Asofiei

11:41 AM Feature #2196: implement proper resource deletion of resources created in a procedure ran persistent

More questions:
1. In the persistent procedure case, do 4GL transactions (and buffer scopes/record locks) end at t... Greg Shah

11:20 AM Feature #2196: implement proper resource deletion of resources created in a procedure ran persistent

I missed a rule; the static resources also get deleted when the internal proc/function which created them ends (regar... Constantin Asofiei

09:59 AM Feature #2196: implement proper resource deletion of resources created in a procedure ran persistent

Does this mean that we just need to defer the processing of TransactionManager.popScope() for external procedures tha... Greg Shah

08:39 AM Feature #2196: implement proper resource deletion of resources created in a procedure ran persistent

Notes about implicit deletion:
* from widget cases, only BUTTON, BROWSE and FILL-IN were tested, but the rules should... Constantin Asofiei

10/29/2013

08:54 AM Feature #2196 (WIP): implement proper resource deletion of resources created in a procedure ran p...

Constantin Asofiei

03:35 AM Feature #2196: implement proper resource deletion of resources created in a procedure ran persistent

From #2120 notes 60, 61, 62 - static queries are undeletable, but their handle can be deleted (and a subsequent HANDL... Constantin Asofiei

10/25/2013

06:04 AM Bug #2197: fix the error message list cleanup when NO-ERROR mode is enabled and ERROR-STATUS:ERRO...

From #1612 notes 27 and 28:
Constantin Asofiei comment:
> remember the ErrorManager.recordOrShowError changes relat... Constantin Asofiei

06:02 AM Bug #2197 (Closed): fix the error message list cleanup when NO-ERROR mode is enabled and ERROR-ST...

Constantin Asofiei

03:13 AM Feature #2196: implement proper resource deletion of resources created in a procedure ran persistent

From #2120 notes 49 and 50:
Comment by Greg Shah:
> Do we need a new "hook" to handle the class unloading "event"... Constantin Asofiei

03:12 AM Feature #2196 (Closed): implement proper resource deletion of resources created in a procedure ra...

Constantin Asofiei

10/21/2013

03:51 AM Feature #1641: implement SAX XML support

Evgeny, following is my review. The main concerns are about abstracting the sax reading and writing, to not write cod... Constantin Asofiei

09:17 PM Feature #1641: implement SAX XML support

First drop for review:
1) SaxAttributes is close to finish version, there are some not important TODOs
2) SaxEntity... Evgeny Kiselev

10/20/2013

05:04 PM Bug #2183 (WIP): fix resource save/restoration when used with string/handle functions

Hynek Cihlar

02:06 PM Feature #2013: implement the SELF system handle

> Perhaps just the NullWidget stuff is open?
Yes, only the NullWidget usage remains to be removed. Constantin Asofiei

01:47 PM Feature #2013: implement the SELF system handle

What is left to do on this task? I think your recent work with async events/sockets probably filled most of the gaps... Greg Shah

01:40 PM Feature #2041: implement runtime support for "shared" attributes that are just stubbed

The BUFFER-FIELD implementation of COLUMN-LABEL, DATA-TYPE, FORMAT and LABEL will be handled in other tasks that are ... Greg Shah

12:09 PM Feature #1781 (Closed): add missing support for the WIDGET-HANDLE builtin function

Greg Shah

10/18/2013

05:39 PM Feature #1781: add missing support for the WIDGET-HANDLE builtin function

Committed to p2j_repo as revision 10401. Hynek Cihlar

12:56 PM Feature #1781: add missing support for the WIDGET-HANDLE builtin function

Check it in and distribute it. Greg Shah

12:46 PM Feature #1781: add missing support for the WIDGET-HANDLE builtin function

Regression tests passed. Hynek Cihlar

02:01 PM Feature #1634: implement full native library (.so or DLL) support

Loading/Unloading
# No loading of a library occurs until a RUN statement is executed which matches a PROCEDURE EXT... Greg Shah

12:24 PM Feature #1634: implement full native library (.so or DLL) support

After a long effort of writing testcases, I have enough results to have sufficient understanding of how the native li... Greg Shah

08:52 AM Feature #1646 (Closed): implement BASE64-ENCODE/BASE64-DECODE built-in functions

Greg Shah

04:44 AM Feature #1641: implement SAX XML support

I think there is a good chance that the progress documentation is wrong; although the code passes, I couldn't find a ... Constantin Asofiei

10/17/2013

07:32 PM Feature #1641: implement SAX XML support

what means in documentation @A CHARACTER or LONGCHAR expression evaluating to the value of the attribute.@ for exampl... Evgeny Kiselev

05:32 PM Feature #1646: implement BASE64-ENCODE/BASE64-DECODE built-in functions

I'm sorry, I forgot..
It passed regression testing. Vadim Gindin

02:12 PM Feature #1646: implement BASE64-ENCODE/BASE64-DECODE built-in functions

What is the status of regression testing? (I know the answer but you were supposed to post it here so there is docum... Greg Shah

01:32 PM Feature #1646: implement BASE64-ENCODE/BASE64-DECODE built-in functions

committed to bzr. Revision 10399. Vadim Gindin

05:04 PM Feature #1781: add missing support for the WIDGET-HANDLE builtin function

Yes, I forgot to mention that the regression test is in progress. Hynek Cihlar

05:02 PM Feature #1781: add missing support for the WIDGET-HANDLE builtin function

It looks good. Please conversion regression test this. Greg Shah

04:52 PM Feature #1781: add missing support for the WIDGET-HANDLE builtin function

Built-in functions ordered alphabetically, see the attached hc_upd20131017a.zip. Hynek Cihlar

02:36 PM Feature #2074: add CAN-QUERY() and CAN-SET() support

Committed to bzr as 10400. Eugenie Lyzenko

02:14 PM Feature #2074: add CAN-QUERY() and CAN-SET() support

Go ahead. Greg Shah

01:54 PM Feature #2074: add CAN-QUERY() and CAN-SET() support

> Runtime testing is enough.
The runtime testing has been passed. The results can be found at 10398_4e69e3d_201310... Eugenie Lyzenko

10/16/2013

11:51 AM Feature #1781: add missing support for the WIDGET-HANDLE builtin function

Sorry, I should have seen this other thing before: all entries in builtin_functions.rules should be in alphabetical o... Greg Shah

11:28 AM Feature #1781 (Review): add missing support for the WIDGET-HANDLE builtin function

Resolved the issues from review.
Attached hc_upd20131016a.zip with p2j changes and hc_upd20131016b.zip with testcas... Hynek Cihlar

10:58 AM Feature #1781: add missing support for the WIDGET-HANDLE builtin function

Code Review 1012a
1. The testcases should not be in the same update zip.
2. There should not be a p2j/ director... Greg Shah

10:37 AM Feature #2074: add CAN-QUERY() and CAN-SET() support

Runtime testing is enough. Greg Shah

10:36 AM Feature #2074: add CAN-QUERY() and CAN-SET() support

> It looks good. Get it tested.
Will the runtime regression be enough or we need conversion testing too? Eugenie Lyzenko

10:14 AM Feature #2074: add CAN-QUERY() and CAN-SET() support

It looks good. Get it tested.
Also, please make sure you add the additional Redmine task to make a more complete ... Greg Shah

09:54 AM Feature #2074: add CAN-QUERY() and CAN-SET() support

1. Moved package private static calls to @date@ class.
2. Modified @DatetimeFormat@ class to reflect the change in 1... Eugenie Lyzenko

09:18 AM Feature #2074: add CAN-QUERY() and CAN-SET() support

>
> > 2. If I understand correctly, I think it is the date version of these which needs to be package private:
Cor... Constantin Asofiei

09:07 AM Feature #2074: add CAN-QUERY() and CAN-SET() support

> 1. LogicalTerminal.getHeightChars(), getWidthChars(),getHeightPixels(), getWidthPixels() have had their annotations... Eugenie Lyzenko

08:20 AM Feature #2074: add CAN-QUERY() and CAN-SET() support

Code Review 1015a
1. LogicalTerminal.getHeightChars(), getWidthChars(),getHeightPixels(), getWidthPixels() have ha... Greg Shah

05:55 AM Feature #2074: add CAN-QUERY() and CAN-SET() support

The minor changes and fix for SessionUtils.java have been uploaded. Eugenie Lyzenko

02:11 AM Feature #2074: add CAN-QUERY() and CAN-SET() support

Eugenie, please do the following changes:
* HandleOps - needs history enry
* LegacyAtribute - change history entry to... Constantin Asofiei

09:44 PM Feature #2074: add CAN-QUERY() and CAN-SET() support

1. and 2. - Done. The new update has been uploaded. In addition to the testcases.
However one question I have face... Eugenie Lyzenko

09:53 PM Feature #2194: Creating testcases to check legacy attributes for resources used in CAN-QUERY/CAN-...

The goal is to provide the 4GL code that is able to check all possible resource/attribute pair for correct result wit... Eugenie Lyzenko

09:48 PM Feature #2194 (New): Creating testcases to check legacy attributes for resources used in CAN-QUER...

Eugenie Lyzenko

10/15/2013

06:56 PM Feature #1646: implement BASE64-ENCODE/BASE64-DECODE built-in functions

Code Review 1014b
I am OK with these changes. Get them regression tested.
> May be I just didn't understand yo... Greg Shah

06:24 PM Feature #2074: add CAN-QUERY() and CAN-SET() support

Code Review 1014a
1. Please change "belonged" to "ignore". Of course, the meaning of the boolean is reversed in t... Greg Shah

04:50 PM Feature #2074: add CAN-QUERY() and CAN-SET() support

The new update uploaded for review.
1. Adding separation for static annotations by belonged() member of the Legacy... Eugenie Lyzenko

01:15 PM Feature #2074: add CAN-QUERY() and CAN-SET() support

> If you re-look the HandleOps code for canQuery/canSet wrappers you will find the code that handles this case:
As l... Constantin Asofiei

12:52 PM Feature #2074: add CAN-QUERY() and CAN-SET() support

> BTW, there is another bug in the code: canQuery/canSet must check if the handle is unknown; in this case these meth... Eugenie Lyzenko

02:50 AM Feature #2074: add CAN-QUERY() and CAN-SET() support

It's a little more complicated than this. Some session handles (like LAST-EVENT, SESSION, etc) are implemented via st... Constantin Asofiei

04:37 PM Bug #2183: fix resource save/restoration when used with string/handle functions

The random 64-bit value is acceptable for now. The 4GL apps have so many other security holes that it isn't worth sp... Greg Shah

04:33 PM Bug #2183: fix resource save/restoration when used with string/handle functions

#1781-15
_Also, for security purposes, we should not make the resource ID algorithm a simple sequence. It should be ... Hynek Cihlar

01:01 PM Feature #1781: add missing support for the WIDGET-HANDLE builtin function

Review request.
The attached changes implement all the points except for the improvements in resource ID allocatio... Hynek Cihlar

10/14/2013

04:33 PM Feature #2074: add CAN-QUERY() and CAN-SET() support

One example. Consider the FORMAT attribute for LAST-EVENT resource.
The handle constructed for this resource will ... Eugenie Lyzenko

10:48 AM Feature #2074: add CAN-QUERY() and CAN-SET() support

> May be the current interfaces class tree does not properly reflect the resource dependency? Don't we need to change... Constantin Asofiei

10:34 AM Feature #2074: add CAN-QUERY() and CAN-SET() support

> Looks like there is a case I've overlooked: for CommonLastEvent interface (defining the LAST-EVENT resource in P2J)... Eugenie Lyzenko

04:18 PM Feature #1646: implement BASE64-ENCODE/BASE64-DECODE built-in functions

I made a corrections and attached an archive.
About LogicalTerminal and ThinClient I want to say following.
1. ... Vadim Gindin

09:41 AM Feature #1646: implement BASE64-ENCODE/BASE64-DECODE built-in functions

Code Review 1014a
1. DynamicOps and TextOps are both missing a history entry.
2. This code in @if(type == null)... Greg Shah

06:44 AM Feature #1646: implement BASE64-ENCODE/BASE64-DECODE built-in functions

I've made proposed corrections. It was enough to make my test case working. Here is and archive. Vadim Gindin

10/12/2013

03:38 PM Feature #2074: add CAN-QUERY() and CAN-SET() support

Looks like there is a case I've overlooked: for @CommonLastEvent@ interface (defining the LAST-EVENT resource in P2J)... Constantin Asofiei

10/11/2013

05:10 PM Feature #1646: implement BASE64-ENCODE/BASE64-DECODE built-in functions

> For this function the converted code is DynamicOps.length(str) and during execution it goes to TextOps.length(Text... Greg Shah

03:50 PM Feature #1646: implement BASE64-ENCODE/BASE64-DECODE built-in functions

Greg Shah wrote:
> Code Review 1010a
> [...]
> 4. In LogicalTerminal, why do this:
>
> [...]
>
> As far as I... Vadim Gindin

12:41 PM Feature #1646: implement BASE64-ENCODE/BASE64-DECODE built-in functions

Interesting... I'll try.
Here is the progress procedure I wroted about.... Vadim Gindin

12:34 PM Feature #1646: implement BASE64-ENCODE/BASE64-DECODE built-in functions

OK, do implement the support for your current findings.
> There is no error, mentioned in the old comment.
I th... Greg Shah

11:42 AM Feature #1646: implement BASE64-ENCODE/BASE64-DECODE built-in functions

There is the old comment in the @SecurityOps.base64Encode@ method: ... Vadim Gindin

10:12 AM Feature #1646: implement BASE64-ENCODE/BASE64-DECODE built-in functions

> How to create memptr with negative size?
Search for REAL_LOWER_BOUND in memptr.java. The short story: in Progre... Greg Shah

09:30 AM Feature #1646: implement BASE64-ENCODE/BASE64-DECODE built-in functions

How to create memptr with negative size? I tried to call @set-size(..)=-100@ but it didn't work. After this call the ... Vadim Gindin

10/10/2013

04:02 PM Feature #1646: implement BASE64-ENCODE/BASE64-DECODE built-in functions

I will make additional testing to answer these questions. Vadim Gindin

02:09 PM Feature #1646: implement BASE64-ENCODE/BASE64-DECODE built-in functions

Code Review 1010a
1. Is this todo in base64Encode() still valid?... Greg Shah

01:37 PM Feature #1646: implement BASE64-ENCODE/BASE64-DECODE built-in functions

I switched to java7, implemented "skip" operator support for a message statement including message truncating. By the... Vadim Gindin

11:41 AM Feature #2074 (Closed): add CAN-QUERY() and CAN-SET() support

Greg Shah

11:39 AM Feature #2074: add CAN-QUERY() and CAN-SET() support

The testcases update. Committed in bzr as 1035. Eugenie Lyzenko

11:14 AM Feature #2074: add CAN-QUERY() and CAN-SET() support

1010b has been committed as 10398. Eugenie Lyzenko

10:54 AM Feature #2074: add CAN-QUERY() and CAN-SET() support

It looks good. Check it in and distribute it. Greg Shah

10:27 AM Feature #2074: add CAN-QUERY() and CAN-SET() support

> This should be the last change for this task.
The new drop fixes the HANDLE attribute issue. The approach does n... Eugenie Lyzenko

09:30 AM Feature #2074: add CAN-QUERY() and CAN-SET() support

Code Review 1010a
I am OK with the changes.
> OK. I'll work on solution.
This should be the last change for ... Greg Shah

09:11 AM Feature #2074: add CAN-QUERY() and CAN-SET() support

This update includes additional fix for ERROR attribute in temp-table. The TempTable.java class has been modified ins... Eugenie Lyzenko

07:52 AM Feature #2074: add CAN-QUERY() and CAN-SET() support

Code Review 1009d
I am fine with all the changes. Greg Shah

07:46 AM Feature #2074: add CAN-QUERY() and CAN-SET() support

> 2. TEMP-TABLE resource should be Errorable. Otherwise we have no support for ERROR rw attribute.
Yes, please fix... Greg Shah

08:03 PM Feature #2074: add CAN-QUERY() and CAN-SET() support

> 2. TEMP-TABLE resource should be Errorable. Otherwise we have no support for ERROR rw attribute.
This issue can ... Eugenie Lyzenko

10:49 AM Feature #1631: implement support for LANDSCAPE/PORTRAIT in I/O statements

I've committed the missed file PrinterStream. New rev is 10397. Vadim Gindin

10:15 AM Feature #1631 (Closed): implement support for LANDSCAPE/PORTRAIT in I/O statements

The update did pass regression testing. Greg Shah

09:54 AM Feature #1631: implement support for LANDSCAPE/PORTRAIT in I/O statements

I've checked in the last update (vig_upd20130924b.zip). Revision number is 10396. Vadim Gindin

10:16 AM Feature #2010 (Closed): runtime support for LANDSCAPE/PORTRAIT I/O options

Greg Shah

10/09/2013

02:58 PM Feature #2074: add CAN-QUERY() and CAN-SET() support

The uploaded drop also fixes the issue with missed annotations for SCHEMA-LOCATION and SCHEMA-PATH in x-document reso... Eugenie Lyzenko

12:20 PM Feature #2074: add CAN-QUERY() and CAN-SET() support

The new update for your review. It fixes almost all deviations in getting annotation. The root cause was the getLegac... Eugenie Lyzenko

10:48 AM Feature #2074: add CAN-QUERY() and CAN-SET() support

The update evl_upd20131009b.zip has been committed in bzr as 10395. Eugenie Lyzenko

10:13 AM Feature #2074: add CAN-QUERY() and CAN-SET() support

> 1. Please add javadoc for the name and setter members of LegacyAttribute.java.
>
> 2. Please add javadoc for the... Eugenie Lyzenko

08:44 AM Feature #2074: add CAN-QUERY() and CAN-SET() support

Eugenie, be aware there is a known case to convert badly in P2J: @h = temp-table tt1:handle@ (getting the handle of a... Constantin Asofiei

08:42 AM Feature #2074: add CAN-QUERY() and CAN-SET() support

After you have checked in your changes, you can continue the debugging. Since these changes affect so many files, I ... Greg Shah

08:39 AM Feature #2074: add CAN-QUERY() and CAN-SET() support

Code Review 1009a
1. Please add javadoc for the name and setter members of LegacyAttribute.java.
2. Please add ... Greg Shah

08:27 AM Feature #2074: add CAN-QUERY() and CAN-SET() support

OK, please work your way through the debugging and report your findings here.
> It can be both CAN-SET/CAN-QUERY i... Greg Shah

08:12 AM Feature #2074: add CAN-QUERY() and CAN-SET() support

> There are many deviations. Are these problems with our CAN-SET/CAN-QUERY implementation or is it some other issue?
... Eugenie Lyzenko

10/08/2013

07:19 PM Feature #1641: implement SAX XML support

Greg Shah wrote:
> When do you expect to have a first version for review?
First version for review will be availabl... Evgeny Kiselev

05:37 AM Feature #1641: implement SAX XML support

When do you expect to have a first version for review? Greg Shah

05:52 PM Feature #2074: add CAN-QUERY() and CAN-SET() support

> I don't understand what SetExp of yes and no mean.
SetExp is Setter Expected
yes - means the attribute should... Eugenie Lyzenko

05:30 PM Feature #2074: add CAN-QUERY() and CAN-SET() support

Please do merge and then re-run the tests.
> Also note in INI file some resources has "?" as SetExp value. This m... Greg Shah

04:31 PM Feature #2074: add CAN-QUERY() and CAN-SET() support

The testcases refresh contains change for can_qset_test1.p to be able to run on 4GL Linux/Windows. For some reasons I... Eugenie Lyzenko

05:36 AM Feature #1646: implement BASE64-ENCODE/BASE64-DECODE built-in functions

This command can be used to display and change the default JDK:... Greg Shah

10/07/2013

06:08 PM Feature #1646: implement BASE64-ENCODE/BASE64-DECODE built-in functions

I got the compile error after update: Category cannot be resolved or is not a field ... Vadim Gindin

03:44 PM Feature #1646: implement BASE64-ENCODE/BASE64-DECODE built-in functions

Please make sure your p2j project is updated to the latest version in bzr. In my version of ErrorManager, the method... Greg Shah

03:30 PM Feature #1646: implement BASE64-ENCODE/BASE64-DECODE built-in functions

Greg Shah wrote:
..
>
> Have you tried the version as @ErrorManager.recordOrShowError(12119, ERROR_12119, false, ... Vadim Gindin

09:17 AM Feature #1646: implement BASE64-ENCODE/BASE64-DECODE built-in functions

> what the method BDT.generateUnknown() needed for?
It is used when you have an instance of a BDT and it can be an... Greg Shah

09:14 AM Feature #1646: implement BASE64-ENCODE/BASE64-DECODE built-in functions

Greg Shah wrote:
> Code Review 1003a
..
> 3. As far as I know, @new longchar()@ does already yield an unknown long... Vadim Gindin

08:25 AM Feature #2074: add CAN-QUERY() and CAN-SET() support

> The INI file and captured 4GL log should also be committed in bzr. Correct?
Yes. Greg Shah

05:25 AM Feature #1612: finish SESSION system handle support

Attached testcases for the high-priority attributes/methods. Constantin Asofiei