Agda-stdlib: Set the fixity of mixfix definitions

Created on 12 Jan 2018  路  8Comments  路  Source: agda/agda-stdlib

Set the fixity of _

bug low-hanging-fruit task

All 8 comments

Closed with 5f4c67eca643120a275bea894716c76cc9f2defc

Using the new -Wall option which I have just introduced, I have
found a lot more mixfix definitions which do not come with a fixity
declaration:

https://gist.github.com/gallais/79386a6ba8826620dc82b3f5d5af2bde

Well, bracketing constructs like [_] or [_,_] do not need a fixity, in fact, since they are neither pre/post nor infix operators, a precedence or associativity value for these does not make sense.
The warning mechanism should be adapted to disregard these.

I've updated the gist with the new set of warnings ignoring well-bracketed mixfix decls

As of Agda 2.6.0 the warnings for the missing fixities should now work properly, so this should be much more doable.

Particularly egregious example in Relation.Binary.Construct.NonStrictToStrict

@gallais can you update the gist please? I think I'm going to spend some of my time working on this.

@JacquesCarette It's just a matter of running agda on Everything with -Wall.

Was this page helpful?
0 / 5 - 0 ratings

Related issues

gallais picture gallais  路  7Comments

gallais picture gallais  路  8Comments

MatthewDaggitt picture MatthewDaggitt  路  5Comments

TOTBWF picture TOTBWF  路  4Comments

andreasabel picture andreasabel  路  6Comments