Project

General

Profile

Actions

Optimization #6855

closed
VJ JL

src: var code cleanups

Optimization #6855: src: var code cleanups

Added by Victor Julien about 2 years ago. Updated about 1 year ago.

Status:
Closed
Priority:
Normal
Assignee:
Target version:
Effort:
Difficulty:
Label:

Description

GenericVar and related types use a field "type" which is 8 bits, but it is used to hold the sigmatch type, which is 16 bits. It's not currently causing issues, but needs to be cleaned up.

PA Updated by Philippe Antoine about 2 years ago Actions #1

How come we did not have a compiler warning for this ?..

PA Updated by Philippe Antoine about 2 years ago Actions #2

Philippe Antoine wrote in #note-1:

How come we did not have a compiler warning for this ?..

Looks like GenericVar type is never set by itself, but setting it in a FlowVar or such and then casting to this common struct...

JL Updated by Jeff Lucovsky over 1 year ago Actions #3

  • Status changed from New to In Review
  • Assignee changed from OISF Dev to Jeff Lucovsky
  • Target version changed from TBD to 8.0.0-beta1

JL Updated by Jeff Lucovsky about 1 year ago Actions #4

  • Status changed from In Review to Closed
Actions

Also available in: PDF Atom