Liquidsoap

foo.bar["gni"] not working in records (LS-592) branch.

Details

  • Type: Bug Bug
  • Status: Resolved Resolved
  • Priority: Blocker Blocker
  • Resolution: Fixed
  • Affects Version/s: 1.0.0+dev
  • Fix Version/s: None
  • Component/s: None
  • Labels:
    None

Description

As said in the title, foo.bar["gni"] (syntactic sugar for list.assoc("gni", foo.bar)) isn't working.

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Samuel Mimram added a comment -
Are you sure that it is not related to the fact that list.assoc is not hardcoded, but is in utils.liq?

$ rlwrap ./liquidsoap --interactive
# list.assoc;;
At line 1, char 11-12:
  this value has type
    [append : _, fold : _, hd : _, iter : _, length : _, map : _, mem : _,
     nth : _, remove : _, rev : _, sort : _, tl : _, ?'_a]
  but it should be a subtype of
    [assoc : _, ...]
Show
Samuel Mimram added a comment - Are you sure that it is not related to the fact that list.assoc is not hardcoded, but is in utils.liq? $ rlwrap ./liquidsoap --interactive # list.assoc;; At line 1, char 11-12:   this value has type     [append : _, fold : _, hd : _, iter : _, length : _, map : _, mem : _,      nth : _, remove : _, rev : _, sort : _, tl : _, ?'_a]   but it should be a subtype of     [assoc : _, ...]
Hide
Permalink
Samuel Mimram added a comment -
Actually, it seems that pervasive is broken at the moment... Did you have to rename fallback into source.fallback (and the same for switch)? It will cause a lot of backward-compatibility issues...
Show
Samuel Mimram added a comment - Actually, it seems that pervasive is broken at the moment... Did you have to rename fallback into source.fallback (and the same for switch)? It will cause a lot of backward-compatibility issues...
Hide
Permalink
Samuel Mimram added a comment -
Ok, we also have to adapt the parser. I'll take care of that soon.
Show
Samuel Mimram added a comment - Ok, we also have to adapt the parser. I'll take care of that soon.
Hide
Permalink
Samuel Mimram added a comment -
Fixed in 270aee56cc6a.
Show
Samuel Mimram added a comment - Fixed in 270aee56cc6a.

People

  • Assignee:
    Unassigned
    Reporter:
    Romain Beauxis
Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: