Changes between Initial Version and Version 1 of Ticket #5885, comment 3


Ignore:
Timestamp:
2020-05-29T13:02:11Z (5 years ago)
Author:
Mahder Alemseged Gebremedhin

Legend:

Unmodified
Added
Removed
Modified

  • Ticket #5885, comment 3

    initial v1  
    55As you can see it only checks for subs in the last part of a cref. Therefore it will treat this as scalar.
    66
    7 I tried fixing this a while back. however, this causes failures in the testsuite. There are things built up on this expectation that only subscripts of the last identifier matter. We have built up on this that it was not easy to fix. Plus at the time working to fix the testsuite (i.e. old NF testing) was deemed a time waste. We might have to revisit it now for NF.
     7I tried fixing this a while back. however, this causes failures in the testsuite. There are things built up on this expectation that only subscripts of the last identifier matter. We have built up on this that it was not easy to fix. Plus at the time working to fix the testsuite (i.e. old NF testing) was deemed a time waste. We might have to revisit it now for NF.  Even better that things have changed since then so this might be easy to fix.
    88
    99I am not sure why removing the subscript gives wrong code. Can you post the generated code?