Weird line wrap. :D I'd indent it one tab more than the wrapped line.


@omidkrad Thank you for the feedback, I appreciate you taking the time. I actually removed this section last week, it just hasn’t been published yet. I’ve tested the tutorial thoroughly, and it does appear to work as written. Again, thanks for reaching out.


It sends a 302 to /2fast.png.

I intended to make 2fast.png return a 429, but forgot. You can detect the redirect.


Yikes. Good catch, @derrickstaten


Need declarations at beginning of scope block for silly C89.


明天还有个电面，周四周五两个聊天。三周后有个onsite。目前暂时这么多。

On Tue, May 31, 2016 at 4:07 PM Linyao Li <notifications@github.com> wrote:

> 面试如何了
>
> —
> You are receiving this because you authored the thread.
> Reply to this email directly, view it on GitHub
> <https://github.com/xiaoyaoworm/Leetcode-JAVA/commit/66aba463156187f8da3af04ce79c1b69f8ef84d1#commitcomment-17688084>,
> or mute the thread
> <https://github.com/notifications/unsubscribe/ACrkeI7X5g7XfIcI5DVZRQ78jBlGj_3rks5qHL9EgaJpZM4Iq_VZ>
> .
>


Your `<head>` generally shouldn't include any markup that is part of the visible document structure.
i.e. your `<nav>` content shouldn't be here
http://www.w3schools.com/tags/tag_head.asp


Is there a reason you included `bootstrap.css` in your repo and are also pulling what seems to be the same resource here?


Same thing above, in rotateMix section. Try: `-std=c89 -pedantic`


NOTE: This branch was commited as reference but I will not continue.

Medium.js has many errors, and does not seem to be solving. :disappointed:


inline styles `style="margin-top:30px"` are usually not a good idea because
> Inline styles added to an element (e.g., style="font-weight:bold") always overwrite any styles in external stylesheets and thus can be thought of as having the highest specificity.
https://developer.mozilla.org/en-US/docs/Web/CSS/Specificity


`list-commands -s` works as desired :-) `list-sessions -s`, could return similar output. How another short needed lists could look to simply use in completion? Probably you know it better than me, and you can find some suggestions in completion scripts.


please use {% trans "Area" %} and so on to get text translated


That's what Core Location requested from us. I guess we should talk to them again.


Can you say something about what the join is accomplishing here?


120281615-ZoneCrossMixing-does-not-allow-overlapping-zones-#5007 (lgu1234) - i386-Windows-7-VisualStudio-12: OK (2211 of 2211 tests passed, 0 test warnings)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


Is this cruft? Looks like the class isn't used anywhere and is missing a `.` or `#`


I guess you did not try to start a cluster on Ubuntu trusty to test the change in this file. It does not work. The shell functions running in upstarts do not support all shell functionalities. The key word "local" is one example, and "<<<" is another. I make a PR #26599 to fix it.


++ hope this works!


Is the above line for removing salt comments? If so, it is not needed as function remove_salt_config_comments is called at line 622.


Was this supposed to be in the `<head>`? Or even better yet, put it before your closing `</body>` tag
http://stackoverflow.com/questions/10994335/javascript-head-body-or-jquery#answer-10994391


yes. created a new repo and again forgot to put app/timeservice/


Having a problem with the additional size changed to nav bar from right to left


Is this used anywhere? Also I'd recommend picking hyphens or underscores and sticking when them :)


y u no like `<h1>`?


fix-bulleted-list-rendering-issue (michael-okeefe) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (2244 of 2244 tests passed, 0 test warnings)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


I know. I added the lt02 blobs to my vendor repo.


300/256 can be reduced to 75/64...maybe that does not mater...maybe the compiler does that anyway...I don't know.


**IMPORTANTE**  La estructura es la siguiente:

1. codigo
2. nombre
3. ciudad
4. estado (el numero correspondiente)
5. direccion
6. telefono


I think I did a few things here


Keep in mind that many people do not end up seeing/reading Github comments. It is better to comment on the Gerrit changelist.

You are correct that this is fundamentally an issue with `godoc` and I was planning on filing an issue with regard to that. Thanks for the reference. I still believe the change in `flate` is beneficial because it helps categorize these fundamentally different constants.


You see some things here have been done.


To initialize the sqlite3 database for all of INSTALLED_APPS in `settings.py`:
`./manage.py migrate`

You can create an admin user to access 0.0.0.0/admin for managing your models and users:
`./manage.py createsuperuser`

Every time you edit `models.py` you must migrate the sqlite3 database.
`./manage.py makemigrations django`
`./manage.py migrate`


Things that are here are the things that I did.


I did things.


fix-bulleted-list-rendering-issue (michael-okeefe) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed, 0 test warnings)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg)


nice! much better!


One generally thing I'm noticing is that your indentation is a little inconsistent which can make things kind of hard to read for reviewers and your future-self. I'd try to make sure you're indenting all html elements appropriately.

e.g. at a glance it's pretty tricky to figure out what's going on here
```
</div>
</div>
</div>

<div class= "container">
<div class="well well-lg" style="margin-top:30px">
  <div class="form-group">
    <form>
    <input type="Name" class="form-control" id="inputName" placeholder="Name">
  </div>
  <div class="form-group">
    <input type="E-mail" class="form-control" id="inputEmail" placeholder="E-mail">
  </div>
  <div class="form-group">
    <input type="Message" class="form-control" id="inputMessage" placeholder="Message">
  </div>
  <button type="submit" class="btn btn-default center-block" value="Send Email">Submit</button>

</form>
</div>
</div>
```

A quick google for html, css, js beautifier can clean up these existing issues!


but 3 other gov are there in the gpu governor setting (simple, powersave and conservative)


That's a bit unexpected, because pin choices are nothing new :-)

Anyways, with some playing I managed to reproduce this. It looks like it happens when the config file doesn't read one of these:

    #define DISPLAY_RW_PIN           PC0
    #define DISPLAY_RW_PIN           xxxx

but this:

    #define DISPLAY_RW_PIN

I.e. the #define is there, but without value. Removing the #define entirely fixes it as well as adding an arbitrary value.

Can you confirm this?


Not sure why you removed this line, but it's essential for allowing the click event to advance reading prompts when not using pinyin input.


Works great !!! thanks


where does `Request#before` come from?


same question: where does `Request.after` come from?


See my comment in #2591


>  As Gilda Radner would say, "Never mind."

That's what I like about this place, it much more of an education than just learning about the weather and Python.


my above questions are assuming that this module will be included into `Reel::Request` when `reel/session` is required. let's call the module `Reel::Session::RequestMixin` following the pattern in `mixins.rb`. also, i think i prefer singular for the module namespace, but i won't fret too much about that :)


Oh, wait, that means the selector won't match either. My bad.


Editing Project to run


> Keep in mind that many people do not end up seeing/reading Github comments. It is better to comment on the Gerrit changelist.

You're right, I should have posted this on Gerrit instead. Glad you saw the message nonetheless.

> You are correct that this is fundamentally an issue with godoc and I was planning on filing an issue with regard to that. Thanks for the reference. I still believe the change in flate is beneficial because it helps categorize these fundamentally different constants.

I am happy to hear that you had intended to address the underlying issue, rather than fixing symptoms. I also understand that the specific change to flate may have ended up with cleaner code, in which case that is great! Just don't want future changes to have to address the same issue by a workaround, especially when these future changes to other packages may make the code less rather than more readable.


BTW, I knew you would react to that commit. And I am glad you did ;)


