https://forge.ispras.ru/https://forge.ispras.ru/favicon.ico?16490126692014-08-21T07:44:25ZOpen-Source ProjectsFortress - Task #5210: [expression] Переименование некоторых сущностейhttps://forge.ispras.ru/issues/5210?journal_id=204152014-08-21T07:44:25ZAndrei Tatarnikovandrewt@ispras.ru
<ul><li><strong>Status</strong> changed from <i>New</i> to <i>Open</i></li><li><strong>% Done</strong> changed from <i>0</i> to <i>30</i></li></ul><p>Class NodeExpr was renamed to NodeOperation - r572.</p> Fortress - Task #5210: [expression] Переименование некоторых сущностейhttps://forge.ispras.ru/issues/5210?journal_id=204182014-08-21T15:55:08ZAndrei Tatarnikovandrewt@ispras.ru
<ul></ul><blockquote>
<p>OperationReducer - упрощается не операция (оператор), а выражение - ExpressionReducer (или ExprReducer) представляется более адекватным названием.</p>
</blockquote>
<p>Здесь как раз все так. OperationReducer и Transformer принимают параметры типа NodeOperation и упрощают их (переменные и константы упрощать нет смысла). Если бы OperationReducer работал с объектами типа Node, тогда переименование имело бы смысл. Нужно ли так сделать?</p> Fortress - Task #5210: [expression] Переименование некоторых сущностейhttps://forge.ispras.ru/issues/5210?journal_id=204222014-08-22T09:49:57ZAndrei Tatarnikovandrewt@ispras.ru
<ul></ul><p>OperationReducer остался как есть, но сделан package-private. А Transformer.reduce может принимать теперь объекты типа Node (если не операции, то возващаются без изменений).</p> Fortress - Task #5210: [expression] Переименование некоторых сущностейhttps://forge.ispras.ru/issues/5210?journal_id=204232014-08-22T09:54:34ZAndrei Tatarnikovandrewt@ispras.ru
<ul><li><strong>Status</strong> changed from <i>Open</i> to <i>Resolved</i></li><li><strong>% Done</strong> changed from <i>30</i> to <i>100</i></li></ul><p>Пока не вижу других проблем. Пакет transformer пока сырой, но это отдельная проблема.</p> Fortress - Task #5210: [expression] Переименование некоторых сущностейhttps://forge.ispras.ru/issues/5210?journal_id=205672014-09-15T11:10:09ZAndrei Tatarnikovandrewt@ispras.ru
<ul><li><strong>Status</strong> changed from <i>Resolved</i> to <i>Closed</i></li><li><strong>Published in build</strong> set to <i>140915</i></li></ul>