perlhacktut - Walk through the creation of a simple C code patch

<!DOCTYPE html> <html lang="en"> <head> <meta charset="utf-8"> <meta http-equiv="X-UA-Compatible" content="IE=edge"> <meta name="viewport" content="width=device-width, initial-scale=1"> <title>perlhacktut - Walk through the creation of a simple C code patch - Perldoc Browser</title> <link rel="search" href="/opensearch.xml" type="application/opensearchdescription+xml" title="Perldoc Browser"> <link rel="canonical" href="https://perldoc.perl.org/perlhacktut"> <link rel="stylesheet" href="https://stackpath.bootstrapcdn.com/bootstrap/4.5.2/css/bootstrap.min.css" integrity="sha384-JcKb8q3iqJ61gNV9KGb8thSsNjpSL0n8PARn9HuZOnIxN0hoP+VmmDGMN5t9UJ0Z" crossorigin="anonymous"> <link rel="stylesheet" href="https://cdnjs.cloudflare.com/ajax/libs/highlight.js/10.5.0/styles/stackoverflow-light.min.css" integrity="sha512-cG1IdFxqipi3gqLmksLtuk13C+hBa57a6zpWxMeoY3Q9O6ooFxq50DayCdm0QrDgZjMUn23z/0PMZlgft7Yp5Q==" crossorigin="anonymous" /> <style> body { background: #f4f4f5; color: #020202; } .navbar-dark { background-image: -webkit-linear-gradient(top, #005f85 0, #002e49 100%); background-image: -o-linear-gradient(top, #005f85 0, #002e49 100%); background-image: linear-gradient(to bottom, #005f85 0, #002e49 100%); filter: progid:DXImageTransform.Microsoft.gradient(startColorstr='#ff005f85', endColorstr='#ff002e49', GradientType=0); background-repeat: repeat-x; } .navbar-dark .navbar-nav .nav-link, .navbar-dark .navbar-nav .nav-link:focus { color: #fff } .navbar-dark .navbar-nav .nav-link:hover { color: #ffef68 } #wrapperlicious { margin: 0 auto; font: 0.9em 'Helvetica Neue', Helvetica, sans-serif; font-weight: normal; line-height: 1.5em; margin: 0; padding: 0; } #wrapperlicious h1 { font-size: 1.5em } #wrapperlicious h2 { font-size: 1.3em } #wrapperlicious h3 { font-size: 1.1em } #wrapperlicious h4 { font-size: 0.9em } #wrapperlicious h1, #wrapperlicious h2, #wrapperlicious h3, #wrapperlicious h4, #wrapperlicious dt { color: #020202; margin-top: 1em; margin-bottom: 1em; position: relative; font-weight: bold; } #wrapperlicious a { color: inherit; text-decoration: underline } #wrapperlicious #toc { text-decoration: none } #wrapperlicious a:hover { color: #2a2a2a } #wrapperlicious a img { border: 0 } #wrapperlicious :not(pre) > code { color: inherit; background-color: rgba(0, 0, 0, 0.04); border-radius: 3px; font: 0.9em Consolas, Menlo, Monaco, monospace; padding: 0.3em; } #wrapperlicious dd { margin: 0; margin-left: 2em; } #wrapperlicious dt { color: #2a2a2a; font-weight: bold; margin-left: 0.9em; } #wrapperlicious p { margin-bottom: 1em; margin-top: 1em; } #wrapperlicious li > p { margin-bottom: 0; margin-top: 0; } #wrapperlicious pre { border: 1px solid #c1c1c1; border-radius: 3px; font: 100% Consolas, Menlo, Monaco, monospace; margin-bottom: 1em; margin-top: 1em; } #wrapperlicious pre > code { display: block; background-color: #f6f6f6; font: 0.9em Consolas, Menlo, Monaco, monospace; line-height: 1.5em; text-align: left; white-space: pre; padding: 1em; } #wrapperlicious dl, #wrapperlicious ol, #wrapperlicious ul { margin-bottom: 1em; margin-top: 1em; } #wrapperlicious ul { list-style-type: square; } #wrapperlicious ul ul { margin-bottom: 0px; margin-top: 0px; } #footer { font-size: 0.8em; padding-top: 0.5em; text-align: center; } #more { display: inline; font-size: 0.8em; } #perldocdiv { background-color: #fff; border: 1px solid #c1c1c1; border-bottom-left-radius: 5px; border-bottom-right-radius: 5px; margin-left: auto; margin-right: auto; padding: 3em; padding-top: 1em; max-width: 960px; } #moduleversion { float: right } #wrapperlicious .leading-notice { font-style: italic; padding-left: 1em; margin-top: 1em; margin-bottom: 1em; } #wrapperlicious .permalink { display: none; left: -0.75em; position: absolute; padding-right: 0.25em; text-decoration: none; } #wrapperlicious h1:hover .permalink, #wrapperlicious h2:hover .permalink, #wrapperlicious h3:hover .permalink, #wrapperlicious h4:hover .permalink, #wrapperlicious dt:hover .permalink { display: block; } </style>  <script async src="https://www.googletagmanager.com/gtag/js?id=G-KVNWBNT5FB"></script> <script> window.dataLayer = window.dataLayer || []; function gtag(){dataLayer.push(arguments);} gtag('js', new Date()); gtag('config', 'G-KVNWBNT5FB'); gtag('config', 'UA-50555-3'); </script> </head> <body> <nav class="navbar navbar-expand-md navbar-dark bg-dark justify-content-between"> <button class="navbar-toggler" type="button" data-toggle="collapse" data-target="#navbarNav" aria-controls="navbarNav" aria-expanded="false" aria-label="Toggle navigation"> <span class="navbar-toggler-icon"></span> </button> <a class="navbar-brand" href="/"><img src="/images/perl_camel_30.png" width="30" height="30" class="d-inline-block align-top" alt="Perl Camel Logo"> Perldoc Browser</a> <div class="collapse navbar-collapse" id="navbarNav"> <ul class="navbar-nav mr-auto"> <li class="nav-item dropdown text-nowrap"> <a class="nav-link dropdown-toggle" href="#" id="dropdownlink-stable" role="button" data-toggle="dropdown" aria-haspopup="true" aria-expanded="false">5.20.2</a> <div class="dropdown-menu" aria-labelledby="dropdownlink-stable"> <a class="dropdown-item" href="/perlhacktut">Latest</a> <div class="dropdown-divider"></div> <a class="dropdown-item" href="/5.40.1/perlhacktut">5.40.1</a> <a class="dropdown-item" href="/5.40.0/perlhacktut">5.40.0</a> <div class="dropdown-divider"></div> <a class="dropdown-item" href="/5.38.3/perlhacktut">5.38.3</a> <a class="dropdown-item" href="/5.38.2/perlhacktut">5.38.2</a> <a class="dropdown-item" href="/5.38.1/perlhacktut">5.38.1</a> <a class="dropdown-item" href="/5.38.0/perlhacktut">5.38.0</a> <div class="dropdown-divider"></div> <a class="dropdown-item" href="/5.36.3/perlhacktut">5.36.3</a> <a class="dropdown-item" href="/5.36.2/perlhacktut">5.36.2</a> <a class="dropdown-item" href="/5.36.1/perlhacktut">5.36.1</a> <a class="dropdown-item" href="/5.36.0/perlhacktut">5.36.0</a> <div class="dropdown-divider"></div> <a class="dropdown-item" href="/5.34.3/perlhacktut">5.34.3</a> <a class="dropdown-item" href="/5.34.2/perlhacktut">5.34.2</a> <a class="dropdown-item" href="/5.34.1/perlhacktut">5.34.1</a> <a class="dropdown-item" href="/5.34.0/perlhacktut">5.34.0</a> <div class="dropdown-divider"></div> <a class="dropdown-item" href="/5.32.1/perlhacktut">5.32.1</a> <a class="dropdown-item" href="/5.32.0/perlhacktut">5.32.0</a> <div class="dropdown-divider"></div> <a class="dropdown-item" href="/5.30.3/perlhacktut">5.30.3</a> <a class="dropdown-item" href="/5.30.2/perlhacktut">5.30.2</a> <a class="dropdown-item" href="/5.30.1/perlhacktut">5.30.1</a> <a class="dropdown-item" href="/5.30.0/perlhacktut">5.30.0</a> <div class="dropdown-divider"></div> <a class="dropdown-item" href="/5.28.3/perlhacktut">5.28.3</a> <a class="dropdown-item" href="/5.28.2/perlhacktut">5.28.2</a> <a class="dropdown-item" href="/5.28.1/perlhacktut">5.28.1</a> <a class="dropdown-item" href="/5.28.0/perlhacktut">5.28.0</a> <div class="dropdown-divider"></div> <a class="dropdown-item" href="/5.26.3/perlhacktut">5.26.3</a> <a class="dropdown-item" href="/5.26.2/perlhacktut">5.26.2</a> <a class="dropdown-item" href="/5.26.1/perlhacktut">5.26.1</a> <a class="dropdown-item" href="/5.26.0/perlhacktut">5.26.0</a> <div class="dropdown-divider"></div> <a class="dropdown-item" href="/5.24.4/perlhacktut">5.24.4</a> <a class="dropdown-item" href="/5.24.3/perlhacktut">5.24.3</a> <a class="dropdown-item" href="/5.24.2/perlhacktut">5.24.2</a> <a class="dropdown-item" href="/5.24.1/perlhacktut">5.24.1</a> <a class="dropdown-item" href="/5.24.0/perlhacktut">5.24.0</a> <div class="dropdown-divider"></div> <a class="dropdown-item" href="/5.22.4/perlhacktut">5.22.4</a> <a class="dropdown-item" href="/5.22.3/perlhacktut">5.22.3</a> <a class="dropdown-item" href="/5.22.2/perlhacktut">5.22.2</a> <a class="dropdown-item" href="/5.22.1/perlhacktut">5.22.1</a> <a class="dropdown-item" href="/5.22.0/perlhacktut">5.22.0</a> <div class="dropdown-divider"></div> <a class="dropdown-item" href="/5.20.3/perlhacktut">5.20.3</a> <a class="dropdown-item active" href="/5.20.2/perlhacktut">5.20.2</a> <a class="dropdown-item" href="/5.20.1/perlhacktut">5.20.1</a> <a class="dropdown-item" href="/5.20.0/perlhacktut">5.20.0</a> <div class="dropdown-divider"></div> <a class="dropdown-item" href="/5.18.4/perlhacktut">5.18.4</a> <a class="dropdown-item" href="/5.18.3/perlhacktut">5.18.3</a> <a class="dropdown-item" href="/5.18.2/perlhacktut">5.18.2</a> <a class="dropdown-item" href="/5.18.1/perlhacktut">5.18.1</a> <a class="dropdown-item" href="/5.18.0/perlhacktut">5.18.0</a> <div class="dropdown-divider"></div> <a class="dropdown-item" href="/5.16.3/perlhacktut">5.16.3</a> <a class="dropdown-item" href="/5.16.2/perlhacktut">5.16.2</a> <a class="dropdown-item" href="/5.16.1/perlhacktut">5.16.1</a> <a class="dropdown-item" href="/5.16.0/perlhacktut">5.16.0</a> <div class="dropdown-divider"></div> <a class="dropdown-item" href="/5.14.4/perlhacktut">5.14.4</a> <a class="dropdown-item" href="/5.14.3/perlhacktut">5.14.3</a> <a class="dropdown-item" href="/5.14.2/perlhacktut">5.14.2</a> <a class="dropdown-item" href="/5.14.1/perlhacktut">5.14.1</a> <a class="dropdown-item" href="/5.14.0/perlhacktut">5.14.0</a> <div class="dropdown-divider"></div> <a class="dropdown-item" href="/5.12.5/perlhacktut">5.12.5</a> <a class="dropdown-item" href="/5.12.4/perlhacktut">5.12.4</a> <a class="dropdown-item" href="/5.12.3/perlhacktut">5.12.3</a> <a class="dropdown-item" href="/5.12.2/perlhacktut">5.12.2</a> <a class="dropdown-item" href="/5.12.1/perlhacktut">5.12.1</a> <a class="dropdown-item" href="/5.12.0/perlhacktut">5.12.0</a> <div class="dropdown-divider"></div> <a class="dropdown-item" href="/5.10.1/perlhacktut">5.10.1</a> <a class="dropdown-item" href="/5.10.0/perlhacktut">5.10.0</a> <div class="dropdown-divider"></div> <a class="dropdown-item" href="/5.8.9/perlhacktut">5.8.9</a> <a class="dropdown-item" href="/5.8.8/perlhacktut">5.8.8</a> <a class="dropdown-item" href="/5.8.7/perlhacktut">5.8.7</a> <a class="dropdown-item" href="/5.8.6/perlhacktut">5.8.6</a> <a class="dropdown-item" href="/5.8.5/perlhacktut">5.8.5</a> <a class="dropdown-item" href="/5.8.4/perlhacktut">5.8.4</a> <a class="dropdown-item" href="/5.8.3/perlhacktut">5.8.3</a> <a class="dropdown-item" href="/5.8.2/perlhacktut">5.8.2</a> <a class="dropdown-item" href="/5.8.1/perlhacktut">5.8.1</a> <a class="dropdown-item" href="/5.8.0/perlhacktut">5.8.0</a> <div class="dropdown-divider"></div> <a class="dropdown-item" href="/5.6.2/perlhacktut">5.6.2</a> <a class="dropdown-item" href="/5.6.1/perlhacktut">5.6.1</a> <a class="dropdown-item" href="/5.6.0/perlhacktut">5.6.0</a> <div class="dropdown-divider"></div> <a class="dropdown-item" href="/5.005_04/perlhacktut">5.005_04</a> <a class="dropdown-item" href="/5.005_03/perlhacktut">5.005_03</a> <a class="dropdown-item" href="/5.005_02/perlhacktut">5.005_02</a> <a class="dropdown-item" href="/5.005_01/perlhacktut">5.005_01</a> <a class="dropdown-item" href="/5.005/perlhacktut">5.005</a> </div> </li> <li class="nav-item dropdown text-nowrap"> <a class="nav-link dropdown-toggle" href="#" id="dropdownlink-dev" role="button" data-toggle="dropdown" aria-haspopup="true" aria-expanded="false">Dev</a> <div class="dropdown-menu" aria-labelledby="dropdownlink-dev"> <a class="dropdown-item" href="/blead/perlhacktut">blead</a> <a class="dropdown-item" href="/5.41.10/perlhacktut">5.41.10</a> <a class="dropdown-item" href="/5.41.9/perlhacktut">5.41.9</a> <a class="dropdown-item" href="/5.41.8/perlhacktut">5.41.8</a> <a class="dropdown-item" href="/5.41.7/perlhacktut">5.41.7</a> <a class="dropdown-item" href="/5.41.6/perlhacktut">5.41.6</a> <a class="dropdown-item" href="/5.41.5/perlhacktut">5.41.5</a> <a class="dropdown-item" href="/5.41.4/perlhacktut">5.41.4</a> <a class="dropdown-item" href="/5.41.3/perlhacktut">5.41.3</a> <a class="dropdown-item" href="/5.41.2/perlhacktut">5.41.2</a> <a class="dropdown-item" href="/5.41.1/perlhacktut">5.41.1</a> <div class="dropdown-divider"></div> <a class="dropdown-item" href="/5.40.1-RC1/perlhacktut">5.40.1-RC1</a> <a class="dropdown-item" href="/5.40.0-RC2/perlhacktut">5.40.0-RC2</a> <a class="dropdown-item" href="/5.40.0-RC1/perlhacktut">5.40.0-RC1</a> <div class="dropdown-divider"></div> <a class="dropdown-item" href="/5.39.10/perlhacktut">5.39.10</a> <a class="dropdown-item" href="/5.39.9/perlhacktut">5.39.9</a> <a class="dropdown-item" href="/5.39.8/perlhacktut">5.39.8</a> <a class="dropdown-item" href="/5.39.7/perlhacktut">5.39.7</a> <a class="dropdown-item" href="/5.39.6/perlhacktut">5.39.6</a> <a class="dropdown-item" href="/5.39.5/perlhacktut">5.39.5</a> <a class="dropdown-item" href="/5.39.4/perlhacktut">5.39.4</a> <a class="dropdown-item" href="/5.39.3/perlhacktut">5.39.3</a> <a class="dropdown-item" href="/5.39.2/perlhacktut">5.39.2</a> <a class="dropdown-item" href="/5.39.1/perlhacktut">5.39.1</a> <div class="dropdown-divider"></div> <a class="dropdown-item" href="/5.38.3-RC1/perlhacktut">5.38.3-RC1</a> </div> </li> <li class="nav-item dropdown text-nowrap"> <a class="nav-link dropdown-toggle" href="#" id="dropdownlink-nav" role="button" data-toggle="dropdown" aria-haspopup="true" aria-expanded="false">Documentation</a> <div class="dropdown-menu" aria-labelledby="dropdownlink-nav"> <a class="dropdown-item" href="/5.20.2/perl">Perl</a> <a class="dropdown-item" href="/5.20.2/perlintro">Intro</a> <a class="dropdown-item" href="/5.20.2/perl#Tutorials">Tutorials</a> <a class="dropdown-item" href="/5.20.2/perlfaq">FAQs</a> <a class="dropdown-item" href="/5.20.2/perl#Reference-Manual">Reference</a> <div class="dropdown-divider"></div> <a class="dropdown-item" href="/5.20.2/perlop">Operators</a> <a class="dropdown-item" href="/5.20.2/functions">Functions</a> <a class="dropdown-item" href="/5.20.2/variables">Variables</a> <a class="dropdown-item" href="/5.20.2/modules">Modules</a> <a class="dropdown-item" href="/5.20.2/perlutil">Utilities</a> <div class="dropdown-divider"></div> <a class="dropdown-item" href="/5.20.2/perldelta">Release Notes</a> <a class="dropdown-item" href="/5.20.2/perlcommunity">Community</a> <a class="dropdown-item" href="/5.20.2/perlhist">History</a> </div> </li> </ul> <ul class="navbar-nav"> <script> function set_expand (expand) { var perldocdiv = document.getElementById('perldocdiv'); var width = window.getComputedStyle(perldocdiv).getPropertyValue('max-width'); var expanded = (width == '' || width == 'none') ? true : false; if (expand === null) { expand = !expanded; } if ((expand && !expanded) || (!expand && expanded)) { perldocdiv.style.setProperty('max-width', expand ? 'none' : '960px'); var button_classlist = document.getElementById('content-expand-button').classList; if (expand) { button_classlist.add('btn-light'); button_classlist.remove('btn-outline-light'); } else { button_classlist.add('btn-outline-light'); button_classlist.remove('btn-light'); } } return expand; } function toggle_expand () { var expand = set_expand(null); document.cookie = 'perldoc_expand=' + (expand ? 1 : 0) + '; path=/; expires=Tue, 19 Jan 2038 03:14:07 UTC'; } function read_expand () { return document.cookie.split(';').some(function (item) { return item.indexOf('perldoc_expand=1') >= 0 }); } if (document.readyState === 'loading') { document.addEventListener('DOMContentLoaded', function () { if (read_expand()) { set_expand(true); } }); } else if (read_expand()) { set_expand(true); } </script> <button id="content-expand-button" type="button" class="btn btn-outline-light d-none d-lg-inline-block mr-4" onclick="toggle_expand()">Expand</button> </ul> <form class="form-inline" method="get" action="/5.20.2/search"> <input class="form-control mr-3" type="search" name="q" placeholder="Search" aria-label="Search" value=""> </form> </div> </nav> <div id="wrapperlicious" class="container-fluid"> <div id="perldocdiv"> <div id="links"> <a href="/5.20.2/perlhacktut">perlhacktut</a> <div id="more"> (<a href="/5.20.2/perlhacktut.txt">source</a>, <a href="https://metacpan.org/pod/perlhacktut">CPAN</a>) </div> </div> <div class="leading-notice"> You are viewing the version of this documentation from Perl 5.20.2. <a href="/perlhacktut">View the latest version</a> </div> <h1><a id="toc">CONTENTS</a></h1> <ul> <li> <a class="text-decoration-none" href="#NAME">NAME</a> </li> <li> <a class="text-decoration-none" href="#DESCRIPTION">DESCRIPTION</a> </li> <li> <a class="text-decoration-none" href="#EXAMPLE-OF-A-SIMPLE-PATCH">EXAMPLE OF A SIMPLE PATCH</a> <ul> <li> <a class="text-decoration-none" href="#Writing-the-patch">Writing the patch</a> </li> <li> <a class="text-decoration-none" href="#Testing-the-patch">Testing the patch</a> </li> <li> <a class="text-decoration-none" href="#Documenting-the-patch">Documenting the patch</a> </li> <li> <a class="text-decoration-none" href="#Submit">Submit</a> </li> </ul> </li> <li> <a class="text-decoration-none" href="#AUTHOR">AUTHOR</a> </li> </ul> <h1 id="NAME"><a class="permalink" href="#NAME">#</a>NAME</h1> <p>perlhacktut - Walk through the creation of a simple C code patch</p> <h1 id="DESCRIPTION"><a class="permalink" href="#DESCRIPTION">#</a>DESCRIPTION</h1> <p>This document takes you through a simple patch example.</p> <p>If you haven't read <a href="/5.20.2/perlhack">perlhack</a> yet, go do that first! You might also want to read through <a href="/5.20.2/perlsource">perlsource</a> too.</p> <p>Once you're done here, check out <a href="/5.20.2/perlhacktips">perlhacktips</a> next.</p> <h1 id="EXAMPLE-OF-A-SIMPLE-PATCH"><a class="permalink" href="#EXAMPLE-OF-A-SIMPLE-PATCH">#</a><a id="EXAMPLE"></a>EXAMPLE OF A SIMPLE PATCH</h1> <p>Let's take a simple patch from start to finish.</p> <p>Here's something Larry suggested: if a <code>U</code> is the first active format during a <code>pack</code>, (for example, <code>pack "U3C8", @stuff</code>) then the resulting string should be treated as UTF-8 encoded.</p> <p>If you are working with a git clone of the Perl repository, you will want to create a branch for your changes. This will make creating a proper patch much simpler. See the <a href="/5.20.2/perlgit">perlgit</a> for details on how to do this.</p> <h2 id="Writing-the-patch"><a class="permalink" href="#Writing-the-patch">#</a><a id="Writing"></a>Writing the patch</h2> <p>How do we prepare to fix this up? First we locate the code in question - the <code>pack</code> happens at runtime, so it's going to be in one of the <i>pp</i> files. Sure enough, <code>pp_pack</code> is in <i>pp.c</i>. Since we're going to be altering this file, let's copy it to <i>pp.c~</i>.</p> <p>[Well, it was in <i>pp.c</i> when this tutorial was written. It has now been split off with <code>pp_unpack</code> to its own file, <i>pp_pack.c</i>]</p> <p>Now let's look over <code>pp_pack</code>: we take a pattern into <code>pat</code>, and then loop over the pattern, taking each format character in turn into <code>datum_type</code>. Then for each possible format character, we swallow up the other arguments in the pattern (a field width, an asterisk, and so on) and convert the next chunk input into the specified format, adding it onto the output SV <code>cat</code>.</p> <p>How do we know if the <code>U</code> is the first format in the <code>pat</code>? Well, if we have a pointer to the start of <code>pat</code> then, if we see a <code>U</code> we can test whether we're still at the start of the string. So, here's where <code>pat</code> is set up:</p> <pre><code>STRLEN fromlen; char *pat = SvPVx(*++MARK, fromlen); char *patend = pat + fromlen; I32 len; I32 datumtype; SV *fromstr;</code></pre> <p>We'll have another string pointer in there:</p> <pre><code> STRLEN fromlen; char *pat = SvPVx(*++MARK, fromlen); char *patend = pat + fromlen; + char *patcopy; I32 len; I32 datumtype; SV *fromstr;</code></pre> <p>And just before we start the loop, we'll set <code>patcopy</code> to be the start of <code>pat</code>:</p> <pre><code> items = SP - MARK; MARK++; sv_setpvn(cat, "", 0); + patcopy = pat; while (pat < patend) {</code></pre> <p>Now if we see a <code>U</code> which was at the start of the string, we turn on the <code>UTF8</code> flag for the output SV, <code>cat</code>:</p> <pre><code>+ if (datumtype == 'U' && pat==patcopy+1) + SvUTF8_on(cat); if (datumtype == '#') { while (pat < patend && *pat != '\n') pat++;</code></pre> <p>Remember that it has to be <code>patcopy+1</code> because the first character of the string is the <code>U</code> which has been swallowed into <code>datumtype!</code></p> <p>Oops, we forgot one thing: what if there are spaces at the start of the pattern? <code>pack(" U*", @stuff)</code> will have <code>U</code> as the first active character, even though it's not the first thing in the pattern. In this case, we have to advance <code>patcopy</code> along with <code>pat</code> when we see spaces:</p> <pre><code>if (isSPACE(datumtype)) continue;</code></pre> <p>needs to become</p> <pre><code>if (isSPACE(datumtype)) { patcopy++; continue; }</code></pre> <p>OK. That's the C part done. Now we must do two additional things before this patch is ready to go: we've changed the behaviour of Perl, and so we must document that change. We must also provide some more regression tests to make sure our patch works and doesn't create a bug somewhere else along the line.</p> <h2 id="Testing-the-patch"><a class="permalink" href="#Testing-the-patch">#</a><a id="Testing"></a>Testing the patch</h2> <p>The regression tests for each operator live in <i>t/op/</i>, and so we make a copy of <i>t/op/pack.t</i> to <i>t/op/pack.t~</i>. Now we can add our tests to the end. First, we'll test that the <code>U</code> does indeed create Unicode strings.</p> <p>t/op/pack.t has a sensible ok() function, but if it didn't we could use the one from t/test.pl.</p> <pre><code>require './test.pl'; plan( tests => 159 );</code></pre> <p>so instead of this:</p> <pre><code>print 'not ' unless "1.20.300.4000" eq sprintf "%vd", pack("U*",1,20,300,4000); print "ok $test\n"; $test++;</code></pre> <p>we can write the more sensible (see <a href="/5.20.2/Test::More">Test::More</a> for a full explanation of is() and other testing functions).</p> <pre><code>is( "1.20.300.4000", sprintf "%vd", pack("U*",1,20,300,4000), "U* produces Unicode" );</code></pre> <p>Now we'll test that we got that space-at-the-beginning business right:</p> <pre><code>is( "1.20.300.4000", sprintf "%vd", pack(" U*",1,20,300,4000), " with spaces at the beginning" );</code></pre> <p>And finally we'll test that we don't make Unicode strings if <code>U</code> is <b>not</b> the first active format:</p> <pre><code>isnt( v1.20.300.4000, sprintf "%vd", pack("C0U*",1,20,300,4000), "U* not first isn't Unicode" );</code></pre> <p>Mustn't forget to change the number of tests which appears at the top, or else the automated tester will get confused. This will either look like this:</p> <pre><code>print "1..156\n";</code></pre> <p>or this:</p> <pre><code>plan( tests => 156 );</code></pre> <p>We now compile up Perl, and run it through the test suite. Our new tests pass, hooray!</p> <h2 id="Documenting-the-patch"><a class="permalink" href="#Documenting-the-patch">#</a><a id="Documenting"></a>Documenting the patch</h2> <p>Finally, the documentation. The job is never done until the paperwork is over, so let's describe the change we've just made. The relevant place is <i>pod/perlfunc.pod</i>; again, we make a copy, and then we'll insert this text in the description of <code>pack</code>:</p> <pre><code class="plaintext">=item * If the pattern begins with a C<U>, the resulting string will be treated as UTF-8-encoded Unicode. You can force UTF-8 encoding on in a string with an initial C<U0>, and the bytes that follow will be interpreted as Unicode characters. If you don't want this to happen, you can begin your pattern with C<C0> (or anything else) to force Perl not to UTF-8 encode your string, and then follow this with a C<U*> somewhere in your pattern.</code></pre> <h2 id="Submit"><a class="permalink" href="#Submit">#</a>Submit</h2> <p>See <a href="/5.20.2/perlhack">perlhack</a> for details on how to submit this patch.</p> <h1 id="AUTHOR"><a class="permalink" href="#AUTHOR">#</a>AUTHOR</h1> <p>This document was originally written by Nathan Torkington, and is maintained by the perl5-porters mailing list.</p> </div> <div id="footer"> <p>Perldoc Browser is maintained by Dan Book (<a href="https://metacpan.org/author/DBOOK">DBOOK</a>). Please contact him via the <a href="https://github.com/Grinnz/perldoc-browser/issues">GitHub issue tracker</a> or <a href="mailto:dbook@cpan.org">email</a> regarding any issues with the site itself, search, or rendering of documentation.</p> <p>The Perl documentation is maintained by the Perl 5 Porters in the development of Perl. Please contact them via the <a href="https://github.com/Perl/perl5/issues">Perl issue tracker</a>, the <a href="https://lists.perl.org/list/perl5-porters.html">mailing list</a>, or <a href="https://kiwiirc.com/client/irc.perl.org/p5p">IRC</a> to report any issues with the contents or format of the documentation.</p> </div> </div> <script src="https://cdnjs.cloudflare.com/ajax/libs/jquery/3.5.1/jquery.slim.min.js" integrity="sha512-/DXTXr6nQodMUiq+IUJYCt2PPOUjrHJ9wFrqpJ3XkgPNOZVfMok7cRw6CSxyCQxXn6ozlESsSh1/sMCTF1rL/g==" crossorigin="anonymous"></script> <script src="https://cdnjs.cloudflare.com/ajax/libs/popper.js/1.16.1/umd/popper.min.js" integrity="sha512-ubuT8Z88WxezgSqf3RLuNi5lmjstiJcyezx34yIU2gAHonIi27Na7atqzUZCOoY4CExaoFumzOsFQ2Ch+I/HCw==" crossorigin="anonymous"></script> <script src="https://stackpath.bootstrapcdn.com/bootstrap/4.5.2/js/bootstrap.min.js" integrity="sha384-B4gt1jrGC7Jh4AgTPSdUtOBvfO8shuf57BaghqFfPlYxofvL8/KUEfYiJOMMV+rV" crossorigin="anonymous"></script> <script src="/js/highlight.pack.js"></script> <script>hljs.highlightAll();</script> </body> </html>

CINXE.COM

perlhacktut - Walk through the creation of a simple C code patch - Perldoc Browser