Skip to content

Commit e3347fa

Browse files
committed
Rename checkSatisfaction -> I.satisfiesSchema, typecheckPresentation -> IP.typecheck. #148
1 parent 9777270 commit e3347fa

File tree

3 files changed

+11
-12
lines changed

3 files changed

+11
-12
lines changed

src/Language/CQL.hs

+3-5
Original file line numberDiff line numberDiff line change
@@ -49,7 +49,7 @@ import Data.Typeable
4949
import Language.CQL.Common as C
5050
import Language.CQL.Graph
5151
import Language.CQL.Instance as I
52-
import Language.CQL.Instance.Presentation as IP
52+
import qualified Language.CQL.Instance.Presentation as IP
5353
import Language.CQL.Mapping as M
5454
import Language.CQL.Options
5555
import Language.CQL.Parser.Program (parseProgram)
@@ -320,11 +320,9 @@ instance Evalable SchemaExp SchemaEx where
320320
getOptions = getOptionsSchema
321321

322322
instance Evalable InstanceExp InstanceEx where
323-
324-
-- | Calls @checkSatisfaction@.
325323
validate (InstanceEx x) = do
326-
typecheckPresentation (schema x) (pres x)
327-
checkSatisfaction x
324+
IP.typecheck (schema x) (pres x)
325+
I.satisfiesSchema x
328326
eval prog env exp = do
329327
i <- evalInstance prog env exp
330328
o' <- toOptions (other env) $ getOptions exp

src/Language/CQL/Instance.hs

+6-5
Original file line numberDiff line numberDiff line change
@@ -51,7 +51,8 @@ import Data.Typeable hiding (typeOf)
5151
import Data.Void
5252
import Language.CQL.Collage (Collage(..), assembleGens, attsFrom, fksFrom, typeOf)
5353
import Language.CQL.Common (elem', intercalate, fromListAccum, mapl, section, toMapSafely, Deps(..), Err, Kind(INSTANCE), MultiTyMap, TyMap, type (+))
54-
import Language.CQL.Instance.Presentation (Presentation(..), presToCol, typecheckPresentation, eqs0)
54+
import Language.CQL.Instance.Presentation (Presentation(..), presToCol, eqs0)
55+
import qualified Language.CQL.Instance.Presentation as IP (typecheck)
5556
import Language.CQL.Mapping as Mapping
5657
import Language.CQL.Options
5758
import Language.CQL.Prover
@@ -133,12 +134,12 @@ down1 (Gen g) = Gen g
133134
down1 (Fk f a) = Fk f $ down1 a
134135
down1 _ = error "Anomaly: please report. Function name: down1."
135136

136-
-- | Checks that an instance satisfies its schema.
137-
checkSatisfaction
137+
-- | Checks that an 'Instance' satisfies its 'Schema'.
138+
satisfiesSchema
138139
:: (MultiTyMap '[Show] '[var, ty, sym, en, fk, att, gen, sk, x, y], Eq x)
139140
=> Instance var ty sym en fk att gen sk x y
140141
-> Err ()
141-
checkSatisfaction (Instance sch pres' dp' alg) = do
142+
satisfiesSchema (Instance sch pres' dp' alg) = do
142143
mapM_ (\(EQ (l, r)) -> if hasTypeType l then report (show l) (show r) (instEqT l r) else report (show l) (show r) (instEqE l r)) $ Set.toList $ eqs0 pres'
143144
mapM_ (\(en'', EQ (l, r)) -> report (show l) (show r) (schEqT l r en'')) $ Set.toList $ obs_eqs sch
144145
mapM_ (\(en'', EQ (l, r)) -> report (show l) (show r) (schEqE l r en'')) $ Set.toList $ path_eqs sch
@@ -559,7 +560,7 @@ evalInstanceRaw
559560
evalInstanceRaw ops ty' t is = do
560561
(i :: [Presentation var ty sym en fk att Gen Sk]) <- doImports is
561562
r <- evalInstanceRaw' ty' t i
562-
_ <- typecheckPresentation ty' r
563+
_ <- IP.typecheck ty' r
563564
l <- toOptions ops $ instraw_options t
564565
if bOps l Interpret_As_Algebra
565566
then do

src/Language/CQL/Instance/Presentation.hs

+2-2
Original file line numberDiff line numberDiff line change
@@ -72,12 +72,12 @@ instance (NFData ty, NFData sym, NFData en, NFData fk, NFData att, NFData gen, N
7272
rnf (Presentation g s e) = deepseq g $ deepseq s $ rnf e
7373

7474
-- | Checks that an instance presentation is a well-formed theory.
75-
typecheckPresentation
75+
typecheck
7676
:: (MultiTyMap '[Show, Ord, NFData] '[var, ty, sym, en, fk, att, gen, sk])
7777
=> Schema var ty sym en fk att
7878
-> Presentation var ty sym en fk att gen sk
7979
-> Err ()
80-
typecheckPresentation sch p = typeOfCol $ presToCol sch p
80+
typecheck sch p = typeOfCol $ presToCol sch p
8181

8282
--created as an alias because of name clashes
8383
eqs0

0 commit comments

Comments
 (0)