Project

General

Profile

Actions

Bug #7942

closed

uts: drm does not need to include font.h

Added by Toomas Soome about 5 years ago. Updated about 5 years ago.

Status:
Closed
Priority:
Normal
Assignee:
-
Category:
kernel
Start date:
2017-03-06
Due date:
% Done:

100%

Estimated time:
Difficulty:
Bite-size
Tags:
Gerrit CR:

Description

Byte size issue - common/io/drm/drm_sunmod.h is set to include sys/font.h, but there is nothing used from it.

The problem was revealed by a bit different issue, in my development I am using sys/queue.h in font.h and for whatever reason, there is also duplicate queue.h in drm tree. While investigating, I found out that the font.h is not actually needed in drm.

Actions #1

Updated by Electric Monk about 5 years ago

  • Status changed from In Progress to Closed
  • % Done changed from 90 to 100

git commit 78b029b7f02730d3acc32084cd9fc419a0d0fb50

commit  78b029b7f02730d3acc32084cd9fc419a0d0fb50
Author: Toomas Soome <tsoome@me.com>
Date:   2017-03-06T20:18:40.000Z

    7942 uts: drm does not need to include font.h
    Reviewed by: Robert Mustacchi <rm@joyent.com>
    Reviewed by: Yuri Pankov <yuri.pankov@gmail.com>
    Reviewed by: Jason King <jason.brian.king@gmail.com>
    Approved by: Richard Lowe <richlowe@richlowe.net>

Actions

Also available in: Atom PDF