Archived Beta Discussions > SlickEdit 2015 v20 Beta Discussion
RC4 - System Verilog Beautify selection - really broken
tim_k:
Start with this:
if (roiEnable)
begin
if (roiLoad)
begin
roiFrameBuffer0StartAddr <= frameBuffer0StartAddr + roiStartOffset;
roiFrameBuffer1StartAddr <= frameBuffer1StartAddr + roiStartOffset;
roiFrameBuffer2StartAddr <= frameBuffer2StartAddr + roiStartOffset;
roiFrameBuffer3StartAddr <= frameBuffer3StartAddr + roiStartOffset;
end
end
else
begin
roiFrameBuffer0StartAddr <= frameBuffer0StartAddr;
roiFrameBuffer1StartAddr <= frameBuffer1StartAddr;
roiFrameBuffer2StartAddr <= frameBuffer2StartAddr;
roiFrameBuffer3StartAddr <= frameBuffer3StartAddr;
end
Select it, then beautify with the default, or modified formatting. Get this:
if(roiEnable)
begin
if(roiLoad)
begin
roiFrameBuffer0StartAddr <= frameBuffer0StartAddr + roiStartOffset;
roiFrameBuffer1StartAddr <= frameBuffer1StartAddr + roiStartOffset;
roiFrameBuffer2StartAddr <= frameBuffer2StartAddr + roiStartOffset;
roiFrameBuffer3StartAddr <= frameBuffer3StartAddr + roiStartOffset;
end
end
else
begin
roiFrameBuffer0StartAddr <= frameBuffer0StartAddr;
roiFrameBuffer1StartAddr <= frameBuffer1StartAddr;
roiFrameBuffer2StartAddr <= frameBuffer2StartAddr;
roiFrameBuffer3StartAddr <= frameBuffer3StartAddr;
end
It doesn't matter what I do to the beautify format options, I always get this. If I beautify an entire file, it works fine.
Same behavior on Mac or Linux.
-Tim
patrick:
Reproduced it, I'm taking a look at it. Thanks for the report.
tim_k:
Wow, that was fast.
While you're at it, this doesn't change if beautified as a selection:
localRoiEnable <= roiEnable;
localRoiStartLine <= roiStartLine;
localRoiStartPix <= roiStartPix;
localRoiWidth <= roiWidth;
localRoiHeight <= roiHeight;
But as a file beautifiy it gets fixed to this:
localRoiEnable <= roiEnable;
localRoiStartLine <= roiStartLine;
localRoiStartPix <= roiStartPix;
localRoiWidth <= roiWidth;
localRoiHeight <= roiHeight;
-Tim
patrick:
Yeah, I introduced a bug at a high level for the beautification of selections, so there aren't many selections that will do the right thing. On the good side, it's easy enough to fix, I'm testing the fix now...
patrick:
Fixed in rc5.
Navigation
[0] Message Index
[#] Next page
Go to full version