HI, thanks for signing up and editing OpenStreetMap. As you requested a review I'm just having a look at your edit. 

Overall the edit is good. One thing to consider is sharing nodes between the rivers and areas such as the river and dam rather than  leaving a gap, unless the gap does make sense or there is a clear area.

The same would be true of the area where the woods have crossed over the dam. I would align these up and take into account that the tree cover will be above the dam, but at the ground level they are distinct area. 