The following doc(s) have been deployed to the [1626prunin](https://1626prunin-static-docs.pantheonsite.io/docs) Multidev environment:
- [docs/pruning-git-repo](https://1626prunin-static-docs.pantheonsite.io/docs/pruning-git-repo)


better yet to replace lines 340-341 with
$(target).val(value);
less code, faster and will trigger correct events and listeners


The following doc(s) have been deployed to the [1626prunin](https://1626prunin-static-docs.pantheonsite.io/docs) Multidev environment:
- [docs/pruning-git-repo](https://1626prunin-static-docs.pantheonsite.io/docs/pruning-git-repo)
- [docs/reducing-large-repos](https://1626prunin-static-docs.pantheonsite.io/docs/reducing-large-repos)


This was a typo, fixed.  Thanks for the report!


120446171-Mod-to-Warning-Less-than-6-Zone-Surface (RKStrand) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1653 of 1653 tests passed, 0 test warnings)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg) ![Coverage Badge](http://img.shields.io/badge/coverage%20status-70.94%25-green.svg)


Read the source.

kgsl_pwrscale_trustzone.c:

static ssize_t tz_governor_show(struct kgsl_device *device,
                                struct kgsl_pwrscale *pwrscale,
                                char *buf)
{
        struct tz_priv *priv = pwrscale->priv;
        int ret;

        if (priv->governor == TZ_GOVERNOR_ONDEMAND)
                ret = snprintf(buf, 10, "ondemand\n");
        else
                ret = snprintf(buf, 13, "performance\n");

        return ret;
}

On 1 June 2016 at 02:24, FaArIsH <notifications@github.com> wrote:

> but 3 other gov are there in the gpu governor setting (simple, powersave
> and conservative)
>
> —
> You are receiving this because you authored the thread.
> Reply to this email directly, view it on GitHub
> <https://github.com/CyanogenMod/android_device_samsung_serrano-common/commit/096a2af58262d7bacb2c418bd870b47f3796780d#commitcomment-17688831>,
> or mute the thread
> <https://github.com/notifications/unsubscribe/AAftjK6LsHHYDNaqOfqtCYcMS0FQbRZkks5qHNE8gaJpZM4IqgX6>
> .
>


That probably wasn't needed : http://stackoverflow.com/a/2391555/846797


OVS-ofctlと互換性を示した文にすればよかったんですね。
上記の内容でコメントタイトルを修正します。
難しくなりにくいよう、今後なるべくコメントタイトルにthatを使用しないようにします。


我这是老毛病了 不好改


Version 1.2 I mean.


It's a good idea, but we also need to make sure we're clear about WHY we're namespacing certain classes; are they part of an separate library, or distinct grouping of services/functions? We should ensure that we have a very specific reason for doing so.

If the only reason to group a set of classes is because they're the same sort of thing, we probably don't need it yet. Finally, if we are following that pattern, I'd suggest that it's more expressive if we pluralise the namespace: `Presenters::UserPresenter` vs `Presenter::UserPresenter`


In case we ever come back to the C implementation…

This loop causes a segfault using cc provided by gcc 4.8.2. It runs as expected on gcc 5 and with clang. The segfault arises because compiled `list_each_elem` makes a *mistake* and tries to dereference an invalid memory location. This isn't particularly reassuring when using this library, even though more modern compilers work fine.


120446171-Mod-to-Warning-Less-than-6-Zone-Surface (RKStrand) - x86_64-Linux-Ubuntu-14.04-custom_check: OK (0 of 0 tests passed, 0 test warnings)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg)


why this change?


TeamCity ReOpenLDAP :: next [Build 135](https://binbug.bigbrother-matrix.ru/teamcity/viewLog.html?buildId=2295&buildTypeId=ReOpenLDAP_Next&guest=1) is now running


Sorry. I missed your comment. Of course I don't. I am happy if this helps!


fix-bulleted-list-rendering-issue (michael-okeefe) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (1094 of 1094 tests passed, 0 test warnings)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg) ![Coverage Badge](http://img.shields.io/badge/coverage%20status-33.91%25-green.svg)


fix-bulleted-list-rendering-issue (michael-okeefe) - x86_64-Linux-Ubuntu-14.04-custom_check: OK (0 of 0 tests passed, 0 test warnings)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg)


Hi,
Sorry to bother you again... We were able to replace OpenSSL with
BoringSSL, but we would like to verify Google's Play Store will not reject
our new updated App, based on the SSL issues we had before.
How can we verify that we comply with Google's requirements regarding SSL
vulnerability? How is it tested?
We wanted to verify it BEFORE we publish our App in Google Play.
Thanks again,
Hagit.

On Thu, May 19, 2016 at 1:34 PM, Kenny Root <notifications@github.com>
wrote:

> It is fine if you include it in your application (i.e., statically link it
> with your JNI shared library).
>
> On Thu, May 19, 2016 at 1:08 PM, hgbread <notifications@github.com> wrote:
>
> > OK. We will try it. Thanks!!!
> > BTW, it says in its README.md file that it is not intended for third
> party
> > use, "because there are no guarantees of API or ABI stability". But I
> guess
> > it is our best bet for now.
> >
> > —
> > You are receiving this because you authored the thread.
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/android/platform_external_openssl/commit/21b86b154c3cbb88bca801b07e93db4fc8face8e#commitcomment-17546692
> >
> >
>
> —
> You are receiving this because you commented.
> Reply to this email directly or view it on GitHub
> <https://github.com/android/platform_external_openssl/commit/21b86b154c3cbb88bca801b07e93db4fc8face8e#commitcomment-17546988>
>


Cleaning repository


確か、`NXActionBundle`の`ofs_nbits`と`dst`は0で指定すべきものであったと認識しておりますので、0埋めしています。


@pletcher @mendelB  I reworked this a lot, kind of working from these slides http://web.stanford.edu/class/archive/cs/cs106a/cs106a.1154/lectures/03/Slides03.pdf (starting around slide 20) for a solid example explanation.  Hard to tell if the tone is right for this point of the curriculum & hard to talk about pass-by-value when methods have not been covered yet.  I'm open to any feedback!


sorry no I just realized I didn't do the scroll mask thing since it wasn't preset. I actually added random platforms that it jumps on. lol sorry bad English- I didn't do good. It's simple to add the masked one right? so yeah I can use it if you want to - same concept.


Next we have to find backgrounds for our levels


Looks good!


Is this commit pending for backport to zfs-0.6.5-release? It is affected by the associated issue this fixes.


Hi Tele42, could you email me through the contact form for CS:GO here? http://www.valvesoftware.com/email.php

Thanks! (And sorry, github no longer has private messaging).


There was missing indent, because the number "10" is longer by one character and the nested code must be indented more than the superior block. All indents are relative in ReST. It was mysterious until I googled an answer.


positive greeting, negative greeting?


yes


cool


Printing to standard output not good enough for ya?


Niiice tests 👍

- Perfect identation and code style
- Good descriptions for the tests


What was wrong with this patch ?


tst


indentation. (not trying to be snarky...just looking through the code since the company i work for wants to implement a similar compaction strategy). Cheers on the good work.


@nyalldawson are these zoom in and out shortcuts customizable? It'd be nice if it was so.


When the rtp_cipher is of type AES_GCM, for example SRTP_AES_128_GCM, the key len for rtp_cipher is 28 (16+12) bytes. The rtp_xtn_hdr_cipher is of type SRTP_AES_128_ICM according to [RFC 7714](https://tools.ietf.org/html/rfc7714#section-8.3), whose key length should be 30.

1. The user of libsrtp would provide a 28-byte key referred by **policy->key**. The **memcpy()** in [line 3067](https://github.com/cisco/libsrtp/commit/1d64fb5b0062be876ebdd6a3360fa02dd3430686#diff-2ba351605443e572f29fc091b8f17939R3067) will cause reading overflow by reading 2 more bytes.
1. Is it also copying 2 more bytes when initializing the extension header cipher key at [line #801](https://github.com/cisco/libsrtp/blob/1d64fb5b0062be876ebdd6a3360fa02dd3430686/srtp/srtp.c#L801)?
1. The [KDF for rtp_xtn_hdr_cipher](https://github.com/cisco/libsrtp/blob/1d64fb5b0062be876ebdd6a3360fa02dd3430686/srtp/srtp.c#L804) is expecting an input  key with master key 16 bytes and salt 14 bytes, however the user could only provide the key with a 12-byte salt.
```
      /* initialize KDF state     */
#if defined(OPENSSL) && defined(OPENSSL_KDF)
      stat = srtp_kdf_init(xtn_hdr_kdf, (const uint8_t *)tmp_xtn_hdr_key, rtp_xtn_hdr_base_key_len, rtp_xtn_hdr_salt_len);
#else
      stat = srtp_kdf_init(xtn_hdr_kdf, SRTP_AES_ICM, (const uint8_t *)tmp_xtn_hdr_key, kdf_keylen);
#endif
```


this is my first progrom,simpale and no tecknology.


Bruh lol


* `json:"-"` added because `json.Unmarshal` would cause the dead circulation.
* If you are not reading config file, the structure have no restrictions. Otherwise, a pointer to self is required.
* `cli.Helper` is a builtin type for convenience.A common rule is that flag repetition is not allowed.So if root is global, subcommand's flags and root's flags can not have the intersection.However, I plan to add `private` property for flag, e.g. `cli:"h,help:private"`.
* Type of `ctx.Argv()` is `interface`, must be convertd to a specific type, and `RootArgv()` as well.
* `RootArgv()` returns the Argument structure of root command and `Argv()` returns current. I suggest you handle `RootArgv` in `OnRootBefore` of root command(I' will write a example).


This may never be completed.


Thanks,

Please take a look at https://github.com/suntong/lang/tree/master/lang/Go/src/sys/CLI/027-global-redo

my question for it are,

- how to put multiple sub command in the command tree?
- how to make my sub command show global options?

Thanks


Changed css class from independent file to www/css/ionic.app.css, deleted independent file ww/css/styles.css


try to connect to git-hub


@Trucido thanks for noticing this, it looks like a legitimate candidate I'll add it to the list of possible patches to apply.


I tried with this commit and `formattable(mtcars)` produces an error:

```
 Error: invalid version specification ‘NULL’
13.
stop(gettextf("invalid version specification %s", paste(sQuote(unique(x[!ok])),
    collapse = ", ")), call. = FALSE, domain = NA)
12.
.make_numeric_version(x, strict, .standard_regexps()$valid_numeric_version)
11.
numeric_version(sapply(deps, `[[`, "version"))
10.
resolveDependencies(findDependencies(singletonInfo$ui))
9.
renderTags(html)
8.
save_html(html, file = index_html, background = background, libdir = "lib")
7.
html_print(htmltools::as.tags(x, standalone = TRUE), viewer = if (view) viewerFunc)
6.
print.htmlwidget(as.htmlwidget(x), ...)
5.
print(as.htmlwidget(x), ...) at formattable.R#194
4.
print_formattable.data.frame(x, ...) at formattable.R#178
3.
print_formattable(x, ...) at formattable.R#199
2.
print.formattable(x)
1.
function (x, ...)
UseMethod("print")(x)
```

Then I debug it, something wrong with `base::.make_numeric_version` where

```
> formattable(mtcars, list(mpg = x ~ x))
Error: invalid version specification ‘NULL’
Called from: .make_numeric_version(x, strict, .standard_regexps()$valid_numeric_version)
Browse[1]> x
[1] "1.11.3"     "NULL"       "0.6"        "0.1.9.9002"
```


teste


I generally don't like that this line does the comparison and return in the same line. I know it's faster, but I find it a bit more complicated to parse. Maybe keep the if statement but change to using `in` rather than the `or`s?


I'm not entirely clear what the changes in this file are trying to do (maybe a commit message in a Pull Request would make this more clear?


can we submit a PR for this upstream to buck?


already submitted :)


I notice that rmarkdown exports `html_dependency_jquery` and shiny exports `bootstrapLib`. Is there a reason to use `:::` here instead of `::`?


@jarun I think I saw an email mentioning this problem earlier this morning when I was half asleep, but now I can't find it. Anyway, fixed it.


Could you add some images of your changes /?


While this is a tricky way to make this work, it feels a bit too magical to me. I'd rather make it explicit, rather than overriding __boo__.

To quote the Zen of Python:
```Explicit is better than implicit.
Simple is better than complex.```


The description in the Obsolete attribute is incorrect, it should say `IToolLocator` instead of `IToolService`.


some more nonsense


_throws party_ :tada: :balloon:


I think this will break this right? Since if `items = [item]` this change will return a list, when really we want to return `item` right?


what I have read so for in Ruby is DRY which is one of key factors in Ruby and Rails. Having

module Record
      class UserRecord
         # class details
       end
 end

isn't it violating DRY because we are repeating Record

secondly one of the purpose Namespaces service in .Net is that you can have same class name in a different Namespace but because of its Namespace hierarchy they are treated as distinct identities.


Does this work, or do you also need a `getContext('2d')` call on the canvas before issuing commands?


Thanks for submitting this code. Generally, I'd try to simplify what you're trying to do into separate commits (if you have multiple purposes). There is some code that is cleanup and I'm not clear exactly what you're trying to fix with the other code. (This might be my lack of familiarity with this project at the moment -- I'm working on that :) )

Also, before merging this (assuming we get it into a good state first), I'd also need to see tests that demonstrate that this code works.

Let me know if I can help with any of that.


Lol, I didn't pay attention to what I was pasting xD.


120281615-ZoneCrossMixing-does-not-allow-overlapping-zones-#5007 (lgu1234) - Win64-Windows-7-VisualStudio-12: OK (2210 of 2211 tests passed, 0 test warnings)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-99.95%25-yellow.svg)


In case this freaks anyone out--the buttons were deleted last commit.


Where did this come from?


working copy


Did you copy and paste my code again? https://github.com/msinkula/web170-demo/blob/master/index.php


Hi

Ungit start mozilla in my last install (On linux Ubuntu)...

I want to use ungit automatically in my chromium browser (as it's the case in my windows configuration)

In my settings of default browser of my linux install, chromium is defined as default browser.

Why ungit start mozilla ?

Thanks.


Related to: backdrop/backdrop-issues#1250


Try running ```raven_script.py --help```


@liemlian 想要实现类似的效果，有什么建议吗


#1746 #1626 #1745 #1747 #1740 #


once https://github.com/ros-drivers/openni2_camera/pull/45 is accepted and released, this would be no longer required


This was the initial implementation of the mongodb into app.js and creating the data and returning it on GET of /testUsers in routes/index.js


++ for now, @SergeyZhukovsky @garvankeeley pls sync.


Save it


@tcking 有进展了，我把每个I/O操作记录起来，再并发这些I/O，看到了明显的改进。

```java
private static void loadBundles(List<Bundle> bundles) {
    sPreloadBundles = bundles;

    // Prepare bundle
    long t0 = System.nanoTime();
    for (Bundle bundle : bundles) {
        bundle.prepareForLaunch();
    }
    long t1 = System.nanoTime();
    System.out.println("## loadBundles: " + (t1 - t0) / 1000000.0f);

    if (sIOActions != null) {
        ExecutorService executor = Executors.newFixedThreadPool(sIOActions.size());
        for (Runnable action : sIOActions) {
            executor.execute(action);
        }
        executor.shutdown();
        try {
            executor.awaitTermination(3, TimeUnit.MINUTES);
        } catch (InterruptedException e) {
            e.printStackTrace();
        }
    }

    System.out.println("## handle I/O: " + (System.nanoTime() - t1) / 1000000.0f);

    // Notify `postSetUp' to all launchers
    for (BundleLauncher launcher : sBundleLaunchers) {
        launcher.postSetUp();
    }
}
```


argument might be a better name?


a


you can take the louie out of the c++ but you can't take the c++ out of the louie


he needs 5 ccs of kotlin and python, stat


fix-bulleted-list-rendering-issue (michael-okeefe) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1653 of 1653 tests passed, 0 test warnings)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg) ![Coverage Badge](http://img.shields.io/badge/coverage%20status-70.94%25-green.svg)


flags_definition:
        cfmask_conf:
          bits: [0,1,2,3,4,5,6,7]
          description: CFmask confidence
          low: '<=12.5%'
          medium: '>12.5%&<= 22.5%'
          high: '>22.5%'
          values:
            0: none
            1: low
            2: medium
            3: high


Change disks to physicalDisks


Noting #880 as a memo


Could you help to give some examples about how to write markdown so that we could make use of code highlight feature?


hello,there?


Weren't disabled buttons always greyed? In the 'Before' pic they are greyed, if they were enabled they would have colors.


That kind if crap happens to me too much!


/cc @jcredding


It is not really wrong to say `else nil` here... but just so you know, it is also not necessary:

```
if can_train_footman?
  # do it...
end
```

If the condition is not true, the whole `if` block will have a value of `nil`.


This constant should be private


HackyQt5.6 (kbenne) - x86_64-Linux-Ubuntu-14.04-clang-3.5: Build Failed

![Build Badge](http://img.shields.io/badge/build%20status-failing-red.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-NA-red.svg)


This locator is not good


Add a constant for number 15
```java
private static final int TIMEOUT_NORMAL = 15;
```


Import statements are divided into the following groups, in this order, with each group separated by a single blank line:
1. java imports
2. javax imports
3. Third-party imports, one group per top-level package, in ASCII sort order
for example: android, com, junit, org, sun
4. local packages imports.
5. All static imports in a single group

Example:
```java
 import java.util.List;
 import java.util.ArrayList;
 import java.util.Date;

 import javax.swing.JFrame;
 import javax.swing.JPanel;
 import javax.swing.event.TableModelListener;

 import org.apache.commons.httpclient.cookie.CookiePolicy;
 import org.apache.commons.httpclient.cookie.CookieSpec;

 import your.own.packagename.here.SomeClass;
 import your.own.packagename.here.OtherClass;

 import static org.junit.Assert.assertEquals;
```


thanks


Be careful here! This makes the test pass, but on an assessment test you would lose some marks here... the test says `it "should do deal 10 (AP) damage to the enemy unit"` meaning it should deal the unit's `attack_power` in damage.

So it should be: `enemy.damage(attack_power)`


naprawdę chciałbym więcej zrobić ale niejaki Kamyk pseud. alkoholik, wbił mi do pokoju i nie chciał go opuścić przez dwie godziny ...


@feralonso creo que primero abria que poner a vlc como reproductor predeterminado opciones->cambiar reproductor->personalizado y luego ponerle PLAYER:// al principio del enlace asi PLAYER://http://pikermollerpvf.dynns.com:8000/live/j.hc4/YQoNFo061f/65.ts


到此一游！


This locator is not good.
Change it.


You have been marked off for 09) Creating a Custom Posts Feed with Featured Images for Your Site.


Nice fix!

https://github.com/mackerelio/mackerel-agent/pull/240
This pull request based on this commit.

Thank you!


HackyQt5.6 (kbenne) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed, 0 test warnings)

![Build Badge](http://img.shields.io/badge/build%20status-warnings-yellow.svg)


This class is very similar to NewTask class
Apply an Abstract class here and apply inheritance.


Once you have a `Unit` class written, you can write this as:

```
def initialize
  super(60, 10)
end
```


*my first project
*test


And github can even make diff between stl files, that is awesome!


delete component .idea/compiler.xml


hhhhhhhhhh


An archive can contain all of those file types as far as I'm aware.


As I mentioned in the previous commit, I get an error if I set G_TYPE_FILE as the return type. G_TYPE_OBJECT works though.


TeamCity OTA+ :: rvi_sota_server :: ci test [Build 253](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=3814&buildTypeId=ota_plus_RviSotaServer_CiTest) is now running


No, I like fn, I'm using it. We have most of stuff rewritten, and currently its kinda feels like a mix of pre-GM13 code, post-GM13 code and our code. You read some stuff and start thinking - damn, this is hella ancient. If we rewrite everything then we'll finish the cycle and we're not dependent on DarkRP code, name could stay though.

I've used to make pull requests to GMod and DarkRP, but over time I've lost hope in open source. Why publish your work to make other servers better too? That's not something I want to do.


¿podria probar de cambiar las lineas 198 y 199 originales por:
`198 value = string_type(value) # cast data to unicode`
`199 return unicodedata.normalize('NFKD', value).encode('ascii', 'ignore')`?


TeamCity OTA+ :: rvi_sota_server :: ci test [Build 252](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=3811&buildTypeId=ota_plus_RviSotaServer_CiTest) outcome was **SUCCESS**
Summary: Tests passed: 199, ignored: 3 Build time: 00:12:00


Public address is `info`.


why is this guy still here?


note that trhis change, coupled with 651155775e74473f0adb3847b282bd4a8b636640, effectively disabled the entire metric system.


and that guy?


Good catch!


@qdp-odoo

Got traceback when trying to open form view of paid invoice. Invoice was created, opened and paid before this commit. Now it(and couple others) can't be opened in form view.

> Odoo Server Error
> Traceback (most recent call last):
>   File "/usr/lib/python2.7/dist-packages/openerp/http.py", line 643, in _handle_exception
>     return super(JsonRequest, self)._handle_exception(exception)
>   File "/usr/lib/python2.7/dist-packages/openerp/http.py", line 680, in dispatch
>     result = self._call_function(**self.params)
>   File "/usr/lib/python2.7/dist-packages/openerp/http.py", line 316, in _call_function
>     return checked_call(self.db, *args, **kwargs)
>   File "/usr/lib/python2.7/dist-packages/openerp/service/model.py", line 118, in wrapper
>     return f(dbname, *args, **kwargs)
>   File "/usr/lib/python2.7/dist-packages/openerp/http.py", line 309, in checked_call
>     result = self.endpoint(*a, **kw)
>   File "/usr/lib/python2.7/dist-packages/openerp/http.py", line 959, in __call__
>     return self.method(*args, **kw)
>   File "/usr/lib/python2.7/dist-packages/openerp/http.py", line 509, in response_wrap
>     response = f(*args, **kw)
>   File "/usr/lib/python2.7/dist-packages/openerp/addons/web/controllers/main.py", line 896, in call_kw
>     return self._call_kw(model, method, args, kwargs)
>   File "/usr/lib/python2.7/dist-packages/openerp/addons/web/controllers/main.py", line 888, in _call_kw
>     return getattr(request.registry.get(model), method)(request.cr, request.uid, *args, **kwargs)
>   File "/usr/lib/python2.7/dist-packages/openerp/api.py", line 250, in wrapper
>     return old_api(self, *args, **kwargs)
>   File "/usr/lib/python2.7/dist-packages/openerp/models.py", line 3193, in read
>     result = BaseModel.read(records, fields, load=load)
>   File "/usr/lib/python2.7/dist-packages/openerp/api.py", line 248, in wrapper
>     return new_api(self, *args, **kwargs)
>   File "/usr/lib/python2.7/dist-packages/openerp/models.py", line 3239, in read
>     values[name] = field.convert_to_read(record[name], use_name_get)
>   File "/usr/lib/python2.7/dist-packages/openerp/models.py", line 5740, in __getitem__
>     return self._fields[key].__get__(self, type(self))
>   File "/usr/lib/python2.7/dist-packages/openerp/fields.py", line 829, in __get__
>     self.determine_value(record)
>   File "/usr/lib/python2.7/dist-packages/openerp/fields.py", line 936, in determine_value
>     self.compute_value(recs)
>   File "/usr/lib/python2.7/dist-packages/openerp/fields.py", line 891, in compute_value
>     self._compute_value(records)
>   File "/usr/lib/python2.7/dist-packages/openerp/fields.py", line 881, in _compute_value
>     getattr(records, self.compute)()
>   File "/usr/lib/python2.7/dist-packages/openerp/api.py", line 248, in wrapper
>     return new_api(self, *args, **kwargs)
>   File "/usr/lib/python2.7/dist-packages/openerp/api.py", line 425, in new_api
>     result = [method(rec, *args, **kwargs) for rec in self]
>   File "/usr/lib/python2.7/dist-packages/openerp/addons/account/models/account_invoice.py", line 158, in _get_payment_info_JSON
>     payment_currency_id = all([p.currency_id == payment.matched_credit_ids[0].currency_id for p in payment.matched_credit_ids]) and payment.matched_credit_ids[0].currency_id or False
>   File "/usr/lib/python2.7/dist-packages/openerp/models.py", line 5744, in __getitem__
>     return self._browse(self.env, (self._ids[key],))
> IndexError: tuple index out of range


Why set?


If/when the debugger is more widely used, we should figure out a way to keep windows compatibility without having to define more mock methods... Maybe just check if `isdebugging() == true` whenever its's used?


这是啥？


What have happened here?


ça se passe comment pour `10` ?


do not use strings directly in expressions. instead create "constants" for them (e.g var DISABLED = 'disabled')
use short expressions like return zoomInEnabled ? ENABLED : DISABLED;


Thanks


```python
for word in dico:
    elimated.add(word)
```

PS: `dico` est une `list` qui devrait être un `set` (ou un `OrderedDict` si l'ordre vous importe)


are you using this class to disable button? if so use ng-disabled. you can also add css based on this state (to design disabled button).


Wenn man htmlspecialchars benutzt, ist removeXSS nicht mehr nötig (removeXSS ist eh kaputt).


Except it is also being called by settingValueChanged and globalStackChanged, so it is a generic function that should be called whenever /something/ changes related to the decorator.


une ligne
```python
return (sum(value[word]
            for word in group])
        for group in group_words)
```


do not use strings directly in view. Instead, use translatable keys "selected_color" and an i18n translator for Angular


why do you need to wrap each directive with a div?


Sûr que vous voulez perdre le troisième joueur?


Does this mean that from this version, the driver doesn't support kernel 3.18?
Because I failed to compile it by using SDK of 15.05.1:
```
/main.c:404:2: error: implicit declaration of function 'ieee80211_hw_set' [-Werror=implicit-function-declaration]
  ieee80211_hw_set(hw, AMPDU_AGGREGATION);
```


#code


same with the divs


TeamCity OTA+ :: rvi_sota_server :: ci test [Build 253](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=3814&buildTypeId=ota_plus_RviSotaServer_CiTest) outcome was **SUCCESS**
Summary: Tests passed: 199, ignored: 3 Build time: 00:10:20


Or we can bug Keno to fix the windows issues so that you can use it too :)

Is ASTInterpreter still not working on windows then?


do not use scopes. instead use controllerAs with bindToController


same for scope


Does this take into account that the themes now reside in `admin-themes` directory?


where did the actual opening/closing go? :]


What if unsuceesful? Is the catch automatic?


The reporter of the bug has confirmed that this change has fixed his problem. Thanks!


What is point for stroke if it is not reused for all constraints of one style. DrawAs is reserved for Drawing mode in Canvas. The same name inside different contexts with different meaning is bad


TeamCity OTA+ :: rvi_sota_server :: ci test [Build 254](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=3818&buildTypeId=ota_plus_RviSotaServer_CiTest) is now running


telnet is not part of core any more
https://github.com/ruby/ruby/blob/v2_3_0/ChangeLog#L8498


this function should return the isOpen boolean


Indent more.


deleted gib ignore


@RouxRC
![ser](https://cloud.githubusercontent.com/assets/14234873/15706252/f56557a0-27f2-11e6-93bc-eec59ad18cde.png)


@luiges90 我改了
if (e.yesEffect.Count > 0 || e.noEffect.Count > 0 || e.yesArchitectureEffect.Count > 0 || e.noArchitectureEffect.Count > 0)
                {
                    //throw new Exception("yesEffect != null " + ev.yesEffect);

                    if (!this.Plugins.ConfirmationDialogPlugin.IsShowing)
                    {
                        //this.Plugins.tupianwenziPlugin.SetConfirmationDialog(this.Plugins.ConfirmationDialogPlugin, new GameDelegates.VoidFunction(base.Scenario.ApplyEvents(true), new GameDelegates.VoidFunction(base.Scenario.ApplyEvents(false)));
                        this.Plugins.ConfirmationDialogPlugin.ClearFunctions();
                        this.Plugins.ConfirmationDialogPlugin.AddYesFunction(new GameDelegates.VoidFunction(base.Scenario.ApplyYesEvents));
                        this.Plugins.ConfirmationDialogPlugin.SetPosition(ShowPosition.Center);
                        this.Plugins.ConfirmationDialogPlugin.AddNoFunction(new GameDelegates.VoidFunction(base.Scenario.ApplyNoEvents));

                        this.Plugins.ConfirmationDialogPlugin.IsShowing = true;
                    }
                }


is it just last block? if yes then please add assert for that. If not something else could be wrong here.


this should return a boolean and be called isNextButtonStateEnabled


Heh, that'd work as well ;)

~~I'm not totally sure, actually -- will try it again on master.~~ [Oh, nice.](https://github.com/Keno/Gallium.jl/commit/42d343de54a2231491a609a5a1c4b298c33ef35a).


do not directly touch DOM. get the element from a link function directive


@applqpak would you like me to revert those commits for you


use ngannotate and not the ['$q']


@applqpak i can also help you b disable permissions for people to edit your plugin just tell me what plugin do you want me to do that on


nice meme dude


K


For OpenWrt, mwlwifi is built with compatible (backports) wireless package. It is possible that the compatible (backports) wireless package you used is old version. You can modify this driver to work with previous version of compatible (backports) wireless package. For example, you can use previous way to set flags (hw->flags |= IEEE80211_HW_AMPDU_AGGREGATION;).


may be std::map or std::unordered_set


std::map<Stroke, id>


Why !current_user.time_zone.blank?,
time_zone can't be blank


ah, you did it!


@tmenier I fixed both  https://github.com/tmenier/Flurl/commit/5f2f468fb3713585961ff0eda069058aa9f68f12


Unnecessary parenthesis?


An example of how to rewrite this in a more functional way:

```scala
val urlParameters = value1.map(listElem => s"${listElem.identifierType},${listElem.identifier}").mkString("\n")
```


TeamCity OTA+ :: rvi_sota_server :: ci test [Build 254](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=3818&buildTypeId=ota_plus_RviSotaServer_CiTest) outcome was **SUCCESS**
Summary: Tests passed: 199, ignored: 3 Build time: 00:10:11


Another way is that you can update your compatible (backports) wireless package. Backports package is used to let updated drivers to work on old kernel.


@chhapil

Here is the commit required to toggle software CRCs in stweaks:
https://github.com/gsstudios/initramfs3/commit/61b8b2e7845381bbfd01696ec24012ccd97265ca


Force push een commit die het overschrijft


:valid -> :valid?


@luiges90 指定与某势力提升外交没有效果是怎么回事


TeamCity Tests Backend :: Build [Build 0.0.1.27](http://pc-kuld.edu.isd.dp.ua:8010/viewLog.html?buildId=430&buildTypeId=TestsBackend_Build) is now running


Угу. Сам проверил - всё работает. Фактически ES5-версия теперь нужна только для некрооперы.


```c++
class StrokeStore {
    std::map<Stroke, uint32_t> strokeToId;
    std::map<uint32_t, Stroke> idToStroke;
    uint32_t curId;
    const Stroke *getStrokeById(uint32_t id) const;
    uint32_t getIdByStroke(const Stroke &stroke) {
        if(found) retrun id;
        idToStroke[curId] = stroke;
        strokeToId[stroke] = curId;
        curId++;
    }
}

```


最新的代码已上传


TeamCity Tests Backend :: Build [Build 0.0.1.27](http://pc-kuld.edu.isd.dp.ua:8010/viewLog.html?buildId=430&buildTypeId=TestsBackend_Build) outcome was **SUCCESS**
Summary: Tests passed: 1 Build time: 00:00:54


@michaelbarton  I fixed with commit https://github.com/bioboxes/ray/commit/2786d26803e808c8944bda8a04cfcad6be5c6ec7


It looks like you have it white listed here. How can I white list this with Brave? This is an app built into Brave.


同时是最新代码


thanks mate

On Wed, Jun 1, 2016 at 3:58 PM, Josh <notifications@github.com> wrote:

> @chhapil <https://github.com/chhapil>
>
> Here is the commit required to toggle software CRCs in stweaks:
> gsstudios/initramfs3@61b8b2e
> <https://github.com/gsstudios/initramfs3/commit/61b8b2e7845381bbfd01696ec24012ccd97265ca>
>
> —
> You are receiving this because you were mentioned.
> Reply to this email directly, view it on GitHub
> <https://github.com/gsstudios/Dorimanx-SG2-I9100-Kernel/commit/5b9bbdcff91bbcc17907abbfa8fe775ca4d53e49#commitcomment-17693931>,
> or mute the thread
> <https://github.com/notifications/unsubscribe/AIpoHZR-Pw5qSpDw5HnrQHylYFwdIEuKks5qHV62gaJpZM4IrVPt>
> .
>

--
Thanks & Regards
Ruturaj


$level seems to be undefined, is it declared anywhere else?

should it not be $timeOfDay = abs($this->getLevel()->getTime() % 24000);


That's still the case, but #defines with value have now an additional boolean value to indicate wether they're used or not. This way the value of unused pins doesn't get lost when saving the configuration. The idea is to indicate the preferred pin to the user with something like "default" behind the pin name, but the user visible part didn't happen, yet.

As far as I can see the misbehavior @phord is seeing isn't really a bug, because it can happen only with handcrafted config files. Still it shouldn't be too difficult to guard against such misconfiguration by leveraging booleans to value/boolean pairs for all names known to be pins.


@joakim-hove All these tests run as expected on VS2015


TeamCity OTA+ :: rvi_sota_server :: ci test [Build 255](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=3821&buildTypeId=ota_plus_RviSotaServer_CiTest) is now running


what was the problem in this function ?
why do you change the code ?


Point and only Point. Dot is for Dot product.


Yep. On fresh installs, if you'd chosen 'hive neutral' from the setup screen, it got overwritten by the update script.

Now the update asks "which ones are valid?" It (currently) gets the answer back "hive and hiveneutral" plus any that have been manually ported to the `admin-themes` directory. If your current theme pref isn't one of the supported list, it defaults to hive.


TeamCity OTA+ :: rvi_sota_server :: ci test [Build 256](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=3822&buildTypeId=ota_plus_RviSotaServer_CiTest) is now running


You could use `ynh_user_exists "$admin"` helper instead if you want - see [here](https://github.com/YunoHost/yunohost/blob/stable/data/helpers.d/user#L7).


why not? does that help/simplify things to do that ?


if yes then do it :)


In fact: don't need to store anything in $self->{_species_tree} !
`return $self->method_link_species_set->species_tree(shift || 'default');` would be more concise


@aancel did you see that now the build are fine for develop branch
now it needs to work for other branches and PR and if it is successful then we want to push the feelpp-lib on dockerhub.


`btnStop.addEventListener('click', song.stop)`


The type `stype` instead of `style` has escalated quite a lot.

Upon rebasing my pr #31, I'm gonna move this whole block and fix the typo. I also use django-suit and had the same problems!


TeamCity OTA+ :: rvi_sota_server :: ci test [Build 256](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=3822&buildTypeId=ota_plus_RviSotaServer_CiTest) outcome was **UNKNOWN**
Summary: Canceled (Exit code 137 (new)) Build time: 00:02:56


Whoops, I need to fix the description a bit.


Ridi hajiii


English translation:

Custom scenery of Poland for FlightGear.

**9.04.2016**

Buildings for EPLR Radawiec have been added.

Airport procedures can be found in the Documentation/Procedures folder.


it is much easier now no ?
we need also to see what what do with different flavors of compilers  (and possibly version or external libs)


security issue.

root@zh-px6:~# touch
I-should-not-leave-root-passwords-on-public-github-repositories
root@zh-px6:~# ll
total 24
drwx------  4 root root 4096 Jun  1 06:26 ./
drwxr-xr-x 22 root root 4096 Jan  8  2015 ../
-rw-r--r--  1 root root 3106 Feb 19  2014 .bashrc
drwx------  2 root root 4096 Jun  1 05:44 .cache/
-rw-r--r--  1 root root  140 Feb 19  2014 .profile
-rw-r--r--  1 root root    0 Jun  1 06:26
I-should-not-leave-root-passwords-on-public-github-repositories
drwxr-xr-x  9 root root 4096 Jun  1 05:45 Skynet2.0/

thnx to Otto for finding this.


Out of interest: Why is this about the only model that is not a UM ListModel?


The package that you should update: package/kernel/mac80211.


This does not compile on Ubuntu 16.04. cpp-alignimages seems to work, if this line is swapped for:
std::to_string(libusb_get_device_address(device.usb_device));


You could use `ynh_package_install php-fpdf` instead if you want - see [here](https://github.com/YunoHost/yunohost/blob/stable/data/helpers.d/package#L45)!


Could we use a jsdoc annotation like `@name` instead of low level hacks?
(What about inheritance)


Hi Israel,

wasn't able to post the translation to the readme.md because as it said I
do not have access to it.

*Hue Peter Coal*
JAFVA, Vice President
"Never fly where you have not flown in your mind, five minutes ago."

On Sat, May 14, 2016 at 8:26 PM, FGDATA <notifications@github.com> wrote:

> @SP-K79 <https://github.com/SP-K79>
>
> Would you mind adding an English translation?
> (selfish reason: My Polish is rough in the edges)
>
> —
> You are receiving this because you were mentioned.
> Reply to this email directly or view it on GitHub
> <https://github.com/FGMEMBERS-SCENERY/Poland-fg-CustomScenery/commit/2fd69bcadfaa0242e4203a94e0f8e529f92df94d#commitcomment-17477745>
>


Wtf never Vec


TeamCity OTA+ :: rvi_sota_server :: ci test [Build 255](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=3821&buildTypeId=ota_plus_RviSotaServer_CiTest) outcome was **SUCCESS**
Summary: Tests passed: 199, ignored: 3 Build time: 00:07:50


Wtf never Box, not inde Graphix 3d app


Ok, thanks for the clarification


You could also use `ynh_user_exists` if you want! :)


Description fixed here: https://github.com/gsstudios/initramfs3/commit/663b15701cdd7eea1d12a877a894d9dfb4b4ae93


Can you update the manaicons with the one you provided on the forums that has new icons, thank you!


What about globes?  ads are still here


Don't you want to store this new setting?


You don't need this line or the others like it. The template should automatically update when the collection changes.


@raphaeljolivet, I'm not familiar with jax-rs sub-project and I'm away from the computer for 2 weeks, so i can't look into it properly. Could You look into it? Is this change reosanable, or should writing be done somewhere else?


Or try harder with setting a `@module` at the top of the file?


Vector is std::vector,
Vec -> Vec2, Vec3, Vec4 etc.
This is mostly gfx app, there is no place for such renaming.
just let leave it std::vector. is will be better.


`return ["c", "c#", "d", ...][noteNumber]`. You could also throw in a guard to ensure that `noteNumber < array length`


Is  "using="  c++14 trick?


Rc says no info. Ptr just better.


Clear first and then create? Looks a bit suspect to me.


.'
e.r.


breaks build off enigma2-plugins


"Hybrid"


This shouldn't have been merged, but I suppose bug squad does not know russian xD


Men sizeof ( integer ) vil jo bli trøbbel?


Or just Array says more. std::vector is array in fact.


I moved my local folder so some files showed up as modified.


後出しで、すみません、、、
UT の中でこのルートって通りますか？
get(vendor, None) にしなくても、大丈夫ですか？


Please tell me why it should be better ? It is for any class but not finished yet.


引数名が experimenter なのは、修正が難しいからでしょうか？


I noticed that kernel adiutor wasn't able to change this parameter, so that's why I included a toggle in stweaks :)


This class is not finished, i have to search more about this in my non public code and make it public.


Now I need to check if this setting stays after rebooting.


I wasn't sure who fully made it so...I may have given the wrong credits to the wrong person


I just pushed a new[]/delete[] fix so that this is not holding people up.
Will install a better solution down the road.

On Tue, May 31, 2016 at 5:06 PM, Jeff Hammond <notifications@github.com>
wrote:

> Rather than alloca, I would just have a fixed-size stack allocation that
> is large enough for all of the common cases, and fall back to malloc if
> necessary.
>
> if (m*n < 10000) {
>   double x[10000];
> } else {
>   double * x = malloc(m*n*sizeof(double));
> }
>
> —
> You are receiving this because you authored the thread.
> Reply to this email directly, view it on GitHub
> <https://github.com/m-a-d-n-e-s-s/madness/commit/1fe04a89eeafa8e269c014d98ed8ee269e6cee9e#commitcomment-17686514>,
> or mute the thread
> <https://github.com/notifications/unsubscribe/AHMgPR3ZuKEJMO0jV1zWEBoXa_KLemlzks5qHKLYgaJpZM4Iq592>
> .
>

--
Robert J. Harrison
tel:  865-272-9262


Jau, das brach die Hydra Builds. Mal schauen, ob ich das beizeiten fixe.


# why delete the hat...? [jew](https://en.wikipedia.org/wiki/Jews) that


Brutal honesty :laughing:


@yuhhaurlin openwrt has not received the mac80211 updates yet. Right now only LEDE has the updated packages, CC is still the stable Openwrt release meaning we should create an inverse patch to add these checks. Then upload them to CC branch to patch the latest code.


chmod
ugoa[+-=
[root@localhost~]#
r--4
w--2
x--1


@copumpkin Done in 9f0e137338a0c8186a104b4af5c6d49d09552784.


@Poincare I'm super excited for this!


No it doesn't. You even said you made psgo which you did not.Stop taking false credit.


```php
<?php

class A{
   public static int $x;

   public static string $z;

   public int $y;
}

A::$x;// nodeType:TypeInteger      propWrapper:PropertyWrapper

A::$z;// nodeType:TypeString        propWrapper:PropertyWrapper

A::$y;// nodeType:TypeError         propWrapper:nullptr

?>

```


Move the cursor with arrow keys. Save the map with S, load a new map with L. Move game forward with enter and enter / exit "play mode" with R.


Also have you added this patch to upstream? https://github.com/lede-project/source/blob/master/package/kernel/mwlwifi/patches/110-api_sync.patch

After trying to use the latest commit to build LEDE it fails at this patch and I was going to fix it but I can't fix it right now.


Evt `if main_forening.handles_child_payments:`...


All good. Settings stay after reboot, works fine. :)


Actually, those quote seems not necessary. It's working without. I believe Gradle add them automatically even if it doesn't print it while logging the command..


With a file "list.txt" with two entries "play" and "ldskjfsdkfsn"

```
user:~/workspace (master) $ go run main.go -m dns -u google.com -t 2 -w list.txt

=====================================================
Gobuster v1.0 (DIR support by OJ Reeves @TheColonial)
              (DNS support by Peleus     @0x42424242)
=====================================================
[+] Mode         : dns
[+] Url/Domain   : google.com
[+] Threads      : 2
[+] Wordlist     : list.txt
=====================================================
Found: play.google.com
=====================================================
user:~/workspace (master) $ go run main.go -m dns -u gooogllkjsdflkjdsfe.com -t 2 -w list.txt
[!] Error : Could not validate base domain.
user:~/workspace (master) $ go run main.go -m dns -u 192.168.1.1 -t 2 -w list.txt

=====================================================
Gobuster v1.0 (DIR support by OJ Reeves @TheColonial)
              (DNS support by Peleus     @0x42424242)
=====================================================
[+] Mode         : dns
[+] Url/Domain   : 192.168.1.1
[+] Threads      : 2
[+] Wordlist     : list.txt
=====================================================
=====================================================
user:~/workspace (master) $ go run main.go -m dns -u 192.168.1.1111 -t 2 -w list.txt
[!] Error : Could not validate base domain.

```

I'm not sure that line 189 is the correct place to put the function, but it seems to resolve the issue that was outlined.  I'm happy to make any requested changes.


Fine by me.


Awesome. Will you now be using libTooling now instead of libclang?


Would it be possible to use the latest version of jsgraph ? It is not easy to integrate jsNMR in the visualizer if it's not compatible with the version of jsgraph that we have there


@kslong @Higginbottom @smangham -- think I had made a mistake here during one of the merges which had accidentally duplicated some code and led to incorrect xi being reported as mentioned in telecon. This fixes it.


si seguramente sea asi pero a lo mejor hay un comando para que salte directamente el reproductor VLC


I fixed it with the latest version...


I guess in it's current state: probably not. I just change stuff as I go along. Summary of the main things I've done:

- archlinux compatibility
  - incomplete. I guess all install-root scripts should be changed before a merge. At least they should print a not-supported message.
  - not sure if you want this in your repo at all?
- ctf-tools virtualenv and install scripts that use pip to install (even from git)
  - would need a uninstall scripts for all those tools.
- multiple gdb extension handling
  - init-$extension gdb commands added to ~/.gdbinit
  - gdb-$extension wrapper shell scripts

If you want I can try to cherry-pick those three things into separate branches and then make a pull request.
We could just merge everything in bulk, which would be less work for me.

In any case this needs a little more work. But if you want to merge it I'll clean up and update the documentation.


兩個勢力也需要同一個`f2`的ID?


why it wasn't working?  that should be a standard annotation.


vanaf deze comit werkt het


I had this:
```
/home/syrcam/git/supernovae/core/src/main/java/supernovae/JmeImageView.java:37: error: method does not override or implement a method from a supertype
	@Override
	^
```


did you pull the alchemist branch?


Tu as raison!

Et en plus j'ai utilisé la syntaxe du test de commande avec trap car j'avais rencontré ce problème!
J'ai bêtement repris les test des scripts sans regarder...

Pour pallier tout les usages, je proposerais ça:
```bash
if ynh_package_is_installed "PACKAGE"; then
	echo "Le package est installé."
else
	echo "Le package n'est pas installé."
fi
```


had made some changes


@PaulParau that's great, thank you :)


Hmm can be that i pulled it afterwards, not sure...


resolve


dfhgdfgh


Apparently, I misuderstood the PETSc behavior.


**[unitTests_php_branches run failed](http://jenkins.wikia-prod:8080/job/unitTests_php_branches/31558/testReport/) for this commit**


Please check README. I modified this driver based on   "package/kernel/mac80211" of OpenWrt main trunk.


Je sais pas si tu as vu, je me suis permis de faire une 'tite relecture tout ça, et j'ai changé pour ça : https://github.com/YunoHost/doc/commit/aa50ef79e96a40dd705b091799202c0e109afcf4#diff-8e5b837b08ee81f9516c8daefdc13c34R60. Je pense que c'est intéressant de garder le `if ! ...`, car généralement, on cherchera plus à faire qqch si le paquet n'est pas installé...

Un grand merci en tout cas pour ce travail ! :)


ok, lets the code like it is currently. I asked just for understanding


Yay! Why removing EVERYWHERE? :)


给他


I am a nice comment


This solves the missing logic that was supposed to switch on PE workqueue if disabled while phone is sleeping, and switch back to original setting when phone wakes up.


This is for supporting legacy AMD drivers back ?


You should use standard function "reverse". Make your own implementation.


Tips, kan göra dessa `inline` så slipper man funktionshopp :)


TODOは終わったら消しましょう。


I've added some basic misconfiguration fixing on branch *fixpins*, it's a single commit: https://github.com/Traumflug/Teacup_Firmware/commit/09bcb0cf4234fbdd8e8b02bcb6e32de0b053d5e4

Does this look good?


Maybe remove this empty space


Actually I guess you don't need to uninstall the readline first - worked for me at least w/o uninstalling. Using **depends_on** makes brew search for readline in the brew "environment" first it seems.


It's probably easy enough to make a javascript checkbox that lets the user decide on subscription or importing the events.


Pas de problème, j'ai fait une branche pour qu'on se permette de retoucher avant de publier justement.

Ça me convient très bien comme ça aussi. J'ai vu que tu l'as fait également sur les autres test. C'est parfait.


多看书，少说胡话，谢谢大家。


If I have to run zpool list or zpool status more than once, the code is useless. The system will be doing too much unnecessary work.

This is why I would prefer the JSON be done correctly, or not at all.


Working snake with endscreen and restart option


Please check README. If version of "package/kernel/mac80211" is 2016-01-10, you just need to rename the folder "patches".
Marvell will base on this version of code to create patch for upstream.


This commit still requires testing.


the ② effect of 守護神官マハード


TeamCity OTA+ :: rvi_sota_server :: ci test [Build 257](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=3831&buildTypeId=ota_plus_RviSotaServer_CiTest) is now running


今後は、コメントアウトは残さずに削除するようにしてください。コードもきれいに保つためには断捨離が欠かせません。
こまめにコミットすれば、全てのコミットが記録されるので、消したコードが必要になった時には履歴から復旧出来ます。


I am pretty sure I fixed it if not please tell me. But we did not intended to take the credit for these chat plugins. I do apologize if it came off like that but Richard Adams changed the credits on a few things.


That said, it might be better to do such fixes during loading the configuration. *board.generic.h* knows which pins exist and which keys are actually pins.


@prudhomm Regarding to merging the repo in feelpp, it should avoid an additional pull in travis, and should make it easier for user to build Feel++. I'll test that and put the dockerfiles in ports/docker.
Indeed, everything seems to be building fine !
Ok I'll look in the other branches and do some cherry-picking to get the travis file back. For dockerhub, we have to use an account for Feel++ to be pushed, cf. https://docs.travis-ci.com/user/docker/#Pushing-a-Docker-Image-to-a-Registry. Could we use the feelpp account info directly ?
I find it easier indeed ! Regarding to different flavours, we could install additional compilers in when building the feelpp-lib image, and keep the dependencies built as they are now, so we don't increase the base image size. What do you think about it ?


I should've read the diff properly :/ Thanks @thwarted

f3cfb45


We could keep the repo separate if you prefer and add a submodule in ports ?


You also said you created this chat plugin which is also false.


@fabled Right, that was not explicit on MPRIS2 specs, but yes definitely on DBus ones. There is room for a future pull-request...


This is missing a closing `}`


Je sais pas comment modifier le commit, j'aime toujours pas git, faut pas déconner quand même!
En changeant le mot, il y a une faute d'accord.


security issue is fixed, password has been changed.
It provided access to a vps used for doing integration tests (this structure is also in master using an even more outdated password)


There's no way to import two pools of the same name, or with a null name. There's no good reason to avoid having the pool name as the key in zpool list or zpool status.

In fact, you can look at it as though "status" is a key, "tank" is an object that contains the info for pool named "tank".

Having to loop through the output each time just to find out the pool names isn't any more efficient than using `zpool list -H` and using regex to split the lines and pull pertinent information.


`01-01 10:23:52.619     0     0 E mmi_wls_chrg_get_psy_info: get_prop Fail!
01-01 10:23:52.619     0     0 E mmi-wls-charger 0-006c: Error Reading Taper Reached status`

Is caused by this commit.


Hi,
i have a crash when reusing adapter with CA:

[01/06 14:31:05.016 AD1]: Requested adapter 1 close due to timeout, result 1 max_rtime 0
[01/06 14:31:05.017 AD1]: sockets_del: 8 -> handle 11, sid 1
[01/06 14:31:05.017 AD1]: closing DVR socket -1 pos 8 aid 1
[01/06 14:31:05.017 AD1]: closing adapter 1  -> fe:10 dvr:11
[01/06 14:31:05.017 AD1]: adapter.c:754: get_adapter returns NULL for adapter_id 1
[01/06 14:31:05.017 AD1]: adapter.c:570: get_adapter returns NULL for adapter_id 1
[01/06 14:31:05.017 AD1]: closing CA device 1, fd 12
[01/06 14:31:05.017 AD1]: ca_session_callback: reason 5 slot_id 0 session_number 1 resource_id 10041
[01/06 14:31:05.017 AD1]: ca_session_callback: reason 5 slot_id 0 session_number 2 resource_id 20041
[01/06 14:31:05.017 AD1]: ca_session_callback: reason 5 slot_id 0 session_number 3 resource_id 30041
[01/06 14:31:05.018 AD1]: done closing adapter 1
[01/06 14:31:05.021 AD1]: sockets_del: 8 Last open socket is at index 6 current_handle 11
[01/06 14:31:05.121 AD1]: No enabled sockets for Thread ID b58ff390 name AD1 ... exiting
[01/06 14:31:05.121 AD1]: add_join_thread: pthread b58ff390
[01/06 14:31:15.633 main]: Mutex init 0x58af0
[01/06 14:31:15.633 main]: sockets_add: handle 8 (type 1) returning socket index 7 [194.54.148.8:58690] read: 0x17b6d
[01/06 14:31:15.634 main]: minisatip.c:657 get_sid returns NULL for s_id = -1
[01/06 14:31:15.634 main]: read RTSP (from handle 8 sock_id 7, len: 0, sid -1):
[01/06 14:31:15.634 main]: detect_dvb_parameters (S)-> src=1&fe=2&freq=11842&sr=27500&msys=dvbs2&mtype=8psk&pol=l&fec=34&pids=0,18,250,251,252
[01/06 14:31:15.634 main]: detect_dvb_parameters (E) -> src=1, fe=2, freq=11842000, fec=3, sr=27500000, pol=4, ro=3, msys=6, mtype=9, plts=2, bw=8000000, inv=2, pids=0,18,250,251,252 - apids=NULL - dpids=NULL x_pmt=NULL
[01/06 14:31:15.634 main]: Setup stream -1 parameters, sock_id 7, handle 8
[01/06 14:31:15.634 main]: stream.c:178 get_sid returns NULL for s_id = -1
[01/06 14:31:15.634 main]: Mutex init 0x63628
[01/06 14:31:15.634 main]: set_sock_lock: sock_id 7 locks also mutex 0x63628
[01/06 14:31:15.634 main]: Mutex init 0x5ee08
[01/06 14:31:15.635 main]: sockets_add: handle -2 (type 0) returning socket index 8 [0.0.0.0:0] read: 0x17ba5
[01/06 14:31:15.635 main]: set_sock_lock: sock_id 8 locks also mutex 0x63628
[01/06 14:31:15.635 main]: Setup stream done: sid: 0 for sock 7 handle 8
[01/06 14:31:15.635 main]: copy_dvb_param start -> src=0, fe=0, freq=0, fec=9, sr=0, pol=0, ro=3, msys=0, mtype=-1, plts=2, bw=8000000, inv=2, pids=NULL, apids=NULL, dpids=NULL x_pmt=NULL
[01/06 14:31:15.635 main]: copy_dvb_parameters -> src=1, fe=2, freq=11842000, fec=3 sr=27500000, pol=4, ro=3, msys=6, mtype=9, plts=2, bw=8000000, inv=2, pids=0,18,250,251,252, apids=NULL, dpids=NULL x_pmt=NULL
[01/06 14:31:15.635 main]: Play for stream 0, type 1, rsock 8, adapter -1, sock_id 7 handle 8
[01/06 14:31:15.635 main]: stream.c:248: get_adapter returns NULL for adapter_id -1
[01/06 14:31:15.635 main]: get free adapter 2 - a[1] => e:0 m:-1 sid_cnt:0 f:11842000 pol=4 sys: dvbs dvbs2
[01/06 14:31:15.635 main]: Mutex init 0x51b28
[01/06 14:31:15.635 main]: trying to open [1] adapter 1 and frontend 0
[01/06 14:31:15.637 main]: opened DVB adapter 1 fe:9 dvr:10
[01/06 14:31:15.641 main]: DVR buffer set to 5775360 bytes
[01/06 14:31:15.641 main]: deleting pids on adapter 1, sid -1, pids=NULL
[01/06 14:31:15.641 main]: clearing type and pat processed for adapter 1
[01/06 14:31:15.642 main]: Mutex init 0x63568
[01/06 14:31:15.642 main]: sockets_add: handle 10 (type 5) returning socket index 9 [0.0.0.0:0] read: 0x17b6d
[01/06 14:31:15.642 main]: set_socket_thread: thread b58ff390 for sockets 9
[01/06 14:31:15.643 main]: pthread_setschedparam failed with error 22
[01/06 14:31:15.643 AD1]: Starting select_and_execute on thread ID b58ff390, thread_name AD1
[01/06 14:31:17.666 main]: initializing CA, fd 11 type 2 flags 0x3
RECEIVED SIGNAL 11 - SP=0 IP=0
[01/06 14:32:15.742 AD1]: Requested adapter 1 close due to timeout, result 1 max_rtime 0
[01/06 14:32:15.743 AD1]: sockets_del: 9 -> handle 10, sid 1
[01/06 14:32:15.743 AD1]: closing DVR socket -1 pos 9 aid 1


I changed the API and let the driver accept AMSDU capability from mac80211. But I don't know why it needs to change IEEE80211XXX to NL802XXX. Mac80211 had redefined IEEE802XXX to NL802XXX. I still keep the driver use IEEE802XXX. BTW, you need to use 10.3.0.17-20160601 to have all these modifications. Thanks.


Thanks!


On risque d'avoir le même problème que précédemment sur cette commande.
Arg! J'ai plus le temps!!!
Je regarderais ce soir...


Left comment


Yep, my fault. I believed to git's merging abilities and didn't noticed that it screw up my changes with Adriano's ones.


Which toolbar are you using? The buttons of 32-bit toolbars were never
grayed out.


ist noch interner test, wird noch geändert


TeamCity OTA+ :: rvi_sota_server :: ci test [Build 257](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=3831&buildTypeId=ota_plus_RviSotaServer_CiTest) outcome was **SUCCESS**
Summary: Tests passed: 199, ignored: 3 Build time: 00:07:50


It didn't work?


Hello
Congratulations for the work .
It is possible to make the video stream to a server?


Buchoはテストコードではなく実コードではないでしょうか。
実コードは、src/main/java配下に置きます。
ShachoTest以外のファイルは全部mainじゃないかと思います。


Thanks...it's still work in progress...


Workqueue control fixed in this commit:
https://github.com/gsstudios/initramfs3/commit/69cb8a0e103265fe9609778cfb74fead6f2093d7


ここって何で消したの？？


user_data 와 project_member의 id는 독립적으로 생성할 수 있으나 user_data의 id를 지우면 project member의 id와 같은 행의 데이터는 모두 삭제되게 작동됨.


BuchoはShachoである、というのは間違えていそうです。Shain?


This commit is all good. Workqueue control is now properly running and now shows up in logcat.


덧붙여, 가장 최근에 회의한 내용을 반영한 데이터베이스 구조임.


> If you're refactoring without tests, you're just changing shit.


This commit is safe. :)


Why does the first argument get discarded here?


@sh00ki Why do we need that? We are not using the data base's data from phpmyadmin?


gfgf


don't know what i'm doing!!! Help!


Fixed with this:
https://github.com/cisco/libsrtp/commit/bb8477a15655e65e61ceba5a2aa259819bc11edd

Paul

------ Original Message ------
From: "hankpeng" <notifications@github.com>
To: "cisco/libsrtp" <libsrtp@noreply.github.com>
Cc: "Paul E. Jones" <paulej@packetizer.com>; "Author"
<author@noreply.github.com>
Sent: 5/31/2016 6:34:10 PM
Subject: Re: [cisco/libsrtp] Removed "ekt" field from policy in GCM test
routine (53a6b89)

>Another missing deletion is found at
>https://github.com/cisco/libsrtp/blob/2_0_0_ekt_dev/test/rtp_decoder.c#L272.
>
>—
>You are receiving this because you authored the thread.
>Reply to this email directly, view it on GitHub, or mute the thread.
>


[CodeFactor](http://localhost:40641/repository/github/cordis-dev/cf-pr-test-1) found multiple issues last seen at c0c96cdf79e8fab59c1ac9ca30fcbc6db7b1254a:

#### A statement containing curly brackets must not be placed on a single line. The opening and closing curly brackets must each be placed on their own line.
1. [c1.cs](http://localhost:40641/repository/github/cordis-dev/cf-pr-test-1/files/master/c1.cs):[13](http://localhost:40641/repository/github/cordis-dev/cf-pr-test-1/files/source/master/c1.cs#13)

2. [c1.cs](http://localhost:40641/repository/github/cordis-dev/cf-pr-test-1/files/master/c1.cs):[16](http://localhost:40641/repository/github/cordis-dev/cf-pr-test-1/files/source/master/c1.cs#16)

3. [c1.cs](http://localhost:40641/repository/github/cordis-dev/cf-pr-test-1/files/master/c1.cs):[14](http://localhost:40641/repository/github/cordis-dev/cf-pr-test-1/files/source/master/c1.cs#14)

4. [c1.cs](http://localhost:40641/repository/github/cordis-dev/cf-pr-test-1/files/master/c1.cs):[15](http://localhost:40641/repository/github/cordis-dev/cf-pr-test-1/files/source/master/c1.cs#15)

#### Class names begin with an upper-case letter: t1.
1. [c1.cs](http://localhost:40641/repository/github/cordis-dev/cf-pr-test-1/files/master/c1.cs):[1](http://localhost:40641/repository/github/cordis-dev/cf-pr-test-1/files/source/master/c1.cs#1)

#### A line may only contain a single statement.
1. [c1.cs](http://localhost:40641/repository/github/cordis-dev/cf-pr-test-1/files/master/c1.cs):[14](http://localhost:40641/repository/github/cordis-dev/cf-pr-test-1/files/source/master/c1.cs#14)

2. [c1.cs](http://localhost:40641/repository/github/cordis-dev/cf-pr-test-1/files/master/c1.cs):[16](http://localhost:40641/repository/github/cordis-dev/cf-pr-test-1/files/source/master/c1.cs#16)

#### Complex Method
1. [c1.cs](http://localhost:40641/repository/github/cordis-dev/cf-pr-test-1/files/master/c1.cs):[3-118](http://localhost:40641/repository/github/cordis-dev/cf-pr-test-1/files/source/master/c1.cs#3-118)

#### Method names begin with an upper-case letter: processResponseOk.
1. [c1.cs](http://localhost:40641/repository/github/cordis-dev/cf-pr-test-1/files/master/c1.cs):[3](http://localhost:40641/repository/github/cordis-dev/cf-pr-test-1/files/source/master/c1.cs#3)

#### A closing curly bracket must not be preceded by a blank line.
1. [c1.cs](http://localhost:40641/repository/github/cordis-dev/cf-pr-test-1/files/master/c1.cs):[78](http://localhost:40641/repository/github/cordis-dev/cf-pr-test-1/files/source/master/c1.cs#78)


@knorth55 FYI


nice :+1:


> 一担当でも主任でも部長でも、おおよそ社員は社長に「起立！」と命じられれば皆立ち上がります。
が、立ち上がり方は三者三様です。

というプログラムで、その物語をコードで表現しています。
どんな意味を持ってこう書いているのか、読み取りながら書いてみてください。


すばらしい！ですが、授業の内容を踏まえるなら、行の追加のない修正がほしいところ
29行目みたいな記述がよりよい記述ですよ！
`@book = Book.new(book_params)`


@lucasmichot IMHO, this limitation is stupid. I need to cache things for less than 1 min. If you don't want to cache for less than one min, then don't pass a minute less than 1 min.


@sh00ki Is this additional is necessary?


mas models referentes a localização nao deveriam ser as models ja definidas na app localização? E a model de person, nao deveria ser a definida na app clientes (que pode ser renomeada, se for o caso)?


ptx0,
If the code (zfs in this case) that produces the json has a bug or any
problems which output a zpool name like None/null, your code is broken
(according the language using the json output).

But if the value is None/null, you can treat the case and trigger an
exception or whatever you want.
We understand the shortcut you want to have, but the good reason is the
security of YOUR code. These are just best practices of development,
nothing else. Believe us :)
Thank you

2016-06-01 13:29 GMT+02:00 ptx0 <notifications@github.com>:

> If I have to run zpool list or zpool status more than once, the code is
> useless. The system will be doing too much unnecessary work.
>
> This is why I would prefer the JSON be done correctly, or not at all.
>
> —
> You are receiving this because you commented.
> Reply to this email directly, view it on GitHub
> <https://github.com/Alyseo/zfs/commit/74b25ae5f4510b4f548f2af06a27f26b5d7cfe2d#commitcomment-17694681>,
> or mute the thread
> <https://github.com/notifications/unsubscribe/ACuCCW2leRC49oleMfOKvVycIhf7BTdiks5qHW0rgaJpZM4IcKPx>
> .
>


Think this should be a named route


We are require the `spec_helper` in the `.rspec` by default


[CodeFactor](http://localhost:40641/repository/github/cordis-dev/cf-pr-test-1) found multiple issues last seen at c0c96cdf79e8fab59c1ac9ca30fcbc6db7b1254a:

#### A statement containing curly brackets must not be placed on a single line. The opening and closing curly brackets must each be placed on their own line.
1. [c1.cs](http://localhost:40641/repository/github/cordis-dev/cf-pr-test-1/files/master/c1.cs):[13](http://localhost:40641/repository/github/cordis-dev/cf-pr-test-1/files/source/master/c1.cs#13)

2. [c1.cs](http://localhost:40641/repository/github/cordis-dev/cf-pr-test-1/files/master/c1.cs):[16](http://localhost:40641/repository/github/cordis-dev/cf-pr-test-1/files/source/master/c1.cs#16)

3. [c1.cs](http://localhost:40641/repository/github/cordis-dev/cf-pr-test-1/files/master/c1.cs):[14](http://localhost:40641/repository/github/cordis-dev/cf-pr-test-1/files/source/master/c1.cs#14)

4. [c1.cs](http://localhost:40641/repository/github/cordis-dev/cf-pr-test-1/files/master/c1.cs):[15](http://localhost:40641/repository/github/cordis-dev/cf-pr-test-1/files/source/master/c1.cs#15)

#### Class names begin with an upper-case letter: t1.
[c1.cs](http://localhost:40641/repository/github/cordis-dev/cf-pr-test-1/files/master/c1.cs):[1](http://localhost:40641/repository/github/cordis-dev/cf-pr-test-1/files/source/master/c1.cs#1)

#### A line may only contain a single statement.
1. [c1.cs](http://localhost:40641/repository/github/cordis-dev/cf-pr-test-1/files/master/c1.cs):[14](http://localhost:40641/repository/github/cordis-dev/cf-pr-test-1/files/source/master/c1.cs#14)

2. [c1.cs](http://localhost:40641/repository/github/cordis-dev/cf-pr-test-1/files/master/c1.cs):[16](http://localhost:40641/repository/github/cordis-dev/cf-pr-test-1/files/source/master/c1.cs#16)

#### Complex Method
[c1.cs](http://localhost:40641/repository/github/cordis-dev/cf-pr-test-1/files/master/c1.cs):[3-118](http://localhost:40641/repository/github/cordis-dev/cf-pr-test-1/files/source/master/c1.cs#3-118)

#### Method names begin with an upper-case letter: processResponseOk.
[c1.cs](http://localhost:40641/repository/github/cordis-dev/cf-pr-test-1/files/master/c1.cs):[3](http://localhost:40641/repository/github/cordis-dev/cf-pr-test-1/files/source/master/c1.cs#3)

#### A closing curly bracket must not be preceded by a blank line.
[c1.cs](http://localhost:40641/repository/github/cordis-dev/cf-pr-test-1/files/master/c1.cs):[78](http://localhost:40641/repository/github/cordis-dev/cf-pr-test-1/files/source/master/c1.cs#78)


This can’t be empty. See [`fog`](https://github.com/caskroom/homebrew-cask/blob/37739567471a154122d55a05d267dc0d8280e246/Casks/fog.rb#L16L21) and [`appearin`](https://github.com/caskroom/homebrew-cask/blob/37739567471a154122d55a05d267dc0d8280e246/Casks/appearin.rb#L16L21) for typical `zap`s of Electron apps.


Now we have double free, if we interrupt server with tasks running.
And `TaskRun` and `af::Render` delete the same `af::TaskExec`.


Would it be possible to add an option to do this on file save, just before elm-format?


# query
`SELECT * FROM Photos LEFT JOIN (SELECT Waardering.PhotoID AS pid, AVG(Rating) AS rating FROM Waardering GROUP BY Waardering.PhotoID) AS ratings ON ratings.pid = Photos.PhotoId WHERE Photos.UserId IN (SELECT UserId FROM USERS WHERE IsAccepted = 1) AND IsProcessed = 1 ORDER BY UploadDate DESC`


やりたいことはわかります！
けど、これだと更新画面にも影響出ちゃうね
新規登録(newアクション)の時だ設定してね


ff


24行目と一緒でここも！


Thanks for clarification. But on my system it's not autodetected and also v1 installed that's why I enabled gpg2.. Crypt_GPG detects v1 only.

Thanks again


Could use the factory user trait here https://github.com/solidusio/solidus/blob/master/core/lib/spree/testing_support/factories/user_factory.rb#L17
e.g. `create(:user, :with_api_key)`


This should be `logger.Error("delete-vhost-failed", err)`


[CodeFactor](http://localhost:40641/repository/github/cordis-dev/cf-pr-test-1) found multiple issues last seen at c0c96cdf79e8fab59c1ac9ca30fcbc6db7b1254a:

#### Duplicate Code
- [t2.cs](http://localhost:40641/repository/github/cordis-dev/cf-pr-test-1/files/master/t2.cs):[3-17](http://localhost:40641/repository/github/cordis-dev/cf-pr-test-1/files/source/master/t2.cs#3-17)
[t1.cs](http://localhost:40641/repository/github/cordis-dev/cf-pr-test-1/files/master/t1.cs):[3-17](http://localhost:40641/repository/github/cordis-dev/cf-pr-test-1/files/source/master/t1.cs#3-17)

#### Class names begin with an upper-case letter: t1.
- [t1.cs](http://localhost:40641/repository/github/cordis-dev/cf-pr-test-1/files/master/t1.cs):[1](http://localhost:40641/repository/github/cordis-dev/cf-pr-test-1/files/source/master/t1.cs#1)


Close #1


Actually, the quote *are* needed, if you want to filter more of them.


@JonCubed might be of some interest - regional settings powershell


Hi,

may i ask why this change was introduced, it does not comply with the standards for Redhat and CentOS

https://access.redhat.com/documentation/en-US/Red_Hat_Enterprise_Linux/7/html/Networking_Guide/ch-Configure_Host_Names.html

in contrary it is recommended to use the FQDN in /etc/sysconfig/network as well as /etc/hostname.

The current behaviour breaks for example the automatic inclusion into something like puppet if that is based on FQDN

Regards
Soeren


Nit: should be “GitHub” instead of “Github”


@rogeriojoras mesmo caso do CPF


Hey, a tester tried it, same problem again, volume up not working but vol down and play/pause works on a 3 button headset.


added checkstyle plugin


Este arquivo tem exatamente o mesmo conteudo que "SG.apps.localizacao.models". Qual a necessidade?


Yea

Get Outlook for Android

On Tue, May 31, 2016 at 8:47 AM -0700, "Simao" <notifications@github.com> wrote:

Move back to 39?

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.


A herança da model esta errada.


Fixtures that are unique to your application. These can't go into a bundle, since they should not appear in other applications using a bundle.


I don't see this release on npm, ideas?
<img src="https://cloud.githubusercontent.com/assets/4278113/15708854/1b828544-27b6-11e6-864a-02347bdde6c8.png">


Dans les descriptions des documents sur NH le <em> est souvent utilisé pour mettre en emphase des noms, par exemples des noms d'émissions. Ici on pert la lisibilité de cette emphase. Est-il possible de prévoir quelque chose pour les <em> ? cc @kernity


このクラスは実処理を行うクラスですね。
実処理を行うクラスは、src/main/javaに置いてください。

mavanは、src/test/以下に置かれたソースファイルはテスト時のみ参照し、ビルド結果のjarやwarといった最終成果物にはsrc/main/配下の物だけを含める、という仕様になっています。
src/mainとsrc/testの区別は大事になりますので、身につけてください。


@rogeriojoras essa model nao deveria estar na app de clientes (que se for o caso, pode ser renomeada)?


I am a nice comment


@kamilsk Does the `$patterns` parameter contain a list of sorted patterns, with the ones that start with `!` at the end of the array?


good catch, fixed


Script generating that iframe:
https://101.livere.co.kr/js/livere8_lib.js


Tracker by Bizspring(bizspring.co.kr), see http://newspim.com/js/bizspring.js for the code.
121.254.169.8/InsightTrk


I think now it is possible to revert this change. this feature works well.


This is gonna remove the default flow, regardless of whether the sequence flow was default or not, right? You'd want to remove it only when the flow was the default flow before.


Javaでは、メソッド名およびフィールド名は小文字で始める慣習があります。最初のGは小文字にしてください。

また、1.05を掛けているので税込価格を取得するのだと思いますが、getIncludedPriceでは何込価格かわかりません。
わざわざメソッドの実装まで見ないと何のメソッドかわからないような名前になってしまっていると、呼出し元のコードを読み解くために全ての呼出しメソッドの中まで辿って行かないと何の処理なのか理解できなくなってしまいます。
名前はとても大切です。名前にしっかりと注意を払う習慣を身につけてください。


Missing: Test case that verifies that the default flow is only unset, if the changed flow was the default flow.


:D


test


TODO: Coding style (space between `if(`, `){`). See below as well.


コメントアウトを残さないようにしてください。


メソッドに対してコメントを書くときは、/**  */で囲むと良いです。
JavaDocコメントと言って、一定の書式で書いたコメントを元に自動でドキュメントを生成してくれます。


这个是我的第一个github的project。


@justincormack thanks for your feedback. On s390x (64 bit) we can run s390 (32 bit) programs in 32 bit compat mode. Therefore, do we need a line like the following?

```
case "s390x":
		return []types.Arch{types.ArchS390, types.ArchS390X}
```


このクラスは不要なように見えます。
いつ作りましたか？テストを通すために最低限必要なコードのみを書くようにしてください。


Lol :-)


Ah ok, so I will remove this patch.


Did you mean 4593 and 4594?


you can use here !_.isNull() && !_.isUndefined() or even !_.isEmpty()


Yes, that will work for that case.


TODOコメントは、TODOに書かれた作業を終えたら削除してください。
この場合は、自動生成されたメソッドでテストが通ったらその時点で削除すべきです。


nie mieszaj frontendu i backendu


TeamCity OTA+ :: rvi_sota_server :: ci test [Build 258](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=3833&buildTypeId=ota_plus_RviSotaServer_CiTest) is now running


dupa debug


nie lepiej jakby te wartości były konfigurowalne?


@laughinghan This looks good, and behavior on Windows with the two screen readers I tried is identical to what it was previously. There is a chance that someone could be using Mathquill with a screen reader with the space punctuation enabled, and they might hear chopped off speech as the alert content quickly changes. I expect this to be an uncommon edge case that probably isn't worth worrying about unless we get complaints later on. If this fixes Safari, then I'm good with it.


needed?


There are a few things I still like to clarify.

First off there is only one variable named price and it is hard coded. Would this explain what hard coded price meant?

Second in case you did not yet link them. Logging on the child server was not requested and logging on the parent is already working so why are you reimplementing the things I already fixed. You are altering SSH commands in CreateFile.py without there being any need for it.

Lastly, the claim "We are not adding things to a class outside it's definition, we are effectively using a static setter. which is contained fully withing the class' definition. Furthermore i would highly suggest you never actually implement your suggestion of adding things to classes that you cannot alter. as this is quite literally against the extend, don't modify principle." is not true. A method will not become static simply because the name starts with static. The function SSH.static_use_logging() is not found within the SSH class. Furthermore it is not what you think it is. Such as it is not static. It contains no definition. And you define a function outside of its class. As stated before there is this useful typing feature that allows you to add (or overwrite) methods and interfaces to classes but that is only useful if the original class cannot be altered (because it is compiled or belongs to someone else for example). What that claim accused me of is exactly what you are doing. It is quite easy to see that altering a class outside of its definition when you can simply alter the definition is not clever. For instance the three previous little slips mentioned would likely have been noticed. In master ssh is logged per ssh instance, not per command. This is done on purpose because logging separate connections is a choice that needs to be made twice. To use the logging all that needs to be done is state that that separate connection needs to be logged. The decision to log or not should not be made in the SSH class. If you really want 1 variable arranging this then a static variable in birth chamber and assigning it thrice would be a better solution.


FWIW this should work ;)  (although I didn't try yet)


I would just set this to zero here and only override in the unusual case.


aa


BTW: Any news on getting these patches upstream to be included into mainline kernel?


I see, the new tally returned is a deep copy of the original.


mieszanie polskich i angielskich nazw? rly?


I have changed require to require_relative. Now, I think it would be fine.


Yeah, nice idea :) .
I will use jruby?


@azturner So would this help facilitate "donors" if the registration was something like a mission trip??


@mlocati, oh, my mistake! `unset($files[$file]);` may not lead to the desired result, this commit in 1.x-dev, but not in 1.0.0-alpha, I will fix it coming soon. thanks!


Hello,
I faced an error while trying to set _$pulp::manage_broker_ to false for some test:
`Could not find resource 'Service[rabbitmq-server]' for relationship on 'Service[pulp_celerybeat]'`
The same applies for Service[qpidd].

If one wants to use another puppet node as AMQP server, shouldn't he set _$pulp::manage_broker = false_? But in this case, the resource _Service[$broker_service]_ is not defined. And this commit made the resource required in any case.
Or maybe I just misunderstood something?


mam jedną bardziej ogólną uwagę głównie odnośnie tej mapy. Koncentrujesz się tutaj na tym, że będą tam województwa. Tak zawsze będzie? Nie zdarzy się tak, że zamiast województw będą rejony czy jakiś inny wynalazek? Wtedy będziesz musiał albo przerabiać, albo dorabiać nowy. Teraz czas zaoszczędzisz - później go stracisz


:shipit:


It's good


120446171-Mod-to-Warning-Less-than-6-Zone-Surface (RKStrand) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed, 0 test warnings)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg)


TeamCity OTA+ :: rvi_sota_server :: ci test [Build 258](http://teamcity.prod01.internal.advancedtelematic.com:8111/viewLog.html?buildId=3833&buildTypeId=ota_plus_RviSotaServer_CiTest) outcome was **SUCCESS**
Summary: Tests passed: 199, ignored: 3 Build time: 00:10:42


Sauf que "lors de le", ça fait quand même pas trop français sur le coup :D


dziwnie wyglądający hack


This is a dupe


jak tak dalej pójdzie to zmienne będą się nazywać VvvVvvVVVvvV :)
używaj nazw zmiennych, które coś mówią.


I would love to get this working, but when I tried it, I could not get even the basic:
  ` var test_pdf = { content: 'This is an sample PDF printed with pdfMake' };`
to work.
A new tab opens, but there is no page inside the tab. I first tried including pdfmake.min.js, then tried pdfmakejs, but still nothing.
Perhaps worth noting that everything is working fine if I include pdfmake.min.js from the main branch.
Could you think of any other dependencies I might be missing?

BTW, thank you so much for your efforts on this; I know I'm not the only person really looking forward to table row height and vertical cell alignment.


delete token on github :)


This is not a backwards-compatible change.
At least in PHP7.0.5, if you already have a subclass which overrides this method with the old signature,
this change causes a warning:
`Declaration of MyClass::editIssue($issue_key, $params) should be compatible with chobie\Jira\Api::editIssue($issue_key, array $params)`


Sorry, I missed it.
I have added
{
	"translate_tabs_to_spaces": true
}
setting to my editor.
I am not accustomed to emacs currently. Using sublime-text currently :(


writing main codes in game constructor


wtf?! w PHP jest coś takiego jak json_encode i korzystaj z tego. A co jeśli w nazwie rejonu będziesz miał podwójny cudzysłów? Rozwali Ci całego JSONa. To aż się prosi o buga


Yeah; Will make sure that these are not there in the new commits.


The Hour property returns 0 to 23, midnight would be zero - as it stands these emails will be sent at midday - but that may be OK for this use case...


Git ignore and git attribute files


Yes, they are there temporarily.


this should be `case "s390", "s390x"`


It's a manual process. The non-leader is the one that needs to be started in recovery mode, since it's the one taking over -- but if you start it in recovery mode it will forcefully take over the lock and become leader. There's no automatic workflow here. That's a risk you take when using this backend.

> Immediately after I kill the leader using kill -9 PID, and run GET /sys/leader, I get following

Load balancers need a while for their health checks to time out to switch over to a new instance, so this is expected. On AWS for instance it's often 10+ seconds.

> But after few seconds, no error is returned. In fact, an empty response is returned.

There's no leader, so there is no leader information to show.


This was just for random testing.


Ok I will do that.


commas for lists of case (or fallthrough).


Collaborator:
A review for this commit was created. [Link to review](http://127.0.0.1:8080/ui#review:id=2)


This is fine, but it would be much clearer with only the s390 unusual case changing the flags


Now I have moved to spec folder. The existing tests will be used.


was thinking maybe using composition could save maintenance/drift on supported properties?

Particularly with zipkin 1+ doing semantic versioning, this should be a safe idea.

ex.
```java
CassandraStorage storage = CassandraStorage.builder().contactPoints(foo).build();
collector = LocalSpanCollector.create(storage, metrics);
```


Boris, I think you need to add the removed case numbers otherwise the header will stop logging after this point (as default: condition will be triggered which ends the header).

e.g
case 19:
             break;
case 20:
              break;


I will be using the existing tests.


I will be using the existing tests. Now I am using the tests in specs directory.


That's subjective, considering JSON Schema was only a draft, and expired 3 years ago with no serious replacement. Implementing some unapproved draft is not a good idea.

If we're going to use JSON, it should be useful.

If ZFS had a null or no pool name, that is a bug in ZFS.


hehe, że niby próbował? :D


a jak będę chciał dodać wiele walidatorów?


co oznacza position 9 albo 4?


yeah this will still be unsafe to do because dereferencing a span consumer will cause i/o creating a session with cassandra. ex. when cassandra is down, that could throw. This is one of the reasons why we always queue and report on a different thread.


I am not happy with this inplementation. Every time I change one parameter
I need to rework the full list.

But please update it
Op 1 jun. 2016 14:38 schreef "MadmanK" <notifications@github.com>:

> Boris, I think you need to add the removed case numbers otherwise the
> header will stop logging after this point (as default: condition will be
> triggered which ends the header).
>
> e.g
> case 19:
> break;
> case 20:
> break;
>
> —
> You are receiving this because you authored the thread.
> Reply to this email directly, view it on GitHub
> <https://github.com/borisbstyle/betaflight/commit/d4c22f1c28e54420681362df6a90616932982d31#commitcomment-17695719>,
> or mute the thread
> <https://github.com/notifications/unsubscribe/AKQlhJ2GV0--INqkxn8ctsezn6vXNhPpks5qHX1VgaJpZM4IrgAE>
> .
>


the new FlushingSpanCollector moves stuff like this onto a different thread. This is used by LocalSpanCollector, which stitches in what's needed for metrics.


人凤飞飞


maybe `expect(rm.apiDocumentations).to.be.instanceof('Array)` would be more readable?


Repair broken event


I can modify the existing one because I think we cannot use those "mtg" icons here.


**[unitTests_php_branches run failed](http://jenkins.wikia-prod:8080/job/unitTests_php_branches/31562/testReport/) for this commit**


Collaborator:
A review for this commit was created. [Link to review](http://127.0.0.1:8686/ui#review:id=1)


Totally agree, the implementation is a little clunky!


howabout confluence? ;)


Yeah; adding a rake task for automated java tests is easy . I would love to write tests for c backend as well if time permits(maybe in the last week of June) :smile: .
Do you want to suggest any method for testing C backend?
Also, I just find a few people on sciruby irc channel. I would like to interact with the sciruby team and ask some doubts(though I can do it on ML). :smile:


Collaborator:
A review for this commit was created. [Link to review](http://127.0.0.1:8080/ui#review:id=3)


Collaborator:
A review for this commit was created. [Link to review](http://127.0.0.1:8080/ui#review:id=4)


Conflicten oplossen? Dit lijkt meer op de boel totaal verprutsen


It worked thanks but what is the reason behind it.


@snicolet, Many people have tried this in the past and failed. I'm curious to know:
1. Why you have not included king
2. Why you have placed this logic in king safety which won't kick in until ei.kingAttackersCount[Them] is non-zero.

I know many past tests have tried knight forks but didn't succeed. Good luck with your attempt.


I guess I am late. :P


I just tried downloading instead of opening in a tab and that works fine!

It seems it's only
`pdfMake.createPdf(doc_def).open();`
that isn't working.

I hope this helps.

Cheers


Collaborator:
A review for this commit was created. [Link to review](http://127.0.0.1:8686/ui#review:id=2)


good


Ok, this is also good, but I like the options.getStringOption("filename") better.
Something like this:
```
fullPath =  options.getStringOption("filename") + options.getStringOption("filepath");
```
```


@christoball I'm sure it's an easy fix, but can you change the $450 to $150 in the recap on Waldos talk please? I thought that was high. I was corrected yesterday. Thanks!


Yes, it's off-topic. https://github.com/gorhill/uBlock/blob/master/CONTRIBUTING.md
Firstly don't post clickable links. Secondly I don't even understand what you want. Disabling uBlock? Check the wiki then.


```
mkdosfs: Too many clusters for file system - try more sectors per cluster
```


When running a groovy script via the groovy maven plugin, the script has access to some special variables such as `project` and resolving folders is then best done relative to `project.basedir`. But I do not know if the verify scripts also have these special variables.


Hm, I'm using just `pdfMake.createPdf(doc).open()` and it works fine. Maybe browser blocks opening a tab?


"stuff" is uncountable.


@Think-Smart
we need little help in app.js of controller, the logic in API work (we tested)


Arf, my merging was really shitty... Sorry.
As I said [here](https://github.com/CGRU/cgru/pull/239#issuecomment-222151036), I had done that. I will try to double check everything then.


Hahaha, bien vu :)


Ouch...does `__attribute__ ((alias))` work here?


Change supplied by spronin


I think to avoid messing up dependencies when we obfuscate, you need to add 'ngInject' : https://github.com/medic/medic-webapp/blob/a58d8026119410f633c1ba269b80b21237e4d826/static/js/controllers/configuration.js#L14

I haven't actually tested it though, just copying from previous examples.


you could also edit the Arma Version to 1.60


Release 7.2.0.0


120446171-Mod-to-Warning-Less-than-6-Zone-Surface (RKStrand) - x86_64-MacOS-10.9-clang: OK (2204 of 2204 tests passed, 12 test warnings)

 * 11 tests had: ERR diffs.

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


I would put here the link:

~~~
www.isir.upmc.fr/files/2009THDR2323.pdf
~~~


I guess we should choose either array or object approach here :)

```
PHP Warning: Attempt to assign property of non-object in /usr/wikia/slot1/11760/src/extensions/wikia/CreateNewWiki/tasks/StartPostCreationTasks.php on line 25
```

is reported by PHP7


TaskHero


120446171-Mod-to-Warning-Less-than-6-Zone-Surface (RKStrand) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (1094 of 1094 tests passed, 1 test warnings)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg) ![Coverage Badge](http://img.shields.io/badge/coverage%20status-33.92%25-green.svg)


Glad this was looked at. I do not often use encoding, but had known that pipe (stdin??) had problems when it got to encoding.  As infrequent as I use encoding, I simply put to a file, but this helps, I think.

One other thing about stdin/pipe.  Should we remove the !rules restriction from -stdin??  Yes, it makes resuming fail ('can' be worked around sort of). The same resume issue is on pipe. The only problem with stdin / pipe is that stdin reads 1 line at a time, and that makes -rules less efficient (at least on many OS's). Pipe solved most of that issue. BUT not allowing stdin+rules just does not seem logical to me, unless I am missing something.  Also it would be a super easy 'fix', just remove a flag I think.


Collaborator:
A review for this commit was created. [Link to review](http://127.0.0.1:8686/ui#review:id=4)


Here's a sample you can use for (most? of) the features I added.

```
  content: [
    {
      layout: GrayLinesLayout(),
      table: {
          widths: [160, 100],
          height: (i) => ((i + 10) * 9),
        body: [ [
          { layers:
            [{ text: 'Layer1 center', verticalAlign: 'center' },
             { text: '2nd Layer top', verticalAlign: 'top' },
             { text: 'layer third bottom', verticalAlign: 'bottom' }
            ]
          },
          { text: '1A2B3C4D5', font: 'Overlays'}
        ],[
          { layers: [
              { stack:
                [{ text: 'AB', alignment: 'left'},
                 { text: 'AB0', alignment: 'right'},
                 { text: 'AB1', alignment: 'right'},
                 { text: 'AB2', alignment: 'right'},
                 { text: 'AB3', alignment: 'right'},
                 { text: 'AB4', alignment: 'right'},
                 { text: 'AB5', alignment: 'right'},
                 { text: 'AB6', alignment: 'right'},
                 { text: 'AB7', alignment: 'right'}],
                verticalAlign: 'center'
              },
              { text: 'A', alignment: 'right', verticalAlign: 'center', font: 'Overlays', margin: [0, -10, 0, 0] },
              { text: 'B', alignment: 'right', font: 'Overlays' },
              { text: 'C', alignment: 'left', font: 'Overlays' },
              { text: 'D', alignment: 'left', verticalAlign: 'center', font: 'Overlays' }
            ],
            pattern: ['stripe45u', 'black'],
            fillOpacity: 0.45
          },
          { text: 'one1here1one2here2one3here3',
            noWrap: true,
            alignment: 'left',
            verticalAlign: 'bottom',
            pattern: ['stripe45d', 'black'],
            fillOpacity: 0.45
          }
        ] ]
      }
    }
  ]
```


If this pass great. If not, this will be an opportunity to learn the tuning job.