I would have liked to see it a little bigger, but it looks pretty good. Cities are probably one of the trickier types of map, too. Well, with magic, city walls could be cheaper than in reality and they might be built with some expansion in mind (and meanwhile, in case of siege, you've got room for tents and some grazing for animals). I would, however, put the graveyard outside the walls in a world where I had to worry about the dead rising from an unquiet grave.