taskGNU TeXmacs - Tasks: task #12916, constref'ing arguments

 
 

You are not allowed to post comments on this tracker with your current authentication level.

task #12916: constref'ing arguments

Submitter:  Miguel de Benito <mdbenito>
Submitted:  Sun 27 Oct 2013 06:17:09 PM UTC
   
 
Category:  None Should Start On:  Sat 26 Oct 2013 10:00:00 PM UTC
Should be Finished on:  Sat 26 Oct 2013 10:00:00 PM UTC Priority:  3 - Low
Status:  In Progress Privacy:  Public
Assigned to:  mdbenito Percent Complete:  0%
Open/Closed:  Open Effort:  0.00

Thu 02 Jan 2014 05:35:00 PM UTC, comment #1: 

I've started a git branch with this stuff. Please contact me to coordinate (one can always hope... hehe)

Miguel de Benito <mdbenito>
Group Member
Sun 27 Oct 2013 06:17:09 PM UTC, original submission:  

A non-negligeable performace improvement is to be had by relying less on smart pointers and constref'ing arguments everywhere. This task is here so that we may comment on the progress and send patches with changes as atomic as possible, i.e. such that they may be applied into the trunk after peer-review.

  See for instance the first attached patch, where I constref'ed a lot of usages of string in analyze.?pp, string.?pp and elsewhere.

Suggestions:

  1. We work with types, instead of files: first string, then tree, etc.
  2. We don't leave source files "unfinished", i.e. with some functions converted others not. Because we don't want to work on all types at the same time, "unfinished" is to be understood wrt. to a given type.
  3. We don't focus on this task, just maybe complete a few files from time to time, and check / update the task in savannah, etc.
  4. I don't know where changes would have greatest impact on performance, but I guess targeting basic data manipulation routines for strings and trees (next in the list) makes sense. Any ideas?
  5. We may start a new branch, but conflict-hell is just around the corner...


Miguel de Benito <mdbenito>
Group Member

 

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

Attached Files
file #29490:  constref_string_uses01-rev7569.patch added by mdbenito (53KiB - application/octet-stream)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by mdbenito (Submitted the item)
  •  

    There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

     

    Follow 2 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2013-10-27 mdbenito Attached File- Added constref_string_uses01-rev7569.patch, #29490
    2013-10-27 mdbenito Summaryconsref\'ing constref'ing arguments

    Back to the top

    Powered by Savane 3.14-f13d.
    Corresponding source code