Project

General

Profile

[logo] 
 
Home
News
Activity
About/Contact
Major Tools
  Dinotrace
  Verilator
  Verilog-mode
  Verilog-Perl
Other Tools
  BugVise
  CovVise
  Force-Gate-Sim
  Gspice
  IPC::Locker
  Rsvn
  SVN::S4
  Voneline
  WFH
General Info
  Papers

Issue #1421

gtkwave/lz4.c uses the "register" storage class, which is deprecated in C++17, removed afterwards

Added by Pieter Kapsenberg 5 months ago. Updated 5 months ago.

Status:
Closed
Priority:
Normal
Category:
Configure/Make/Compiling
% Done:

0%


Description

https://en.cppreference.com/w/cpp/keyword/register

Line 251:
https://en.cppreference.com/w/cpp/keyword/register
Fix is to just remove the specifier. Currently compiling with warnings on will expose this (Clang-7 at least).

History

#1 Updated by Pieter Kapsenberg 5 months ago

Woops - line 251:
static unsigned LZ4_NbCommonBytes (register size_t val)

#2 Updated by Wilson Snyder 5 months ago

  • Status changed from New to Resolved

Pushed to git towards 4.013.

Note this file comes from GTKwave sources. I've emailed Tony Bybell to fix it, it would save a step next time to please ask GTKWave directly to fix it, then ask Verilator to pull the fix.

#3 Updated by Pieter Kapsenberg 5 months ago

Thanks Wilson. Will do for next time.

#4 Updated by Wilson Snyder 5 months ago

  • Status changed from Resolved to Closed

In 4.014.

Also available in: Atom