Dyalog / link

Source code for Link – the built-in component that enables the use of text files as the primary storage mechanism for APL source code
https://dyalog.github.io/link
MIT License
19 stars 11 forks source link

Link should not use the term "failed to fix", it is confusing #625

Open mkromberg opened 4 months ago

mkromberg commented 4 months ago
      ]link.create # c:\tmp\aplabug
Linked: # ←→ c:\tmp\aplabug [directory was created] 
      (⊂'aaa' 'bbb' 'ccc')⎕NPUT'c:\tmp\aplabug\list.apla'
Link Warning: ⎕SE.Link.Notify: created c:/tmp/aplabug/list.apla: creating #.aaa - failed to fix file
Link Warning: ⎕SE.Link.Notify: changed c:/tmp/aplabug/list.apla: updating un-linked #.aaa - failed to fix file
      ⎕nl -⍳10
(returns nothing)
abrudz commented 4 months ago

The given file should indeed fail to fix, as

aaa
bbb
ccc

isn't valid APLAN.

mkromberg commented 4 months ago

The problem here was that the "failed to fix" led me to believe that we had passed the contents to ⎕FIX rather than trying to de-serialise the array.