building inside building test

[anonymous]

• it took too many time, even if there's no buildings.
• it checks only buildings with "building=yes" (i suppose it should checking everything what have tag "building", except "building=no")

[mrwojo]

• The apparent duration of the "building in building" check is affected by #7759. That bug makes it looks slow even when it's actually quick.

• Also need to consider normal uses of building=entrance.

[anonymous]

...

private static boolean isBuilding(OsmPrimitive p) {
  String v = p.get("building");
  return v != null && !v.equals("no") && !v.equals("entrance");
}

[Don-vip]

In 5293/josm:

fix #7758 - Do not restrict building validator tests to "building=yes", but take into account "no" and "entrance"