patchFreeciv - Patches: patch #2831, Alternative to gui-qt function...

 
 
Show feedback again

patch #2831: Alternative to gui-qt function pointer layer

Submitted by:  Christian Prochaska <cproc>
Submitted on:  Sun 24 Jul 2011 08:57:15 PM UTC  
 
Category: client-qtPriority: 5 - Normal
Status: Wont DoPrivacy: Public
Assigned to: mir3x <mir3x>Open/Closed: Closed
Planned Release: 

Add a New Comment (Rich MarkupRich Markup):
   

You are not logged in

Please log in, so followups can be emailed to you.

 

Sat 22 Jun 2013 12:13:35 PM UTC, comment #5:

Not applicable any more.

mir3x <mir3x>
Project MemberIn charge of this item.
Thu 18 Aug 2011 11:26:00 PM UTC, comment #4:

Assuming that there are no problems realizing it, the modular approach sounds more elegant.

David Lowe <doctorjlowe>
Mon 15 Aug 2011 07:55:41 PM UTC, comment #3:

Whta's your plan with this patch? I'll make it clear in case you misunderstood my previous comment: I have no strong feelings either way about this patch; if nobody else has anything to say, feel free to commit or not to commit.

Marko Lindqvist <cazfi>
Project Administrator
Sun 24 Jul 2011 09:22:10 PM UTC, comment #2:

Updated patch. Some of the 'extern "C"'s in the previous patch were actually not necessary.

(file #13656)

Christian Prochaska <cproc>
Project Administrator
Sun 24 Jul 2011 09:22:02 PM UTC, comment #1:

Well, changing that probably makes sense, though I have been toying with opposite idea; to start using function pointer layer for all guis. That would allow for example design where we would hjave just one client program to which different guis are loaded as modules.

Marko Lindqvist <cazfi>
Project Administrator
Sun 24 Jul 2011 08:57:15 PM UTC, original submission:

The attached patch defines the GUI functions called from common code as 'extern "C"' to avoid the naming problems. This seems to work as well as using a function pointer layer.

Christian Prochaska <cproc>
Project Administrator

 

(Note: upload size limit is set to 1024 kB, after insertion of the required escape characters.)

Attach File(s):
   
   
Comment:
   

Attached Files
file #13656:  gui_qt_names_2.diff added by cproc (88kB - text/x-patch)
file #13653:  gui-qt_names.diff added by cproc (88kB - text/x-patch)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -unavailable- added by mir3x (Posted a comment)
  • -unavailable- added by doctorjlowe (Posted a comment)
  • -unavailable- added by cazfi (Posted a comment)
  • -unavailable- added by cproc (Submitted the item)
  •  

    Do you think this task is very important?
    If so, you can click here to add your encouragement to it.
    This task has 0 encouragements so far.

    Only logged-in users can vote.

     

    Please enter the title of George Orwell's famous dystopian book (it's a date):

     

     

    Follow 5 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Sat 22 Jun 2013 12:13:35 PM UTCmir3xStatusNone=>Wont Do
      Assigned toNone=>mir3x
      Open/ClosedOpen=>Closed
    Sun 24 Jul 2011 09:22:10 PM UTCcprocAttached File-=>Added gui_qt_names_2.diff, #13656
    Sun 24 Jul 2011 08:57:15 PM UTCcprocAttached File-=>Added gui-qt_names.diff, #13653
    Show feedback again

    Back to the top


    Powered by Savane 3.1-cleanup