From ab908b8c44325dc6f942b77b5617e2522bb0f315 Mon Sep 17 00:00:00 2001
From: Jean-Paul Chaput <Jean-Paul.Chaput@lip6.fr>
Date: Mon, 19 Apr 2021 14:29:23 +0200
Subject: [PATCH] Issue an early warning for offgrid coordinates in GdsParser.

---
 crlcore/src/ccore/gds/GdsParser.cpp | 10 +++++++++-
 1 file changed, 9 insertions(+), 1 deletion(-)

diff --git a/crlcore/src/ccore/gds/GdsParser.cpp b/crlcore/src/ccore/gds/GdsParser.cpp
index 55762539..19c6982a 100644
--- a/crlcore/src/ccore/gds/GdsParser.cpp
+++ b/crlcore/src/ccore/gds/GdsParser.cpp
@@ -1327,11 +1327,19 @@ namespace {
 
   void  GdsStream::xyToComponent ( const Layer* layer )
   {
+    DbU::Unit oneGrid = DbU::fromGrid( 1 );
+    
     vector<Point>   points;
     vector<int32_t> coordinates = _record.getInt32s();
-    for ( size_t i=0 ; i<coordinates.size() ; i += 2 )
+    for ( size_t i=0 ; i<coordinates.size() ; i += 2 ) {
       points.push_back( Point( coordinates[i  ]*_scale
                              , coordinates[i+1]*_scale ) );
+      if ( (points.back().getX() % oneGrid) or (points.back().getX() % oneGrid) ) {
+        cerr << Error( "GdsStream::xyToComponent(): Offgrid %s (foundry grid: %s)."
+                     , getString(points.back()).c_str()
+                     , DbU::getValueString(oneGrid).c_str() ) << endl;
+      }
+    }
 
     _stream >> _record;