On 01/25/2013 01:20 AM, Simon Sapin wrote:
Hi dev-servo,
I started a rust implementation of the css3-syntax editor’s draft:
https://github.com/SimonSapin/rust-cssparser/
http://dev.w3.org/csswg/css3-syntax/
This is very much work in progress (right now it only has the
tokenizer) but I hope it will be useful to servo eventually.
I’m learning rust along the way, so any feedback is welcome.
Have fun,
Hi Simon,
I'm super excited about this! Good luck.
I took a quick peek and noticed that you have both a `cssparser.rc` and
`cssparser.rs`. This pattern is more or less deprecated, a relic from a
time when `rc` files had a special syntax (I'm assuming you copied this
pattern from some of the other servo submodules that were ported forward
from older language revisions). I would recommend putting the content of
`cssparser.rs` directly in `cssparser.rc`. Some background on this
topic: https://mail.mozilla.org/pipermail/rust-dev/2012-December/002679.html
Regards,
Brian
_______________________________________________
dev-servo mailing list
dev-servo@lists.mozilla.org
https://lists.mozilla.org/listinfo/dev-servo
