2012-07-16 17:27:04 -07:00
|
|
|
/// Correctness for protocols
|
|
|
|
|
|
|
|
/*
|
|
|
|
|
|
|
|
This section of code makes sure the protocol is likely to generate
|
|
|
|
correct code. The correctness criteria include:
|
|
|
|
|
|
|
|
* No protocols transition to states that don't exist.
|
|
|
|
* Messages step to states with the right number of type parameters.
|
|
|
|
|
|
|
|
In addition, this serves as a lint pass. Lint warns for the following
|
|
|
|
things.
|
|
|
|
|
|
|
|
* States with no messages, it's better to step to !.
|
|
|
|
|
|
|
|
It would also be nice to warn about unreachable states, but the
|
|
|
|
visitor infrastructure for protocols doesn't currently work well for
|
|
|
|
that.
|
|
|
|
|
|
|
|
*/
|
|
|
|
|
2012-09-04 11:37:29 -07:00
|
|
|
use ext::base::ext_ctxt;
|
2012-07-16 17:27:04 -07:00
|
|
|
|
2012-09-04 11:37:29 -07:00
|
|
|
use proto::{state, protocol, next_state};
|
2012-07-16 17:27:04 -07:00
|
|
|
|
2012-08-07 18:10:06 -07:00
|
|
|
impl ext_ctxt: proto::visitor<(), (), ()> {
|
2012-07-16 17:27:04 -07:00
|
|
|
fn visit_proto(_proto: protocol,
|
|
|
|
_states: &[()]) { }
|
|
|
|
|
|
|
|
fn visit_state(state: state, _m: &[()]) {
|
|
|
|
if state.messages.len() == 0 {
|
|
|
|
self.span_warn(
|
2012-07-24 16:58:48 -07:00
|
|
|
state.span, // use a real span!
|
2012-08-22 17:24:52 -07:00
|
|
|
fmt!("state %s contains no messages, \
|
2012-07-16 17:27:04 -07:00
|
|
|
consider stepping to a terminal state instead",
|
2012-08-22 17:24:52 -07:00
|
|
|
state.name))
|
2012-07-16 17:27:04 -07:00
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2012-10-15 14:56:42 -07:00
|
|
|
fn visit_message(name: ~str, _span: span, _tys: &[@ast::Ty],
|
2012-07-16 17:27:04 -07:00
|
|
|
this: state, next: next_state) {
|
2012-08-06 12:34:08 -07:00
|
|
|
match next {
|
2012-08-20 12:23:37 -07:00
|
|
|
Some({state: next, tys: next_tys}) => {
|
2012-07-17 12:51:24 -07:00
|
|
|
let proto = this.proto;
|
|
|
|
if !proto.has_state(next) {
|
|
|
|
// This should be a span fatal, but then we need to
|
|
|
|
// track span information.
|
2012-07-16 17:27:04 -07:00
|
|
|
self.span_err(
|
2012-07-24 16:58:48 -07:00
|
|
|
proto.get_state(next).span,
|
2012-08-22 17:24:52 -07:00
|
|
|
fmt!("message %s steps to undefined state, %s",
|
|
|
|
name, next));
|
2012-07-17 12:51:24 -07:00
|
|
|
}
|
|
|
|
else {
|
|
|
|
let next = proto.get_state(next);
|
|
|
|
|
|
|
|
if next.ty_params.len() != next_tys.len() {
|
|
|
|
self.span_err(
|
2012-07-24 16:58:48 -07:00
|
|
|
next.span, // use a real span
|
2012-08-22 17:24:52 -07:00
|
|
|
fmt!("message %s target (%s) \
|
2012-07-17 12:51:24 -07:00
|
|
|
needs %u type parameters, but got %u",
|
2012-07-18 16:18:02 -07:00
|
|
|
name, next.name,
|
2012-07-17 12:51:24 -07:00
|
|
|
next.ty_params.len(),
|
2012-08-22 17:24:52 -07:00
|
|
|
next_tys.len()));
|
2012-07-17 12:51:24 -07:00
|
|
|
}
|
2012-07-16 17:27:04 -07:00
|
|
|
}
|
|
|
|
}
|
2012-08-20 12:23:37 -07:00
|
|
|
None => ()
|
2012-07-16 17:27:04 -07:00
|
|
|
}
|
|
|
|
}
|
2012-10-15 14:56:42 -07:00
|
|
|
}
|