Closed Bug 1652253 Opened 4 years ago Closed 4 years ago

Update SVG include guards

Categories

(Core :: SVG, task)

task

Tracking

()

RESOLVED FIXED
mozilla80
Tracking Status
firefox80 --- fixed

People

(Reporter: longsonr, Assigned: longsonr)

Details

Attachments

(1 file)

No description provided.
Assignee: nobody → longsonr
Status: NEW → ASSIGNED

If you take the previous patch here and then run

#!/bin/bash
d=`pwd`
d=`echo $d | rev`
f1=`echo $d | cut -d/ -f1 | rev`
f2=`echo $d | cut -d/ -f2 | rev`

for var in "$@"
do
    define="${f2}_${f1}_${var}"
    define=`echo $define | tr '[:lower:]' '[:upper:]' | tr . _`
    sed -i '.bak' "s/^#ifndef mozilla_.*/#ifndef $define/" $var
    sed -i '.bak' "s/^#define mozilla_.*/#define $define/" $var
    sed -i '.bak' "s/^#endif  \/\/ mozilla_.*/#endif  \/\/ $define/" $var
done

On all the header files in dom/svg dom/smil and layout/svg

you'll get the new patch

Updated script. As usual runs on top of the previous patch version.

#!/bin/bash
d=`pwd`
d=`echo $d | rev`
f1=`echo $d | cut -d/ -f1 | rev`
f2=`echo $d | cut -d/ -f2 | rev`

for var in "$@"
do
    define="${f2}_${f1}_${var}"
    define=`echo $define | tr '[:lower:]' '[:upper:]' | tr . _`
    define2="${define}_"
    sed -i '.bak' "s/^#ifndef $define/#ifndef $define2/" $var
    sed -i '.bak' "s/^#define $define/#define $define2/" $var
    sed -i '.bak' "s/^#endif  \/\/ $define/#endif  \/\/ $define2/" $var
done
Pushed by longsonr@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/1a1da1cacf42
Update SVG include guards r=dholbert
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla80
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: