Bug 14691 - libSM ansification and bug fixes.
Summary: libSM ansification and bug fixes.
Status: RESOLVED NOTABUG
Alias: None
Product: xorg
Classification: Unclassified
Component: Lib/other (show other bugs)
Version: git
Hardware: Other All
: medium normal
Assignee: Xorg Project Team
QA Contact: Xorg Project Team
URL:
Whiteboard:
Keywords: patch
Depends on:
Blocks:
 
Reported: 2008-02-26 12:26 UTC by Paulo César Pereira de Andrade
Modified: 2008-07-23 13:39 UTC (History)
0 users

See Also:
i915 platform:
i915 features:


Attachments
0001-libSM-ansification.patch (16.28 KB, patch)
2008-02-26 12:26 UTC, Paulo César Pereira de Andrade
no flags Details | Splinter Review
0002-Allow-globals.h-to-be-included-by-multiple-files.patch (1.87 KB, patch)
2008-02-26 12:26 UTC, Paulo César Pereira de Andrade
no flags Details | Splinter Review
0003-Add-missing-prototypes.patch (3.03 KB, patch)
2008-02-26 12:27 UTC, Paulo César Pereira de Andrade
no flags Details | Splinter Review
0004-GCC-warning-fix-cast-const-char-to-char-to-m.patch (1.57 KB, patch)
2008-02-26 12:27 UTC, Paulo César Pereira de Andrade
no flags Details | Splinter Review
0001-libSM-ansification.patch (16.51 KB, patch)
2008-03-13 18:37 UTC, Paulo César Pereira de Andrade
no flags Details | Splinter Review

Description Paulo César Pereira de Andrade 2008-02-26 12:26:17 UTC
Created attachment 14591 [details] [review]
0001-libSM-ansification.patch

Like the libICE, these should have no problems as there should
not exist type promotion.

  Only problem not handled is definition of prototypes of internal
libICE functions. This probably should also be handled in libICE,
to have a single header files to define those private functions,
instead of local globals.h header file.
Comment 1 Paulo César Pereira de Andrade 2008-02-26 12:26:45 UTC
Created attachment 14592 [details] [review]
0002-Allow-globals.h-to-be-included-by-multiple-files.patch
Comment 2 Paulo César Pereira de Andrade 2008-02-26 12:27:13 UTC
Created attachment 14593 [details] [review]
0003-Add-missing-prototypes.patch
Comment 3 Paulo César Pereira de Andrade 2008-02-26 12:27:42 UTC
Created attachment 14594 [details] [review]
0004-GCC-warning-fix-cast-const-char-to-char-to-m.patch
Comment 4 Paulo César Pereira de Andrade 2008-03-13 18:37:39 UTC
Created attachment 15102 [details] [review]
0001-libSM-ansification.patch

  Missed one K&R definition, found now, when compiling
with -Wold-style-definition
Comment 5 Paulo César Pereira de Andrade 2008-07-23 13:39:25 UTC
  This was one of the first ansification patches I sent,
and the description already tells that it is not complete
because some prototypes defined in C source files were not
moved to globals.h.

  Other than that, the patch is trivial, but doesn't address
any real bugs.


Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.