Hi. Thanks for checking. Most examples print the size used and it's 95 bytes for spp_counter and spp_and_le_counter.

I'm not happy with this hack and plan to finally first clean up the SDP Server API and then pass in the SDP record again (without the struct at the beginning). The need for alignment will disappear then.


@vasim89  We need critic rating - please scrape it and store it in the db.


Lint might complain that you'll need positional holders here which could be nice (e.g. `%1$s` and `%2$d`)


Модуль норм. А как уйти от копипасты? =)
```javascript
resp.status.match(request.okStatus)
```


TIL this is a public repo.


:-) Welcome to join GVRf's community.


I think this could be simplified a bit to

```ruby
supported_os = [ "openSUSE 13.2", "SLES 12"]

os_release_file = "/etc/os-release"
os_release = File.read(os_release_file) if File.exists?(os_release_file)

if os_release
  version = os_release[/VERSION_ID="(.*)"/, 1]
  distribution = os_release[/NAME=(.*)/, 1]
end

if !os_release || !supported_os.include?("#{distribution} #{version}")
  log "Warning: Pennyworth is not tested upstream on this platform.  Use at your own risk."
end
```

And what about having a test for this method?


try this:

http://gredler.at/download/traces/aigp.pcap

On 5/22/15 9:24, fxlb wrote:
> @hannesgredler <https://github.com/hannesgredler> To put this on master
> branch, we would like a pcap test file.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/the-tcpdump-group/tcpdump/commit/94e23e3b7b7b6d9fcab9565b65613fd1bd47f018#commitcomment-11322056>.
>


13.1 is not part of the list of supported operating systems in the other commit, this has to be consistent.


Nevermind, I just saw that you removed it in a later commit.


This is looking OK but we still need docs on what every group installs.


See #2


Line 2875 is missing the first part, 120300113.
120300114 only covers "…おじさんにわからないことはないからね…。", that's why I joined both audios.


You mean it did not work at all anyways?


The conversion to a byte array is needed since T is not necessarily serializable. Even if we are using Kryo serialization, we want to use the coder since it's more in line with the dataflow model. I added a comment in the code to explain this.


Проветряването е ОК, въпросът е да не се стигне до счупени прозорци от вятър и локви от дъжд в лаба ;)


I think scipy is an optional dependency, so the imports can't be at module level?

I think this (sp.) isn't even used here, but it might trip up people without scipy.


Ah, indeed. This does work. I'm updating the commits.


Why should the cache type be different for primitive types and for object types? With memo-modulo reference, shouldn't `LongMap` be sufficient for all types?  One can look up primitive types by `_.hashCode` and object types by `System.identityHashCode`. To distinguish primitive types and object types, we can use a series of overloaded `getKey` methods in some library object:
```scala
def getKey(i: Int): Long = i.hashCode
def getKey(b: Boolean): Long = b.hashCode
def getKey(j: AnyRef): Long = System.identityHashCode(j)
```


OK.


Please report the bug, so we can fix it, I don't see it right now.

About subtitle downloading from youtube, we can not leave this responsability to youtube-dl since not many of the videos have the subtitles in youtube. For that reason it is better to download the subtitles from the edx platform too.


Your solution is not working if Thelia is installed in a subdirectory.

You have to use `{url path="/ajax/addCartMessage"}` to get a proper URL. As thelia.js is not parsed by Smarty, yous should find a way to pass the URL to the function. Maybe as a parameter of the form ?


Why do you initialise `config` here?


Added Call Utils to the list of apps that use FAB.


Otherwise it doesn't work :D


http://hastebin.com/bafomovawa.js

@HashBang173 Help?


does the $rootScope.$broadcast work if there is no $rootScope reference in the controller declaration?


What doesn't work?


Actually no you are right, it is stupid. The failure I had was because you were reading from "config" when the module is imported, which is too early because it is not yet created.


Thanks! One last thing; can you add one more test where you don't set `on_empty_404=True` and assert that the response code is *not* `404`? It's good for completeness for testing all paths.


:moneybag:


it's a tiny one though. I can have tons of RAM!!


@allgaeu Ich schreibe gerade meine Präsentation, mit Notizen. und überlege mir nebenbei das Bild, in ein paar Stunden schicke ich dir eine Idee, die auch zu meiner Erklärung passt die ich gerade erstelle.
Aber mach dir kein Stress damit, ich würde es auch selbst machen habe allerdings gerade nicht so viel lust mich in tickz einzuarbeiten, würde es in inkscape machen wenn du sagst dass dir das nichts ausmachen würde in inkscape^^


no


:+1: whats your nickname on PyPI? same as here?


I prefer to keep the full url since it gives more information to someone who is debugging in the other functions, even if youtube-dl can download just with the ids. Additionally since the argument is called url I expect to have a full url there, so I think it is good to keep it.


Grundsätzlich richtig! Problem ist, dass ```i``` nicht bekannt ist. Der TurnTable muss noch wissen, welches Fach gerade aufgeschlossen wird. Und evtl. müssen wir bei jedem drehen der Machine eine current_position and den TurnTable geben. Damit der weiss, welche ```section[xyz]``` geöffnet werden soll.


Y


Agree 100% this debug mode will really help us with both bug reports and clarity.


PlaceholderX.verify() was itself verified with the following definition:

    image bg cafeteria9 = PlaceholderX("drops/eggs.png", "drops/cafe1.png", pretext = "(PLACEHOLDER) ")


+1 for making the parents context explicit with a class


`$(function SelectRobot() {` heeft geen toegevoegde waarde, kan wel weg. Zelfde geldt voor `Control` en `Logout`.


Also you could write as `it 'creates lvm partitions'` (I think it's more RSpec-like but it's just a matter of taste).


@yehudasa commit message updated. Pull request available here:

https://github.com/ceph/ceph/pull/4745

Thanks Yehuda.


:+1: thank you!


FYI, maybe it's an error on my side, but `msm8974pro-max77804k-klte_chn_cmcc_r07.dtsi` is referenced above, but not included in the commit. This breaks the build for me.


Indeed you are right, I was just considering the current implementation...where everything passes by ```require()``` and ```alreadyRequiredNamespaces``` is not exposed (or I am reading the code in a wrong way) . But if you are planning something like you said, indeed you need a ```ConcurrentHashMap```...


Maybe we could check that those sections are present in the `ntp.conf` file. Or is that out of the scope of this test?


It felt right to put this right after MAPCACHE_CACHE_MISS in the list. But I see your point and I can change this :)


Need's to changed to:
```ruby
permit_params :name, relations_attributes: [:user_id, :id, :_destroy]
```


Can we have this as a parameter? The application logic like the URL scheme should not be defined by the style guide.


Tips,
1. For clean code.
If not required,
   Pull request should not contain commented code. It may leads to unnecessary confusion.

1.To increase performance .
If after_rename method, you can get customer and its abbr in single query. with get_value method you are initiated two database call.


@unho there was a whole discussion about this file.  It's generated, it shouldn't be in our repo.


yup :)


后续考虑升级Lua5.3或者云风的sproto。


Accidently added those brackets at line 11, fixed here -
https://github.com/netguru/review/commit/a27ba277956db9c0934e8f931a3a56495223e4a7#toc


Wrong indentation. There are similar problems, so I won't comment every line. Check everything please.


`@var string`


Greets. Mind if I pull wx_na_do?


要不要加一些内容


pypi access granted


Hehe, if you started with this, then you're on the good way! :smile: :+1:


Please use camelCase in variable names -> `$syntaxRegexp`. Again, check everything.

No such class `LiquidRegexp`.

No such constant `LIQUID_ALLOWED_VARIABLE_CHARS`.


No such property '_type'. Please, do not use '_' in variable names.


@d4gg4d no we actually shouldn't. We still need to store custom radiator configuration somewhere (localStorage will do for now) so we are able to resolve them when the page loads.
Module configs allow us to use one graph _type_ and load it with different data / metric.


Ahhh ok works fine now :) Thanks mate. Restarted fixed.


Бих предпочел да имаме процедура сутрин да отваряме за проветряване докато е свежо, защото иначе наистина рискуваме някоя буря да направи мизерии, или някое цигане да хвърли някой боклик вътре. Нашио лаб - нашио замик.


Hi bejond,
Could you please resent your pull request? we moved api to osgi module


Thanks so much for this Rob! I'll try to test this out properly asap.


Syntax error: double `{`.


by *their* resource


Multiply-Add => seems the character is encoded in a bad way :-(.. no worry


Ok, this description was a little bit misleading for me.


LGTM


Please place the opening `{` on the same line. It is places on the new line only for classes definitions, for complex if statements and complex method definitions.


This test adds 1 sec to the build and only checks that the matcher properly does its job of ignoring the commented out header. This is akin to check that:

    assertTrue(matchesPropertiesOutput(new Properties()).matches("#a comment\n"));

Isn't it ?


Use type hint `Context`, no passing by reference.


It appears the "this" variable is not pointing to the correct DOM element. Navigating the DOM can be quite a process in any javascript library. I'm not sure how long it'll take me to find it using javascript, but I might be able to find it faster using jquery. It shouldn't require rewriting all the functions, but I'll try to write a jquery fix after EM service.


Awesome; you are welcome. :)


ok i will improve writing css patteren.


Short syntax arrays are available only from PHP 5.4, but the library must work with PHP 5.3+.


Must be `TagIfChanged`


Also, could you please add me as a contributor to the repo? GBY!


Unused. Please check everywhere.


`default` is a reserved word in PHP.


camelCase


It's a quick and dirty un-kebabCase-er :) Like I said. Needs some tidying.


You might try to use an absolute path as we do in resources.js for: var API_PATH = '/bonita/API/';


You cannot change the behavior of the filter, because there are users who rely on it. Create a new filter instead.


pp may be useless for modern HD video, but there are a lot of videos out there that are *not* modern HD videos (e.g. lower-resolution MPEG 4 ASP) for which the deringing and deblocking parts of vf_pp are very useful. Since pp also now seems to have been removed from libavfilter, I will have to stick to using mplayer to view this videos, which is a pity because mpv is otherwise a great improvement.


In case of a filename that contains format strings this was an - probably exploitable - format string vulnerability. Fixed with commit bce566a.


Ok - thats fine - but surely the point is to try and match the functionality of the ruby version of Liquid, which this change does?


I agree, I’ll change it.


时而使用tab缩进，时而使用空格缩进，格式混乱。


what do you mean? for me it looks ok...


Yes, this is the list of improved objects, except the NoseWheelStrut

- Left and Right wheel struts: they are more rounded now.
- NoseWheelStrut: dg-505 proposed a version with significantly more vertices, but I felt our old version feels right once textured with smooth surfaces. I preferred not merging an object with many more vertices because I was worried about the performance hit. This was just my opinion and I can be wrong, of course.
- TopLink and BottomLink are significantly improved and much more detailed now.
- Exhaust: I commited the object proposed by dg-505, but I must confess I didn't notice any difference with the old object.


* StringBufferInputStream is deprecated


Прозорецът в посока съседите в лекционната гледа директно към къщата и е под стряхата, може да се подпре и да очакваме, че от него няма да вали. Мисля, че същото важи за прозореца в коридора при стълбите (горе).


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


И на кенефа, и на кенефа!

2015-05-22 11:10 GMT+03:00 Vasil Kolev <notifications@github.com>:

> Прозорецът в посока съседите в лекционната гледа директно към къщата и е
> под стряхата, може да се подпре и да очакваме, че от него няма да вали.
> Мисля, че същото важи за прозореца в коридора при стълбите (горе).
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/initLab/initLab/commit/8ba61673063c1df4c2269ddbd997b722d67164b6#commitcomment-11322676>
> .
>


dsadsadsafa


Thanks, @anuchandy!


The logic here is hard to understand. I think the `&&` should actually be `||`. Does that sound correct?


This is not without complexity.  While having a _smaller_ cutoff size would work in a straightforward way, administrators may also expect that they can raise the cutoff and have data moved _back_ into the database -- which would require additional code.

I'm not saying that `--all` is a bad idea, merely that it needs additional code or additional careful documentation to cover expected use cases.


could use `-z` to check the variable?  i see
```python
if not ret and 'CEPH_CLI_TEST_DUP_COMMAND' in os.environ:
```
in `ceph.in`


Consider adding 2 helper methods that do String <-> Properties conversion to clean the code a bit

    static Properties unmarshallProperties(String s);
    static String marshallProperties(Properties s);

Both would make the proper use of UTF-8 (and avoid StringBufferInputStream)


Create PR then, I'll merge :)

On Fri, 22 May 2015 at 11:59 Maxim <notifications@github.com> wrote:

> Yes :-)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mpyatishev/django_audit_trail/commit/ba6b9f1e74d12c7de939c42c45407c0617cfb07b#commitcomment-11321786>
> .
>


>Since pp also now seems to have been removed from libavfilter,

It's still there. Maybe libavfilter was compiled without libpostproc or so. Or it's a Libav libavfilter.


@Dexberry There are plans for ACL in Laravel?! Did I miss some news? :) @GrahamCampbell @taylorotwell


Versione funzionante con cat e read


> I assume `git branch -r | fzf` does work for you, right?

Yep.

Just updated fzf to your version and still have the same issue. Weird...


这里是留言
# 标题
![](https://www.baidu.com/img/bd_logo1.png)


这错了。。。
]![bd_logo1](https://cloud.githubusercontent.com/assets/6385380/7766645/edb714ee-009d-11e5-809d-fd7f95b7e943.png)


@llxff Да, Secret. Никому не рассказывай


You're are probably be right, I didn't fully thought this, when I wrote it. Even so, this store/read actions should be isolated to `configuration-store.js` like we do with the custom radiator configurations.


Поправил 631bad2ee2f89b15231eec39d9759791f5e6e752


Already fixed that one in a new version. Going to push it today. Thanks!


Jeg er klar på en enkelt øl efter arbejde i dag, men skal gå ret hurtigt ;)


:peach: :open_hands:


And where do you want to add them? They are implemented all over ejabberd source code files, not in a single file.


Certainly an error of my web browser. does not matter...


`-Og` for debug builds


We're currently using GCC 4.8 because the builds produced by GCC 4.9 don't boot on some targets (for a as-yet undiagnosed reason). Also GCC 4.9's codesize is quite a bit larger.


Средненько.


If it is referring to remembering the computer, then maybe stick to that?

"The computer is now remembered and does not require Two Factor Authentication in the future"
"Forgotten about this computer. Two Factor Authentication will be required on future logins."

Something like that?


thanks akash i try to improve my css instead of % , it used to easy to handle responsive design.


@DavidePrincipi : want me to merge this one to master?


I think you misunderstand (or I'm not communicating clearly, or both -- probably both). I'm saying that origin-clean could be overloaded by an extension (e.g., always forcing it to 'false') to make a canvas unreadable, thereby avoiding fingerprinting exposure. While origin-clean isn't designed for this, it's neat that it could be used in that way.

Will try to clarify.


Indeed.


This commit is (might be?) the greatest contribution the open source world has benefited in a very long time. And I don't mince my words.


Thanks Rob for the fix. I think the code change is okay for my issue. How can I update my plugin for the fix?


see issue #284


# EVERY GITHUB REPO EVER


productionのtokenさえ漏れていなければsecets.ymlはアップロードしても良いはずなので


Can we get the AMI ID out of the code and in the yaml file?


see issue #285


WTF??


Hey, what was this bug about. Can you put reference to bug report please?


Possibly `typeof SVGFEColorMatrixElement !== 'undefined'`?


It was mostly a test to see whether the GitHub mirroring worked. You know, the GitHub mirroring you asked for.


Точно за споменатите от Крокодила прозорци имам забележки, които не бих искал да коментирам тук.


If you don't want to put absolute paths in source files you could use resource loading - http://www.mkyong.com/java/java-read-a-file-from-resources-folder/, though I'm not sure if you want to keep these files in the source


abc


Move the onclick behavior into the link.


The draft is a good starting point. Covers all the normal terms and conditions. Good work


URL bad, href better!


agentkeepalive seems to need version 0.11: https://github.com/node-modules/agentkeepalive


:+1:
Let alleen ff op consistentie met single-quotes `'` en double-quotes `"` in JavaScript. Liever alles met single-quotes.


This change should be done in the Controller.

Instead document.location can we use $location.path?


Put that in the PaginationLink too


aaaaa


@Rbn3D @wesnick

I'll remove this dependency in next sandbox update.


of cause


Right, I've fixed, thanks!


有什么讲究？一致性？


@behlendorf @ryao @dun what do you guys think of adding something like:

```
# In case not shutdown cleanly.
[ -n "$init" ] && rm -f /etc/dfs/sharetab
```

here (before doing the import)? I've noticed the last couple of days that if (when!) the system crashes, this file will exist and be populated when the machine starts up again. So I think it makes somewhat sense to remove this in the boot up (hence the check for <code>$init</code>).


What's the purpose of this union? `buf` isn't used anywhere.


Very minor: I believe it is "iOS".


¿Y el código dónde está? :stuck_out_tongue_winking_eye:

Si no recuerdo mal, esto requería algunas modificaciones, en lugar de estar implementado como un módulo, ¿no? En tal caso, ¿no sería mejor un pull request al repositorio de SimpleSAMLphp?


Hey it's actually not working for me lol


I didn't add that, it was added in https://github.com/uNiversaI/kodi/commit/5fd5072d1f327a82045d97c77445f4749da41c3e
and I think @fernetmenta cares as much about the wording of this stuff as I do ;)


I don't know how to make it work, would you mind helping?


Though it increases indenting, here I'd prefer to see a proper `if MOD/else` structure. It's easy to miss the `return` statement here, which leads to confusion below.


Ah, damn, you are right.
Probably happened when i was pasting the regex.


Should `/inst/doc` also be added to `.Rbuildignore`? Adding only to `.gitignore` gives surprising results when `build_vignettes` has been run accidentally. See also the documentation of `build_vignette` [(code ref)](https://github.com/hadley/devtools/blob/d2bdbada3f8921978ab13dbcad5897a60cf5f965/R/vignettes.r#L7).

(Better use `"/inst/doc"` instead of `"inst/doc"`, too.)


This request is not good,please more better!


You have to use your email & name in the commits and push the repository into your account. What are you trying to do?


В кой-то веки)


потому что вы позволяете (случайно или не случайно) создать именованный объект с пустым именем


While this is correct, main calling Run() is not.
main is only looking for one response not 2. The first one to respond will exit gorouter regardless if the second fails.
Its a separate issue but this made it more obvious.


You can't use the end argument here, it will break on Python 2. Can you rephrase to just do ``print(some_string)`` or ``print(some_string % (var1, var2, var3))`` instead?

Probably the simplest solution is to print these as separate lines (rather than forcing one single line of output for each ``SeqRecord``?


This is what I did step my step (I have a mac btw):

I generated the calendar on the server, and after unzipping the zip file, I simply did git remote add origin (URL of the empty repo I created) and did git push origin master with the -u tag, but it didn't really generate the calendar.

Where does the using my email and name in the commits part come in? (Sorry I'm new to github lol)


我想，我接下来这样做：
1. 先把项目skip一次；
2. 站在项目整体的角度，找出其中的认为可能不尽完善的地方，避免历史第三次重演；
3. 找出能够做得更好的方式并实现和测试，但水平有限，可能还是会存在欠缺；
4. 你来校对并提出更多指导。

ps:由于个人时间分配有限，同时我也需要在阅读source code的过程中学习并消化这些算法，并把这些运用到我自己的项目中来，所以进度可能会比较慢。


Excellent additions Vincent, I'll be pulling this in pronto...


ок


The C specification allows paths non `\0`-terminated. `buf` makes sure that there will be enough memory to accommodate an extra `\0` after the `sockaddr_un` struct.


Do we document error codes in the API for receptor?


Not convinced this is the best approach for nsync. It should be smarter rather than making CC smarter.


why did you change the colours :disappointed: ?


именованная сущность даёт по требованию своё имя, вы вторгаетесь в этот процесс и добавляете перевод строки, это неожиданно с точки зрения того смысла, который обрисован именами


It'd be good to have a note here as to why you rejected `fmod` as this is quite heavy-weight.


In the same repository do:

```sh
$ git remote remove origin
$ git remote add origin https://github.com/beautyqueen18/MyRepository
$ git push --all
```

On push, it will ask you to provide the GitHub credentials. The problem here is that you have two accounts on GitHub. You pushed on this account (`shashmi1`) the commits using the @beautyqueen18's email.


Impressive guys. I like your working rate. Login and registration screens are looking good.


License header year should be the year this file was added IMO.


You could do the same with an anonymous struct, and your explanation should be in a comment in the code. But I think my other concerns are more important.


Great plugin! Thanks very much for it.
One small problem with this change is that you cannot start numbers with "0" this way. That is, typing numbers like 0.34, 0.1, 0.9999 are kind of impossible to do. (This also occurs on the 'Demo as you type' page.)


Given an array of integers, find two numbers such that they add up to a specific target number.

The function twoSum should return indices of the two numbers such that they add up to the target, where index1 must be less than index2. Please note that your returned answers (both index1 and index2) are not zero-based.

You may assume that each input would have exactly one solution.

Input: numbers={2, 7, 11, 15}, target=9
Output: index1=1, index2=2


я про енам не спрашивал :)
не понятна непоследовательность: Circle("Some Circle name", rand(), rand(), rand()); -- здесь вы не делали никаких отдельных переменных. Я хочу понять, почему в аналогичных местах сделано по-разному. Где логика?


I think this one should go to decorator/helper. Keep views as simple as possible. I.e:

    app/helpers/articles_helper.rb

    def author(article)
        article.user.email.to_s.split("@")[0].titleize
    end

    app/views/articles/index.html.erb

    <%= author article.user %>

Otherwise, you can specify `to_s` method in your user model and use just `article.user` then, without any helper method.


почему похожие вещи сделаны по-раному?


вы же выше перебрали все возможные значения. нет?


same as above - when you have to repeat code, this means that these part of code should be extracted to something else.


net-config belongs to other commit


Please change the name of the prototype in  libnemo-private/nemo-progress-info.h as well.


переменное количество аргументов -- опасная штука (= распространённый источник ошибок), её не нужно использовать без ОСОБОЙ необходимости


use helper/to_s


:beers: In general it would be best to add such information to https://github.com/NixOS/nixpkgs/pull/7237, but I'm fine if it's also in README. I'm happy if we juts have it :)


Yes.


What if context is null because an older client (in case of rolling upgrade) call this function a null context?


Yeah, now it is saying "Permission to beautyqueen18/MyRepository.git denied to shashmi1.
fatal: unable to access 'https://github.com/beautyqueen18/MyRepository/': The requested URL returned error: 403".

How do I change the configuration settings so that I commit through the e-mail linked to this account (since I used a different e-mail to sign up)?


I currently don't have time to test it, but what's with the patchset.txt?


And sorry for bothering! thanks again


hand the full path of the xml file to vm-start


дефолтный параметр -- это обычно плохо


куча статиков, по одному на каждый класс конкретной фигуры


thanks @fmbiete
1bbcbf42d67dc64e2d6668233bf70377fda098f6


обсудим :)


You are given two linked lists representing two non-negative numbers. The digits are stored in reverse order and each of their nodes contain a single digit. Add the two numbers and return it as a linked list.

Input: (2 -> 4 -> 3) + (5 -> 6 -> 4)
Output: 7 -> 0 -> 8


Ideally it should be: default = "unlimited", and no max value.


Possibly/probably but having it defaulted like the VPC config is might be nice

In this PR i was trying to keep it functional identical just replacing the JSON with Troposphere to make it easier to review.


I think this was already fixed by d52d3ca580399f3a2bef58312388892d0384c40f


You are climbing a stair case. It takes n steps to reach to the top.

Each time you can either climb 1 or 2 steps. In how many distinct ways can you climb to the top?


ok, then I'm fine with that :-)


With an anonymous struct can allocate some unnecessary padding. I need just one byte more.


"Initial photon gathering radius, in world units"


Given an integer, convert it to a roman numeral.

Input is guaranteed to be within the range from 1 to 3999.


correct :)


This actually needs further correction -- we don't specify what divisor and remainder must be -- positive int/double etc. I think you only validate for NSNumber, so either, which means we need a `number` type.

On that note, you probably need to test for non-integer types, if we're going to support them. I imagine your equality might fail in some cases due to floating point precision. Possibly `1 % 5 = 0.2` thought that might be okay. It's hard to figure the values which might cause the problems, and I'm not sure NSNumber's floating point accuracy. I chose 0.2 as http://stackoverflow.com/a/588014/290514 claimed it was not representable in 64-bit floating point.

Using `fmod` and doubles in the mod operator implementation would make this easier to reason about, I guess, one argument for that.


License header year should be corrected as
`Copyright (c) 2015, WSO2 Inc. (http://www.wso2.org) All Rights Reserved.`


Given a binary tree, find its maximum depth.

The maximum depth is the number of nodes along the longest path from the root node down to the farthest leaf node.


You can avoid padding by adding `align(1):` before the first struct member.


Hi,
You can de-dupe "Jonathan Stowe" and "RabidGravy" as I am both of those ;-)


Determine whether an integer is a palindrome. Do this without extra space.

Some hints:
Could negative integers be palindromes? (ie, -1)

If you are thinking of converting the integer to string, note the restriction of using extra space.

You could also try reversing an integer. However, if you have solved the problem "Reverse Integer", you know that the reversed integer might overflow. How would you handle such case?

There is a more generic way of solving this problem.


Currently you're using `sahashmi <sahashmi@uwaterloo.ca>`. If this is the correct email, [add it to this GitHub account](https://github.com/settings/emails).

If you want to change the commit email you have to change it in `gitconfig` (edit the `~/.gitconfig` file).


Remove all elements from a linked list of integers that have value val.

Example
Given: 1 --> 2 --> 6 --> 3 --> 4 --> 5 --> 6, val = 6
Return: 1 --> 2 --> 3 --> 4 --> 5


"Evolution rate of photon gathering radius"


define in config


Reverse a singly linked list.

Hint:
A linked list can be reversed either iteratively or recursively. Could you implement both?


ok, thanks for contribution.


Ah, I didn't realise that the libav source used by the Ubuntu packages wasn't the original FFMPEG. The pp filter does indeed seem to be present in mainline FFMPEG. Thanks.


I think I changed the settings correctly now. Do you think it is correct now? (Correct email : shashmi.inquiries@gmail.com)?


identityHashCode isn't guaranteed to be injective (AFAIK), so the equality
would be wrong.

What's wrong is that I'm not using yet IdentityHashMap, but I'll fix that.
IdentityHashMap will do something similar to what you describe, but will
still use Java's == on objects.

We do need some casting to Long for primitives, but it must be injective.
--
Paolo Giarrusso
On May 22, 2015 09:44, "Cai Yufei" <notifications@github.com> wrote:

> Why should the cache type be different for primitive types and for object
> types? With memo-modulo reference, shouldn't LongMap be sufficient for
> all types? One can look up primitive types by _.hashCode and object types
> by System.identityHashCode. To distinguish primitive types and object
> types, we can use a series of overloaded getKey methods in some library
> object:
>
> def getKey(i: Int): Long = i.hashCodedef getKey(b: Boolean): Long = b.hashCodedef getKey(j: AnyRef): Long = System.identityHashCode(j)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/inc-lc/ilc-scala/commit/6a4eae906f817b042385f64a2c10c1c386c05254#commitcomment-11322312>
> .
>


Мне показалось это логичным. Можно отнаследовать полигон от полилинии и сделать разных конструкторов на все случаи жизни


Reverse digits of an integer.

Example1: x = 123, return 321
Example2: x = -123, return -321

Have you thought about this?
Here are some good questions to ask before coding. Bonus points for you if you have already thought through this!

If the integer's last digit is 0, what should the output be? ie, cases such as 10, 100.

Did you notice that the reversed integer might overflow? Assume the input is a 32-bit integer, then the reverse of 1000000003 overflows. How should you handle such cases?

For the purpose of this problem, assume that your function returns 0 when the reversed integer overflows.


Reverse digits of an integer.
Example1: x = 123, return 321
Example2: x = -123, return -321

Have you thought about this?
Here are some good questions to ask before coding. Bonus points for you if you have already thought through this!

If the integer's last digit is 0, what should the output be? ie, cases such as 10, 100.

Did you notice that the reversed integer might overflow? Assume the input is a 32-bit integer, then the reverse of 1000000003 overflows. How should you handle such cases?

For the purpose of this problem, assume that your function returns 0 when the reversed integer overflows.


Implement atoi to convert a string to an integer.

Hint: Carefully consider all possible input cases. If you want a challenge, please do not see below and ask yourself what are the possible input cases.

Notes: It is intended for this problem to be specified vaguely (ie, no given input specs). You are responsible to gather all the input requirements up front.


В данном случае он очень полезен


Requirements for atoi:
The function first discards as many whitespace characters as necessary until the first non-whitespace character is found. Then, starting from this character, takes an optional initial plus or minus sign followed by as many numerical digits as possible, and interprets them as a numerical value.

The string can contain additional characters after those that form the integral number, which are ignored and have no effect on the behavior of this function.

If the first sequence of non-whitespace characters in str is not a valid integral number, or if no such sequence exists because either str is empty or it contains only whitespace characters, no conversion is performed.

If no valid conversion could be performed, a zero value is returned. If the correct value is out of the range of representable values, INT_MAX (2147483647) or INT_MIN (-2147483648) is returned.


I am sorry, if you are really busy, but perhaps we can screenshare for just a few minutes to try to resolve this? I would honestly, honestly really appreciate it!


Method comment?


Given an array of integers, find two numbers such that they add up to a specific target number.

The function twoSum should return indices of the two numbers such that they add up to the target, where index1 must be less than index2. Please note that your returned answers (both index1 and index2) are not zero-based.

You may assume that each input would have exactly one solution.

Input: numbers={2, 7, 11, 15}, target=9
Output: index1=1, index2=2


These are good use-cases for "auto iter = ...", IMHO.


@shashmi1 You're still using the [old email in commits](https://github.com/shashmi1/ThisIsNew/commit/54d6e31df6e29f79c7315df30e80df0bc813ae2a.patch).

You can [share your terminal](http://share-term.me/).


多了一行不需要的


В свою защиту могу сказать, что у листа вообще не должно быть метода взятия по индексу


Bakal bermasalah suatu saat, coba cari alternatif lain.


Access = FALSE untuk node add, tidak untuk node edit


vector<int> ls;


Before pushing I saw d52d3ca and I was hoping that it would fix this, then I checked that #30060 was still occurring.


fdsafasfa


HAHAHAHAHHAHAHAH :dancer: :dancer:  :dancers:


It's just a file containing all changes to Assembly-UnityScript.dll so I don't forget anything after a Besiege update.


a


We need to delete all favorites for mobiledemo


@struart Would you mind translating these new strings again? I'm planning a release for Helix soon. Thanks!


may be wrong!


fdsa


Oh, that's good. I don't see anywhere a method that gets the gameobject variable?


I'm having trouble getting that to install. Can we do a screenshare if you don't mind?


@shashmi1 What errors do you get?


Change the RHBug number according to this https://github.com/rpm-software-management/dnf/wiki/Become-a-contributor#implement-dnf-repoquery---tree-switch-that-takes-no-param---tree--functionality-from-repoquery-1 (I have fixed the link) and make it work with `--requires` - thats what the original bug reporter asked for.


Syeeds-MacBook-Pro:source shasmi$ npm install -g share-term
node.js:720
    var cwd = process.cwd();
                      ^
Error: ENOENT, no such file or directory
    at Error (native)
    at Function.startup.resolveArgv0 (node.js:720:23)
    at startup (node.js:63:13)
    at node.js:814:3


GOOD!
maybe prefix by am-


@eventhorizonpl I've already removed it

https://github.com/surexdirect/backend/pull/771


Remove Extra space


Most probably you are in a directory which was deleted. Can you do `cd && npm install -g share-term`?


ok


remove commented out code


"to set" => "to set the"


Hm, ok, but the posts in that thread had children with item_private=0 and were gone from my zotfeed once I set item_private=1 for those children, which by my following of the code was what was making them visible. I never touched ITEM_WALL.


TeamCity Selenium POM :: CI [Build 12](https://teamcity.sponte.uk/viewLog.html?buildId=74&buildTypeId=SeleniumPom_Ci) is now running


Really need that endTime watcher ...


remove space


TeamCity Selenium POM :: CI [Build 12](https://teamcity.sponte.uk/viewLog.html?buildId=74&buildTypeId=SeleniumPom_Ci) outcome was **SUCCESS**
Summary: Tests passed: 34 Build time: 00:00:35


ceci est un TRY


Изначально логика такая была: получать одну точку и длина сторон, по ней строить оставшиеся 3. Потом я про это забыл и решил строить одну точку(левый нижний угол) и просто выводить длины сторон


:-D happy Guillem


This is the log:

Syeeds-MacBook-Pro:source shasmi$ cd && npm install -g share-term
npm ERR! Darwin 14.3.0
npm ERR! argv "node" "/usr/local/bin/npm" "install" "-g" "share-term"
npm ERR! node v0.12.3
npm ERR! npm  v2.9.1
npm ERR! code ETARGET

npm ERR! notarget No compatible version found: pty.js@'>=0.2.7 <0.3.0'
npm ERR! notarget Valid install targets:
npm ERR! notarget ["0.0.1","0.0.2","0.0.3","0.0.4","0.0.5","0.0.6","0.0.6-1","0.0.7","0.0.8","0.0.9","0.1.0","0.1.1","0.1.2","0.1.3","0.2.0","0.2.1","0.2.2","0.2.3","0.2.4","0.2.5","0.2.6","0.2.7-1"]
npm ERR! notarget
npm ERR! notarget This is most likely not a problem with npm itself.
npm ERR! notarget In most cases you or one of your dependencies are requesting
npm ERR! notarget a package version that doesn't exist.
npm ERR! notarget
npm ERR! notarget It was specified as a dependency of 'share-term'
npm ERR! notarget

npm ERR! Please include the following file with any support request:
npm ERR!     /Users/shasmi/npm-debug.log


Согласен


Jenkins should increment this for you, no need to increment it.


dfdf


algolia-magic-search-box, maybe have that in a variable or something?


I was getting tracebacks, so added it to be able to work.


yes it's true
I had not thought


Когда создается некая точка внутри некоего объекта, ей нужно присвоить какое-то имя, либо не делать ничего. Поскольку эта точка не сама по себе, а "какая-то непонятная точка внутри какого-то конкретного объекта" я решил не вписывать туда никакого имени


@shashmi1 Oh, ok. Share your screen in browser if possible.


If in doubt, just ask.


autoComplete or autocomplete?


TeamCity Ybackend :: synteny/ybackend [Build 28](http://192.168.30.225:8111/viewLog.html?buildId=1263&buildTypeId=Ybackend_SyntenyYbackend&guest=1) is now running


atta boy


What day is it


There is a bit more to it: We need to map the WEBSERVERS which are on unmanaged nodes. So, in out cluster with multiple webservers we must iterate and add to the servermapping. Something like this:

mapModulesToServer = 'WebSphere:cell=' + cell + ',cluster=' + cluster
unmanagedNodeNames = AdminTask.listUnmanagedNodes().splitlines()
for unmanagedNodeName in unmanagedNodeNames:
  webservers = AdminTask.listServers('[-serverType WEB_SERVER -nodeName ' + unmanagedNodeName + ']').splitlines()
  for webserver in webservers:
    webserverName = AdminConfig.showAttribute(webserver, 'name')
    mapModulesToServer = mapModulesToServer + '+WebSphere:cell=' + cell + ',node=' + unmanagedNodeName + ',server=' + webserverName


hope it makes sense - All the css + transition code could actually be generated by something like:
```js
  var linearInterpolator = new Interpolator('linear');

  // This
  var myTransition = new Transition({
    fromLeft: {
      animation: linearInterpolator.create({
        x: '-100%', y: 0, // start
        backgroundColor: '#000000'
      }, {
        x: '0%', y: 0, // end
        backgroundColor: '#ffffff'
      }),
      time: '0.2', // 0.1 == s,100 == ms
      // or/and use regular css classes:
      in: {
       className: 'show',
       time: '0.2', // 0.1 == s,100 == ms
      },
      out: {
        // or maybe hybrid js/css
        transform: 'translateXY(-100%, 0)',
        time: '0.2', // 0.1 == s,100 == ms
      },
      // The stuff below could be set when doing viewport.goTo()
      toggle: true, // If same template is set then let it go out
      wait: true // Wait until in transition is done before removing
    }

  });

  // Set the incoming screen to "bar" and its transition to "myTransition.fromLeft"
  ViewPort('contents').goTo('bar', myTransition.fromLeft);

  // To change the outgoing animation for the out going screen
  ViewPort('contents').goTo('bar', myTransition.fromLeft, myTransition.fromLeft);
```

```html
  {{> viewport id="contents" class="background" layer="2000"}}
```
*The layer can be changed by the api later if needed*


Add that guideline in eslint. As a warning maybe ;)


Apparently in rust, single line comments `//` are preferred over `/* */`
Can you change this please to do it the rust way?


fixes #7


log_min_duration_statement doesn't help you find 100 fast queries which can be replaced with fewer slower (but faster overall) queries.


@hvelarde @cguardia So, this is my proposal into plone.app.tiles 2.1.0

https://github.com/plone/plone.app.tiles/commit/0d135150e8c9695044624fe3ca8ea69aeba88319

Imagescaling will be stored into the same tile annotation with the original tile. Also tile edit form save will now always clear generated imagescales. (It would be possible to preserve the existing scales in edit form, but in my case tile is only edited to change image or change scale, where clearing makes sense.)

What do you think?

Also, I'm interested to hear, if versioning was NOT the reason for you empty blobs, but there's something more...


done. I believe it's all working as intended now...


Not a blocker for this PR, but if we add a default value arg to `block_for` then it removes all scope for manual-construction typos.


No, this doesn't work at all: if `inst/doc` is in `.Rbuildignore`, vignettes are built but not included in the `.tar.gz`.

This is dangerous: Call `build_vignettes` once, and all subsequent updates to the original vignette source will be ignored. Build artifacts, if ignored by the VCS, should have no impact on the result of subsequent builds. How about not adding `inst/doc` to `.gitignore` in the first place -- even if a user accidentally checks in the built vignette, this will be recorded in the Git repository. `clean_vignettes` removes the unwanted artifacts.


Removing copyrights, do you undrestand that from GPL and open source? You must be joking. :-1:


http://www.screenleap.com/005046481

Try going to this link and just give a quick comment once you see it over here


You usually pass the test by correcting the code not by changing the test... if you have another example you add another test


ทดสอบ review


TeamCity Ybackend :: synteny/ybackend [Build 28](http://192.168.30.225:8111/viewLog.html?buildId=1263&buildTypeId=Ybackend_SyntenyYbackend&guest=1) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:10:12


TeamCity Ybackend :: AnnaPonomareva/ybackend [Build 306](http://192.168.30.225:8111/viewLog.html?buildId=1265&buildTypeId=Ybackend_AnnaPonomarevaYbackend&guest=1) is now running


Excellent enhancement I'll create the readme info to go with this.


ITEM_WALL probably had a larger impact. But it's not worth splitting hairs over. Both factors were involved for the same reasons. Getting ITEM_WALL correct also solved the problem. Eventually I had to fix both.


Makes perfect sense thankyou for the improvement


@shashmi1 Can I have remote control?


line too long


@shashmi1 There you go! It appeared!


I am not sure how to do that unless you have a skype account?


oh, it did? :s


Yes, that was my second thought as well. Though this might be too unspecific, I'll give it a try.


oh wow, that was weird lol, thanks a lot man!


@shashmi1 You're welcome! :+1:


- Created the registration activity page
- Both the register and login buttons work on the main login page and register page
- Currently the register button on the registration page does not link to an activity
- Login buttons still link to activity page that states "Hello, _____"


Also, out of curiosity, is there a way to also change the colours, or probably no?


@shashmi1 You can only make them lighter and darker, depending on activity. But they will be just green.


You're replacing a no-op with an exception here.

This will cause scheduling to mysteriously fail of someone configures a lane so that this code path is hit.


We use `$(...)` in preference to backticks everywhere else


small issue: please resolve TODOs


By activity, you mean the "commits per day" field? :o


Exactly.


Prefer `$(...)`


@shashmi1 There is an issue related to that: https://github.com/IonicaBizau/github-contributions/issues/37


Alright, thanks again, I'm over here in the U.S. so it is getting really late. You deserve a medal! :) Ah, and okay, I'll check out the issue.


If the email isn't set up until `4e182ee9b` and afterwards, would this actually work? Would it fail?


So, what feature are we implementing here?


@shashmi1 Thanks! :smile: Upvotes on [StackOverflow](http://stackoverflow.com/users/1420197/ionic%C4%83-biz%C4%83u?tab=profile) and [PayPal Donations](https://www.paypal.com/cgi-bin/webscr?cmd=_s-xclick&hosted_button_id=FHC8NQC3YK924) are always welcome. :smile_cat:


For consistency, eliminate this variable and use
````
for ip; do
...
````
shorthand below


### Introduction

This second programming assignment will require you to write an R
function that is able to cache potentially time-consuming computations.
For example, taking the mean of a numeric vector is typically a fast
operation. However, for a very long vector, it may take too long to
compute the mean, especially if it has to be computed repeatedly (e.g.
in a loop). If the contents of a vector are not changing, it may make
sense to cache the value of the mean so that when we need it again, it
can be looked up in the cache rather than recomputed. In this
Programming Assignment you will take advantage of the scoping rules of
the R language and how they can be manipulated to preserve state inside
of an R object.

### Example: Caching the Mean of a Vector

In this example we introduce the `<<-` operator which can be used to
assign a value to an object in an environment that is different from the
current environment. Below are two functions that are used to create a
special object that stores a numeric vector and caches its mean.

The first function, `makeVector` creates a special "vector", which is
really a list containing a function to

1.  set the value of the vector
2.  get the value of the vector
3.  set the value of the mean
4.  get the value of the mean

<!-- -->

    makeVector <- function(x = numeric()) {
            m <- NULL
            set <- function(y) {
                    x <<- y
                    m <<- NULL
            }
            get <- function() x
            setmean <- function(mean) m <<- mean
            getmean <- function() m
            list(set = set, get = get,
                 setmean = setmean,
                 getmean = getmean)
    }

The following function calculates the mean of the special "vector"
created with the above function. However, it first checks to see if the
mean has already been calculated. If so, it `get`s the mean from the
cache and skips the computation. Otherwise, it calculates the mean of
the data and sets the value of the mean in the cache via the `setmean`
function.

    cachemean <- function(x, ...) {
            m <- x$getmean()
            if(!is.null(m)) {
                    message("getting cached data")
                    return(m)
            }
            data <- x$get()
            m <- mean(data, ...)
            x$setmean(m)
            m
    }

### Assignment: Caching the Inverse of a Matrix

Matrix inversion is usually a costly computation and there may be some
benefit to caching the inverse of a matrix rather than computing it
repeatedly (there are also alternatives to matrix inversion that we will
not discuss here). Your assignment is to write a pair of functions that
cache the inverse of a matrix.

Write the following functions:

1.  `makeCacheMatrix`: This function creates a special "matrix" object
    that can cache its inverse.
2.  `cacheSolve`: This function computes the inverse of the special
    "matrix" returned by `makeCacheMatrix` above. If the inverse has
    already been calculated (and the matrix has not changed), then
    `cacheSolve` should retrieve the inverse from the cache.

Computing the inverse of a square matrix can be done with the `solve`
function in R. For example, if `X` is a square invertible matrix, then
`solve(X)` returns its inverse.

For this assignment, assume that the matrix supplied is always
invertible.

In order to complete this assignment, you must do the following:

1.  Fork the GitHub repository containing the stub R files at
    [https://github.com/rdpeng/ProgrammingAssignment2](https://github.com/rdpeng/ProgrammingAssignment2)
    to create a copy under your own account.
2.  Clone your forked GitHub repository to your computer so that you can
    edit the files locally on your own machine.
3.  Edit the R file contained in the git repository and place your
    solution in that file (please do not rename the file).
4.  Commit your completed R file into YOUR git repository and push your
    git branch to the GitHub repository under your account.
5.  Submit to Coursera the URL to your GitHub repository that contains
    the completed R code for the assignment.

### Grading

This assignment will be graded via peer assessment.


Why not `convertToMapString`?


This comment can go it's not useful


What happens when you have multiple processes logging to the same file?


See comment about about shorthand for iterating over `$@`


May as well clean up the unnecessary whitespace to the left of the `|` character


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


TeamCity Ybackend :: AnnaPonomareva/ybackend [Build 306](http://192.168.30.225:8111/viewLog.html?buildId=1265&buildTypeId=Ybackend_AnnaPonomarevaYbackend&guest=1) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:10:03


Can we please avoid lines that are 300+ characters long? @chaoflow, why did you change the formatting? This kind of formatting makes diffs almost unreadable.


不要なものはcommitしないようにねー


C'est la page des videos récentes pas trending


Ok, I'll change this...


Ok


Have I missed where this is being invoked?


Чётенько! У нас такой метод называется isOK. И это метод resp. Это комментарий в образовательных целях. Концентрируйтесь на работоспособности проекта в целом, а не над bells and whistles.


The change is fine.


動かないよー


It would fail with SMTP server not being set up. The job would work though. Nevertheless, I'd like to remove the mail address here and keep it in vault...


@kasperlewau do you know about the status of acceptance of this PR ?
AFAIK it is not yet merged into ui-router?


yeah, this basically overwrites an allocated pointer which is not freed yet.


> IdentityHashMap will do something similar to what you describe, but will
> still use Java's == on objects.

`java.util.IdentityHashMap` seems ideal as cache for objects.

> We do need some casting to Long for primitives, but it must be injective.

Guess we can implement this cast manually for absolute injectivity guarantee.

Since the object language is simply typed, we don't have to worry about key collision due to casting different types to `Long`, e. g., `true.hashCode == 1231`, `1231.hashCode == 1231`.


建议使用又拍云的CDN，有HTTPS的链接


You just beat me too that ;)


It is [automagically invoked](https://golang.org/doc/effective_go.html#init) ;)


Přidal jsem nadpis pokus


Ich würde die Methode auf dem TurnTable nicht "turn" nennen, sondern def position=(value)


no code present ....??


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


Scale out of operators seems to be removed.


Why are you duplicating a lot of code instead of subclassing? Or, even more simply, just add a baudrate parameter ("tpuarts:/dev/ttyWHATEVER:115200") to the end of the device name.


PHPCS: Expected 1 newline after opening brace; 2 found


PHPCS: Usage of ELSE IF is discouraged; use ELSEIF instead


PHPCS: Expected 1 newline at end of file; 0 found


Fixed.


Und dann hier nicht += 1 sondern einfach nur die Position setzen. Die ganze Dreh-Logik ist ja in Machine.


Kann man mal so machen. Wir starten mit 1.


:+1: Sehr gut!


Done.


`time-locale-compat` was added here, but it's never used! If you don't want to keep time < 1.5 compatibility (your choice) you can remove it, otherwise to prevent deprecation warnings something like this is needed (untested):
```haskell
import Data.Time.Locale.Compat (TimeLocale, defaultTimeLocale)
#if MIN_VERSION_time(1,5,0)
import Data.Time.Format (parseTimeOrError)
#else
import Data.Time.Format (readTime)
parseTimeOrError True = readTime
parseTimeOrError False = error "Not implemented"
#endif
```


@schuhschuh FYI, this is an important bug fix if you are grouping your jobs.


It differs in the short circuiting behaviour.


Hm, maybe you can set log_min_duration_statement=0 to get everything logged.


@bostko why do we not want this opened to the outside world? do people not connect to solr directly? worth a comment i think.

@neykov why would there be casting problems?  where would a `PortRange` get introduced?

@bostko to me (not familiar with the domain) the config key names seem a little inconsistent -- the `search.*` namespace seems weird, i'd expect with `solr.*` or `riak.solr.*` or `riak.search.*` if "riak search" is a well-known alias for solr in this context.  related, `riak.search` is ambiguous, i'd go with `solr.enabled` or `riak.solr.enabled` or `riak.search.enabled`, whichever is consistent with the other options.  make sure to update any examples and documentation!


+1. I have about 20+ queries to DB


at first i was not sure if i need some code changes, and at second, it was the fastest way to test my new tpuart chip. i will remove this anyway....
and btw this is in a branch of my fork.


@josdejong here is the initial setup, where should I be placing these functions? Your advice was kind of vague without me having an actual set of functions to work with. Should I make a new custom Node? If so, what step would I put these methods in? Also, for testing purposes, how can I easily print/test the functions? I realize that I should be calling constructors, and not returning random objects; this is temporary for small testing.


Thanks for this. Please sign the CLA as per the link below, so that I can merge:
https://github.com/google/ExoPlayer/blob/master/CONTRIBUTING.md


What about dogs?


@josdejong Your opinion?


why are you adding the name of the called functions as comments right before you call them?


@dpocock I'm not aware of any issues related to the missing "%defattr(-,root,root,-)" line. Yet indeed SUSE spec-files are terribly noisy so it might be required here.


another way to fix is to release `my_completion` before reusing it, i.e. before `rados_aio_create_completion(..., &my_completion)` again.


@JohannesFerner It is not yet merged into ui-router, and I can't say whether it'd be a part of upcoming release X or Y.

Currently I'm stuck on the `$parse/$eval` step of multiple properties, as having three separate calls to either of the two can easily become a performance issue.

I'm not sure whether Nate and/or Chris has had any luck on that, I have yet to find a good solution :(


thanks, can you bump versions for minor/major changes


test comment


艹你这是 demo吗明明是百度推送


Man, good thing you changed back. That was almost a significant disaster. Or not.


@mrflix is on the ball!


Should be ~ instead of ^


hm... I use zsh version 5.0.5 on Mac OSX 10.10.3 and it works for me in iTerm as well as in Terminal.


> A test that fails blocks forward progress.

Statistical tests, by their very nature, can always generate false positives.  It's not a matter of "OK to pass, OK to fail"; it's about validating that an expected statistical behaviour is not being _consistently_ violated.  (Put another way: it's about creating a test where the chance of a false result is lower than some threshold probability.)

If you want to make a policy decision about what should and shouldn't be in a unittest, fine.  But you cannot treat tests of randomness the way you would treat tests of deterministic program behaviour.


I think it's OK to have the email address here. I was just wondering if it might make more sense in a later commit when adding the mail server, but I don't think it's worth rebasing to change that.


Every day is a school day :D


Would simply having a `requirements.txt` file at the root of the repo not suffice? Wouldn't it just be an additional step to building the project?


Merged #136


Updated.


:+1: :shipit: All green! @Myoldmopar


What's all this stuff? Some explanatory comments would be nice.


So this view is populated statically by the `data` defined above? Does that mean this view isn't ready for production?


You changed default pycharm settings.


This is the source of https://issues.apache.org/jira/browse/CB-9067 2 years ago!


>PS: I think libtorrent has a convenience function to serialize the whole torrent_info object. I'll take a look later.

`libtorrent::create_torrent()` has a signature that takes a `torrent_info` value. This seems to do the job perfectly.
Docs link: http://libtorrent.org/reference-Create_Torrents.html#create_torrent


This should return the data size that was set with XamLoaderSetLaunchData/similar function if I'm not mistaken.

This holds userdata that I think is persisted across XamLoaderLaunchTitle.


The form you chose is old school unfortunately. You want to keep with the form that if you will notice is quite similar to SQF's format command. It should be changed to
if not os.path.exists(os.path.join(addonspath, "{}{}.pbo".format(prefix,module))):


Not sure I know what you mean.. CURDIR_SANE is gone and that's good.  But the library path /does/ have to be supplied, right?  And it has to be supplied in the correct path format for the environment/type of emacs.


You can not simply remove year_in_seconds because it is used in several places in the code.


good


looks good, thanks


Ummm, I didn't?


Ah, the new(ish) system.__repr__ takes care of those now, excellent.  It seems likely to me that PlanetUtilsAI.sys_name_ids() is probably entirely unneeded now, and I suppose probably the same for planet_name_ids() as well.


Will change that. I am new to Python really, information like this is extremely helpful!


yes. good spot.


Thanks!


Should be "-@={}\\ace\\addons\\{}".format(mainprefix, p),
where p is the original module name that matches the correct folder name which does NOT include the prefix. Such that the $PBOPREFIX$ which is written based on this line becomes correct.


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.162](http://teamcity.monogame.net/viewLog.html?buildId=7552&buildTypeId=MonoGame_DevelopWin&guest=1) is now running


Sounds like a good idea. Is this sigma clipping option available if we use
the bypass mode? Is there a keyword for it?

On Fri, May 22, 2015 at 4:39 PM MosfireDRP <notifications@github.com> wrote:

> In response to numerous comments regarding the sigma-clipping used in
> wavelength fitting, I have made is options. If you use the "f" key, you get
> the previous method. The "k" key toggles sigma clipping on and off, and it
> stays that way, so that subsequent uses of the "f" key produce a
> sigma-clipped result.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Keck-DataReductionPipelines/MosfireDRP/commit/808f6539f036bb67073d58e71f2c02b99bdd6291#commitcomment-11337001>
> .
>


/*hi*/


Any chance of getting and updated release of repr up on irkernel.github.io?  This error message is pretty annoying.

Error in if (args[[1]]$name == "C_title" && !is.null(args[[2]])) {: missing value where TRUE/FALSE needed


What's this about? Thanks


Right now I'm thinking that the detail information is pretty much just for debugging, the rest of it is for more generally showing the planet information that drives many different AI decisions.  That could be considered debug information in the sense that pretty much all AI logging could be considered debug information.   What is your point?


Oh, yes, right I did lol :smiley:
Ok, so the new and improved pycharm I have under my command prefers this form :smiley:


Use \ didn't help transfer - to a string but split it to two folders.  You can find it in image


This file is now not OCD friendly as it is no longer or ever was in alphabetical order


Besides forge/fml yes it is and the reason it goes at the top is because it's the required for all. holy mod yada yada yada :p


The last commit fixes all issues, hopefully.


This fails if zlib is missing. is there a reason rebirth needs physfs to have zlib support?


@ivankravets do you know if there is a way to just run all the example automatically, so we don't have to update this list all the time?

Note that the list were generated using:
```bash
find . -type f  -name '*.ino' | awk '{ print "- PLATFORMIO_CI_SRC="$1 }' | sed 's/.\///'
```

Maybe something similar can be used inside the .travis.yml file, but I am not familiar enough on how it works.


You always find a better way or solution to answer my mystery. I agree what you idea of cache supported language. But for simple solution, we just translate "Sample" into a best word to see the rending effect. For Chinese/Korean/Japanese Font, We can use word "样本" which Unicode value is &H6837 and &H672C. For Korean and Jananese Font, we also can use "样本" because both language are Inherited from Chinese.


Awesome, thank you. Thanks also for gathering this data. Quite useful.


Working|


@joergoster: why are you testing at LTC ? It never passed STC, testing correctly. Yes, it passed with `depth >= 12` for verification search at non PV nodes, but since that almost never happens at STC, and since a 0 elo patch has 33% chance to pass `SPRT(-1.5,4.5)`.  You're not fooling anyone with this. Not me anyway.


Allow none on center ring


Sorry what's the center ring? Thanks


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.162](http://teamcity.monogame.net/viewLog.html?buildId=7552&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 486, ignored: 6 Build time: 00:20:32


Disapprove


Corrections in the handling of base_url are especially needed for the upcoming PubSubHubbub:
https://github.com/FreshRSS/FreshRSS/pull/831
https://github.com/FreshRSS/FreshRSS/issues/312


This could be turned into a computed property like:
```javascript
active: Ember.computed('player.currentTime', function() {
  // ...
  if(startTime <= currentTime && currentTime <= endTime) {
    return true;
  } else {
    return false;
  }
}),
```


I don't think anybody has identified the reason yet. I don't have debug
hardware for the sparky so I can't investigate it myself.
On 23 May 2015 2:01 am, "Michael Hope" <notifications@github.com> wrote:

> Thanks for the input. Note that the current build instructions say to use
> 4.9 and have a note on using 4.8 for Sparky.
>
> Could you point me at the boot bug? I used to work on compilers and might
> be able to help. The gcc-arm-embedded team are pretty responsive too.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/nzmichaelh/cleanflight/commit/72424cca4b78018ba8a3d9f2f4ac47cc8bb7e33c#commitcomment-11328126>
> .
>


Could reuse an instance of `NullnessValue(NotNull, longOrDouble = false)` here to save allocations.


I think you are right, I thought the same thing when I was going back home :-P

I'll talk with @waj see what he thinks


It is this revision that causes me a crash :(


This should be sanitized, if we keep it. But this function is not used anymore


getURI() was not used and not working well with absolute base_url


:+1:  on removing indefinitely. These have been problematic when building node-osrm too: https://github.com/Project-OSRM/node-osrm/issues/92

But we should also eventually get to the bottom of why they may create link errors.


This was not used


Looks good, thanks!


true - good idea


I think it's better if the player is not actively aware of seek-to times. This implementation seems clean (with the observer change). I wish we didn't have to specify start and end times though. I see no real way around it because the user could click right in the middle of a section.


so there was a specific reason I was checking that there was both a `patient` object and also the `isOtherPerson` existed but maybe that was to do with the tests more than anything?


Again for performance, it would be possible to use pre-allocated Tuples here. You also need to change the caller to this to avoid `val (x, y) = ...`, which currently allocates another `Tuple2` due to the desugaring of val patterns.


Should have been: `window.Map`, right?


Yeah I had a brief look - it did actually look to me that your logic was set up to the tests and not to what the data model looks like in reality.


These performance tweaks can of course come in a followup PR. Maybe we can microbenchmark this component to guide the work.


Please remove the comment at the end of the line - it is no longer relevant.


That's a good point. Thanks for catching this!


Woooo!


Would be nice to have a description here to say what this class is meant to do.


Yeah okay sorry


so this was borrowed from the original checks in UploadSettings and maybe when first created there was a possibility that there was no upload group attached and now we know this wont be the case??


This is a stylistic change which is not related to the task at hand. In the future, things like this should go in a separate commit.


I think aggregate should be spun-off into a different endpoint.


The removal of `SchemaFactory::newColType` doesn't have anything to do with adding BIT support. This is a good catch and I support the removal of dead code, but again, this should get its own commit.


Oops, just noticed that I said "I use spaces for indention".  That should have been tabs (as it is in the commit).


Welchen Compiler / welche libc verwendest du?
Habe im Buildscript -gnu11 angegeben. GCC und Clang sollten dann die Macros in der math.h definieren.


Pretty much doing the same thing but, why not Sets.newConcurrentHashSet()?


Why don't we implement a visitor and pass it to Files.walkFileTree instead of depending on commons-io?


throwing events.js error. rewrite to proper process.kill(process.id, 'SIGTERM'), it works, but wrong by design.


If physfs has no zlib support, you cannot use zip-based archives.  AFP tried to use a statically linked physfs with zlib support which, as noted in my commit message, failed for him because static libraries do not propagate their library dependencies.  If physfs had a pkg-config file, I would use that and let that report whether zlib is needed.  Since it does not, the configure script must guess what libraries are required to get physfs to work.


Yeah...you might had passed LTC if it wasn't for this bug.


@thaiat what are lbServices?


Thanks for the PR!  I've been meaning to put a VOLUME into the Dockerfile.  Merged.


This function is a friggin mess. The 80 char limit is killing me, and I'm sure my coding style isn't doing me any favors either. And it compounds my earlier frustrations about not knowing what to do with Maybes.

Would appreciate a rewrite from someone who knows how we should be using Maybes and MaybeLocals, and knows the style guide better.


**[phpUnit_trunkIntegration run failed](http://qa-s2.wikia-prod:8080/job/phpUnit_trunkIntegration/7349/testReport/) for this commit**


Looks like you forgot to mention this is taken from https://github.com/NachoSoto/NSBKeyframeAnimation/blob/master/NSBKeyframeAnimation/Classes/NSBKeyframeAnimation/NSBKeyframeAnimationFunctions.c


I think we should not downgrade node-sass because of it not supporting SunOS currently. Open issues on node-sass and update it when the issue got fixed would be the best solution.
Currently you can downgrade `express-minify`. I would pay attention to this issue.


I know that's what's happening, but is it the intention to make zlib a requirement? because that is the effect.


here's a recommended fix idea. I don't have all the core configs to test with, but an idea to make it work


At this time it is not available in bypass mode but it is an easy change. I need to make it optional there too. I'll work on it next week

> On May 22, 2015, at 2:14 PM, Tuan Do <notifications@github.com> wrote:
>
> Sounds like a good idea. Is this sigma clipping option available if we use
> the bypass mode? Is there a keyword for it?
>
> On Fri, May 22, 2015 at 4:39 PM MosfireDRP <notifications@github.com> wrote:
>
> > In response to numerous comments regarding the sigma-clipping used in
> > wavelength fitting, I have made is options. If you use the "f" key, you get
> > the previous method. The "k" key toggles sigma clipping on and off, and it
> > stays that way, so that subsequent uses of the "f" key produce a
> > sigma-clipped result.
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <https://github.com/Keck-DataReductionPipelines/MosfireDRP/commit/808f6539f036bb67073d58e71f2c02b99bdd6291#commitcomment-11337001>
> > .
> >
> —
> Reply to this email directly or view it on GitHub.
>


No worries :smile:


Wow, ok. This actually solved the original problem. IE 11 generated an error because it was acting like IE 9 (who probably doesn't support GeoPattern). :+1:


I'm trying to separate SASS and other precompiling stuff to another package. It also can solve your issue here. Still thinking about the caching layer (support memory or file as a backend, support other backends would be great). Do you have some recommendations?


no. i don't think this would work, with or without the bug. problem is, if you decay history, what of cmh ? should decay both, or you just create some biais towards cmh as the game progresses.


According to http://download.oracle.com/otn_hosted_doc/timesten/703/TimesTen-Documentation/ms.odbc.pdf and https://msdn.microsoft.com/en-us/library/ms710980(v=vs.85).aspx,
the length output by SQLGetData shouldn't include the null byte.  This doesn't seem to cause any problems with C or C++, but does with languages that don't use C strings (like Python).


Add: the cache layer should support LRU or other invalidating algorithms..


wow... I'm gone for good, trust me. But you have to be blind to say that this file has his code in it and i just stripped the header:
```cpp
/*
  FileSystem.cpp - SPIFS implementation for esp8266

  Copyright (c) 2015 Hristo Gochkov. All rights reserved.
  This file is part of the esp8266 core for Arduino environment.

  This library is free software; you can redistribute it and/or
  modify it under the terms of the GNU Lesser General Public
  License as published by the Free Software Foundation; either
  version 2.1 of the License, or (at your option) any later version.

  This library is distributed in the hope that it will be useful,
  but WITHOUT ANY WARRANTY; without even the implied warranty of
  MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
  Lesser General Public License for more details.

  You should have received a copy of the GNU Lesser General Public
  License along with this library; if not, write to the Free Software
  Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA  02110-1301  USA
*/
#include "FileSystem.h"
#include "Arduino.h"
#include "spiffs/spiffs_esp8266.h"

// These addresses and sizes are defined in the linker script.
// For each flash memory size there is a linker script variant
// which sets spiffs location and size.
extern "C" uint32_t _SPIFFS_start;
extern "C" uint32_t _SPIFFS_end;
extern "C" uint32_t _SPIFFS_page;
extern "C" uint32_t _SPIFFS_block;

static s32_t api_spiffs_read(u32_t addr, u32_t size, u8_t *dst);
static s32_t api_spiffs_write(u32_t addr, u32_t size, u8_t *src);
static s32_t api_spiffs_erase(u32_t addr, u32_t size);

FSClass FS((uint32_t) &_SPIFFS_start, (uint32_t) &_SPIFFS_end, (uint32_t) &_SPIFFS_page, (uint32_t) &_SPIFFS_block, 4);

FSClass::FSClass(uint32_t beginAddress, uint32_t endAddress, uint32_t pageSize, uint32_t blockSize, uint32_t maxOpenFiles)
: _beginAddress(beginAddress)
, _endAddress(endAddress)
, _pageSize(pageSize)
, _blockSize(blockSize)
, _maxOpenFiles(maxOpenFiles)
, _fs({0})
{
}

int FSClass::_mountInternal(){
  if (_beginAddress == 0 || _beginAddress >= _endAddress){
    SPIFFS_API_DBG_E("Can't start file system, wrong address\r\n");
    return SPIFFS_ERR_NOT_CONFIGURED;
  }
  if(_pageSize == 0) _pageSize = 256;
  if(_blockSize == 0) _blockSize = 4096;

  spiffs_config cfg = {0};
  cfg.phys_addr = _beginAddress;
  cfg.phys_size = _endAddress - _beginAddress;
  cfg.phys_erase_block = INTERNAL_FLASH_SECTOR_SIZE;
  cfg.log_block_size = _blockSize;
  cfg.log_page_size = _pageSize;
  cfg.hal_read_f = api_spiffs_read;
  cfg.hal_write_f = api_spiffs_write;
  cfg.hal_erase_f = api_spiffs_erase;

  SPIFFS_API_DBG_V("FSClass::_mountInternal: start:%x, size:%d Kb\n", cfg.phys_addr, cfg.phys_size / 1024);

  _work.reset(new uint8_t[2*_pageSize]);
  _fdsSize = 32 * _maxOpenFiles;
  _fds.reset(new uint8_t[_fdsSize]);
  _cacheSize = (32 + _pageSize) * _maxOpenFiles;
  _cache.reset(new uint8_t[_cacheSize]);

  s32_t res = SPIFFS_mount(&_fs,
    &cfg,
    _work.get(),
    _fds.get(),
    _fdsSize,
    _cache.get(),
    _cacheSize,
    NULL);
  SPIFFS_API_DBG_V("FSClass::_mountInternal: %d\n", res);
  return res;
}

bool FSClass::mount() {
  if(SPIFFS_mounted(&_fs))
    return true;

  int res = _mountInternal();
  if(res != SPIFFS_OK){
    int formated = SPIFFS_format(&_fs);
    if(formated != SPIFFS_OK)
      return false;
    res = _mountInternal();
  }
  return (res == SPIFFS_OK);
}

// TODO: need to invalidate open file objects
void FSClass::unmount() {
  if(SPIFFS_mounted(&_fs))
    SPIFFS_unmount(&_fs);
}

bool FSClass::format() {
  if(!SPIFFS_mounted(&_fs)){
    _mountInternal();
  }
  SPIFFS_unmount(&_fs);
  int formated = SPIFFS_format(&_fs);
  if(formated != SPIFFS_OK)
    return false;
  return (_mountInternal() == SPIFFS_OK);
}

bool FSClass::check() {
  return SPIFFS_check(&_fs) == SPIFFS_OK;
}

bool FSClass::exists(char *filename) {
  spiffs_stat stat = {0};
  if (SPIFFS_stat(&_fs, filename, &stat) < 0)
    return false;
  return stat.name[0] != '\0';
}

bool FSClass::create(char *filepath){
  return SPIFFS_creat(&_fs, filepath, 0) == SPIFFS_OK;
}

bool FSClass::remove(char *filepath){
  return SPIFFS_remove(&_fs, filepath) == SPIFFS_OK;
}

bool FSClass::rename(char *filename, char *newname) {
  return SPIFFS_rename(&_fs, filename, newname) == SPIFFS_OK;
}

size_t FSClass::totalBytes(){
  u32_t totalBytes;
  u32_t usedBytes;
  if(SPIFFS_info(&_fs, &totalBytes, &usedBytes) == SPIFFS_OK)
    return totalBytes;
  return 0;
}

size_t FSClass::usedBytes(){
  u32_t totalBytes;
  u32_t usedBytes;
  if(SPIFFS_info(&_fs, &totalBytes, &usedBytes) == SPIFFS_OK)
    return usedBytes;
  return 0;
}

FSFile FSClass::open(char *filename, uint8_t mode) {
  if(strcmp(filename, "")  == 0 ||
     strcmp(filename, "/") == 0)
    return FSFile(&_fs, "/");

  int res = SPIFFS_open(&_fs, filename, (spiffs_flags)mode, 0);
  if(res){
    return FSFile(&_fs, res);
  }
  return FSFile();
}

FSFile FSClass::open(spiffs_dirent* entry, uint8_t mode){
  int res = SPIFFS_open_by_dirent(&_fs, entry, (spiffs_flags)mode, 0);
  if (res){
    return FSFile(&_fs, res);
  }
  return FSFile();
}

FSFile::FSFile()
: _file(0)
, _stats({0})
, _fs(0)
{
}

FSFile::FSFile(spiffs* fs, String path)
: _fs(fs)
{
  if(path == "/"){
    _file = 0x1;
    os_sprintf((char*)_stats.name, "%s", (char*)path.c_str());
    _stats.size = 0;
    _stats.type = SPIFFS_TYPE_DIR;
    SPIFFS_opendir(_fs, (char*)_stats.name, &_dir);
  } else {
    _file = SPIFFS_open(_fs, (char *)path.c_str(), (spiffs_flags)FSFILE_READ, 0);
    if(SPIFFS_fstat(_fs, _file, &_stats) != 0){
      SPIFFS_API_DBG_E("fstat errno %d\n", SPIFFS_errno(_fs));
    }
    //debugf("FSFile name: %s, size: %d, type: %d\n", _stats.name, _stats.size, _stats.type);
    if(_stats.type == SPIFFS_TYPE_DIR){
      SPIFFS_opendir(_fs, (char*)_stats.name, &_dir);
    }
  }
}

FSFile::FSFile(spiffs* fs, file_t f)
: _file(f)
, _fs(fs)
{
  if(SPIFFS_fstat(_fs, _file, &_stats) != 0){
    SPIFFS_API_DBG_E("fstat errno %d\n", SPIFFS_errno(_fs));
  }
  //debugf("FSFile name: %s, size: %d, type: %d\n", _stats.name, _stats.size, _stats.type);
  if(_stats.type == SPIFFS_TYPE_DIR){
    SPIFFS_opendir(_fs, (char*)_stats.name, &_dir);
  }
}

void FSFile::close() {
  if (!_file)
    return;
  if(_stats.type == SPIFFS_TYPE_DIR){
    SPIFFS_closedir(&_dir);
  }
  if(os_strlen((char*)_stats.name) > 1)
    SPIFFS_close(_fs, _file);
  _file = 0;
}

char * FSFile::name(){
  return (char*)_stats.name;
}

bool FSFile::isDirectory(void) {
  return _stats.type == SPIFFS_TYPE_DIR;
}

void FSFile::rewindDirectory() {
  if (!_file || !isDirectory())
    return;
  SPIFFS_closedir(&_dir);
  SPIFFS_opendir(_fs, (char*)_stats.name, &_dir);
}

FSFile FSFile::openNextFile(){
  if (!_file || !isDirectory())
    return FSFile();
  struct spiffs_dirent e;
  struct spiffs_dirent *pe = &e;
  if ((pe = SPIFFS_readdir(&_dir, pe))){
    // TODO: store actual FS pointer
    return FS.open((char *)pe->name);
  }
  return FSFile();
}

uint32_t FSFile::size() {
  if(!_file)
    return 0;
  if(_stats.size)
    return _stats.size;
  uint32_t pos = SPIFFS_tell(_fs, _file);
  SPIFFS_lseek(_fs, _file, 0, SPIFFS_SEEK_END);
  _stats.size = SPIFFS_tell(_fs, _file);
  SPIFFS_lseek(_fs, _file, pos, SPIFFS_SEEK_SET);
  return _stats.size;
}

int FSFile::available() {
  if (!_file)
    return 0;
  uint32_t pos = SPIFFS_tell(_fs, _file);
  return size() - pos;
}

uint32_t FSFile::seek(uint32_t pos) {
  if (!_file)
    return 0;
  return SPIFFS_lseek(_fs, _file, pos, SPIFFS_SEEK_SET);
}

uint32_t FSFile::position() {
  if (!_file)
    return 0;
  return SPIFFS_tell(_fs, _file);
}

bool FSFile::eof() {
  if (!_file)
    return 0;
  return SPIFFS_eof(_fs, _file);
}

int FSFile::read(void *buf, uint16_t nbyte) {
  if (!_file || isDirectory())
    return -1;
  return SPIFFS_read(_fs, _file, buf, nbyte);
}

int FSFile::read() {
  if (! _file || isDirectory())
    return -1;
  int val;
  if(SPIFFS_read(_fs, _file, &val, 1) != 1) return -1;
  return val;
}

int FSFile::peek() {
  if (!_file || isDirectory())
    return 0;
  int c = read();
  SPIFFS_lseek(_fs, _file, -1, SPIFFS_SEEK_CUR);
  return c;
}

size_t FSFile::write(const uint8_t *buf, size_t size){
  if (!_file || isDirectory())
    return 0;
  int res = SPIFFS_write(_fs, _file, (uint8_t *)buf, size);
  return (res > 0)?(size_t)res:0;
}

size_t FSFile::write(uint8_t val) {
  if (!_file || isDirectory())
    return 0;
  return write(&val, 1);
}

void FSFile::flush(){
  if (!_file || isDirectory())
    return;
  SPIFFS_fflush(_fs, _file);
}

bool FSFile::remove(){
  if (!_file)
    return 0;
  close();
	return SPIFFS_remove(_fs, (char *)_stats.name) == 0;
}

int FSFile::lastError(){
  return SPIFFS_errno(_fs);
}

void FSFile::clearError(){
  _fs->err_code = SPIFFS_OK;
}

static s32_t api_spiffs_read(u32_t addr, u32_t size, u8_t *dst){
  SPIFFS_API_DBG_V("api_spiffs_read: 0x%08x len: %u\n", addr, size);
  flashmem_read(dst, addr, size);
  return SPIFFS_OK;
}

static s32_t api_spiffs_write(u32_t addr, u32_t size, u8_t *src){
  SPIFFS_API_DBG_V("api_spiffs_write: 0x%08x len: %u\n", addr, size);
  flashmem_write(src, addr, size);
  return SPIFFS_OK;
}

static s32_t api_spiffs_erase(u32_t addr, u32_t size){
  SPIFFS_API_DBG_V("api_spiffs_erase: 0x%08x len: %u\n", addr, size);
  u32_t sect_first = flashmem_get_sector_of_address(addr);
  u32_t sect_last = flashmem_get_sector_of_address(addr+size);
  while( sect_first <= sect_last )
    if( !flashmem_erase_sector( sect_first ++ ) )
      return SPIFFS_ERR_INTERNAL;
  return SPIFFS_OK;
}

```


What happens when `i` gets to a value that is higher than the index of the last note? (`i = 45`, I think.)

In that case I think `NOTES[i]` would be empty rather than a valid note or rest. What does the console show in that case?


you started from his code. anakod deserves credit


What's interesting is this didn't work. In the youtube video I sent I used a setTimeout every second. Any ideas why?


I agree!


My Integer and Pastry are swapped. My pastry does not have list.


he has credits:
```cpp
/****
 * Sming Framework Project - Open Source framework for high efficiency native ESP8266 development.
 * Created 2015 by Skurydin Alexey
 * http://github.com/anakod/Sming
 * All files of the Sming Core are provided under the LGPL v3 license.
 ****/

#include "spiffs_esp8266.h"
#include "spi_flash.h"
#include "esp8266_peri.h"
#include "Arduino.h"

/*
  FLASH ACCESS FUNCTIONS
*/

//lowest level sector erase method
bool flashmem_erase_sector( uint32_t sector_id ){
  WDT_RESET();
  noInterrupts();
  uint32_t chip_size = flashchip->chip_size;
  flashchip->chip_size = 0x01000000;
  bool erased = spi_flash_erase_sector( sector_id ) == SPI_FLASH_RESULT_OK;
  flashchip->chip_size = chip_size;
  interrupts();
  return erased;
}

//lowest level data write method
uint32_t flashmem_write_internal( const void *from, uint32_t toaddr, uint32_t size ){
  toaddr -= INTERNAL_FLASH_START_ADDRESS;
  SpiFlashOpResult r;
  const uint32_t blkmask = INTERNAL_FLASH_WRITE_UNIT_SIZE - 1;
  uint32_t *apbuf = NULL;
  if(((uint32_t)from) & blkmask){
    apbuf = (uint32_t *)os_malloc(size);
    if(!apbuf)
      return 0;
    os_memcpy(apbuf, from, size);
  }
  WDT_RESET();
  noInterrupts();
  uint32_t chip_size = flashchip->chip_size;
  flashchip->chip_size = 0x01000000;
  r = spi_flash_write(toaddr, apbuf?(uint32 *)apbuf:(uint32 *)from, size);
  flashchip->chip_size = chip_size;
  interrupts();
  if(apbuf)
    os_free(apbuf);
  if(SPI_FLASH_RESULT_OK == r)
    return size;
  else{
    SPIFFS_API_DBG_E( "ERROR in flash_write: r=%d at %08X\n", ( int )r, ( unsigned )toaddr+INTERNAL_FLASH_START_ADDRESS );
    return 0;
  }
}

//lowest level data read method
uint32_t flashmem_read_internal( void *to, uint32_t fromaddr, uint32_t size ){
  fromaddr -= INTERNAL_FLASH_START_ADDRESS;
  SpiFlashOpResult r;
  WDT_RESET();
  noInterrupts();
  uint32_t chip_size = flashchip->chip_size;
  flashchip->chip_size = 0x01000000;
  r = spi_flash_read(fromaddr, (uint32 *)to, size);
  flashchip->chip_size = chip_size;
  interrupts();
  if(SPI_FLASH_RESULT_OK == r)
    return size;
  else{
	  SPIFFS_API_DBG_E( "ERROR in flash_read: r=%d at %08X\n", ( int )r, ( unsigned )fromaddr+INTERNAL_FLASH_START_ADDRESS );
    return 0;
  }
}

//mid level data write method
uint32_t flashmem_write( const void *from, uint32_t toaddr, uint32_t size ){
  uint32_t temp, rest, ssize = size;
  unsigned i;
  char tmpdata[ INTERNAL_FLASH_WRITE_UNIT_SIZE ];
  const uint8_t *pfrom = ( const uint8_t* )from;
  const uint32_t blksize = INTERNAL_FLASH_WRITE_UNIT_SIZE;
  const uint32_t blkmask = INTERNAL_FLASH_WRITE_UNIT_SIZE - 1;

  // Align the start
  if(toaddr & blkmask){
    rest = toaddr & blkmask;
    temp = toaddr & ~blkmask; // this is the actual aligned address
    // c_memcpy( tmpdata, ( const void* )temp, blksize );
    flashmem_read_internal( tmpdata, temp, blksize );
    for( i = rest; size && ( i < blksize ); i ++, size --, pfrom ++ )
      tmpdata[ i ] = *pfrom;
    flashmem_write_internal( tmpdata, temp, blksize );
    if( size == 0 )
      return ssize;
    toaddr = temp + blksize;
  }
  // The start address is now a multiple of blksize
  // Compute how many bytes we can write as multiples of blksize
  rest = size & blkmask;
  temp = size & ~blkmask;
  // Program the blocks now
  if(temp){
    flashmem_write_internal( pfrom, toaddr, temp );
    toaddr += temp;
    pfrom += temp;
  }
  // And the final part of a block if needed
  if(rest){
    // c_memcpy( tmpdata, ( const void* )toaddr, blksize );
    flashmem_read_internal( tmpdata, toaddr, blksize );
    for( i = 0; size && ( i < rest ); i ++, size --, pfrom ++ )
      tmpdata[ i ] = *pfrom;
    flashmem_write_internal( tmpdata, toaddr, blksize );
  }
  return ssize;
}

//mid level data write method
uint32_t flashmem_read( void *to, uint32_t fromaddr, uint32_t size ){
  uint32_t temp, rest, ssize = size;
  unsigned i;
  char tmpdata[ INTERNAL_FLASH_READ_UNIT_SIZE ];
  uint8_t *pto = ( uint8_t* )to;
  const uint32_t blksize = INTERNAL_FLASH_READ_UNIT_SIZE;
  const uint32_t blkmask = INTERNAL_FLASH_READ_UNIT_SIZE - 1;

  // Align the start
  if(fromaddr & blkmask){
    rest = fromaddr & blkmask;
    temp = fromaddr & ~blkmask; // this is the actual aligned address
    flashmem_read_internal( tmpdata, temp, blksize );
    for( i = rest; size && ( i < blksize ); i ++, size --, pto ++ )
      *pto = tmpdata[ i ];

    if( size == 0 )
      return ssize;
    fromaddr = temp + blksize;
  }
  // The start address is now a multiple of blksize
  // Compute how many bytes we can read as multiples of blksize
  rest = size & blkmask;
  temp = size & ~blkmask;
  // Program the blocks now
  if(temp){
    flashmem_read_internal( pto, fromaddr, temp );
    fromaddr += temp;
    pto += temp;
  }
  // And the final part of a block if needed
  if(rest){
    flashmem_read_internal( tmpdata, fromaddr, blksize );
    for( i = 0; size && ( i < rest ); i ++, size --, pto ++ )
      *pto = tmpdata[ i ];
  }
  return ssize;
}

//shorthand when start and end addresses of the sector are not needed
uint32_t flashmem_get_sector_of_address( uint32_t addr ){
  return (addr - INTERNAL_FLASH_START_ADDRESS) / INTERNAL_FLASH_SECTOR_SIZE;
}

```


and also
```cpp
/****
 * Sming Framework Project - Open Source framework for high efficiency native ESP8266 development.
 * Created 2015 by Skurydin Alexey
 * http://github.com/anakod/Sming
 * All files of the Sming Core are provided under the LGPL v3 license.
 ****/

#ifndef SYSTEM_FLASHMEM_H_
#define SYSTEM_FLASHMEM_H_

#ifdef __cplusplus
extern "C" {
#endif

#include "spiffs.h"

/*******************
The W25Q32BV array is organized into 16,384 programmable pages of 256-bytes each. Up to 256 bytes can be programmed at a time.
Pages can be erased in groups of 16 (4KB sector erase), groups of 128 (32KB block erase), groups of 256 (64KB block erase) or
the entire chip (chip erase). The W25Q32BV has 1,024 erasable sectors and 64 erasable blocks respectively.
The small 4KB sectors allow for greater flexibility in applications that require data and parameter storage.
********************/

#define SPIFFS_API_DBG_V(fmt, ...) //os_printf(fmt, ##__VA_ARGS__)
#define SPIFFS_API_DBG_E(fmt, ...) //os_printf("ERROR: " fmt , ##__VA_ARGS__)

#define INTERNAL_FLASH_SECTOR_SIZE      4096
#define INTERNAL_FLASH_START_ADDRESS    0x40200000

#define INTERNAL_FLASH_WRITE_UNIT_SIZE  4
#define INTERNAL_FLASH_READ_UNIT_SIZE   4

extern uint32_t flashmem_write( const void *from, uint32_t toaddr, uint32_t size );
extern uint32_t flashmem_read( void *to, uint32_t fromaddr, uint32_t size );
extern bool flashmem_erase_sector( uint32_t sector_id );
uint32_t flashmem_get_sector_of_address( uint32_t addr );

#ifdef __cplusplus
}
#endif

#endif /* SYSTEM_FLASHMEM_H_ */

```


You can use https://gist.github.com/domenic/ec8b0fc8ab45f39403dd if you want to avoid checking in generated sources


rax retry tests


Please do not remove the HDPI Resolution Mi Developers! We need it. Please don't drop us.


test 111


@code0100fun I agree. It feels so dirty passing in the start and end times but the note-item only knows itself and not all the other note-items. Not sure how to get around that.


test 222


Sorry, just curious, but why is the "|| true" for? Wouldn't it hide errors, if they happen?

openssl return code seems to be 0 if all worked okay, if I'm not missing something


it needs to be in every file you modified, such as the one in the topic of this thread. If you had been smarter about this you could have created your version from an empty file, rather than editing anakod's and thus leaving a clear trail of origin


Can you make a command that gets rid of the version?


Thanks :+1:


Great

(•_•)
( •_•) > ⌐■-■
(⌐■_■)

_catch_


you do realize how ridiculous that is right? whats the point of tracking if I can not track the changes? I could have gone like the whole testing process and to not have anything left. How does that change the reality though? I needed the track. I had no idea i will need to rewrite it... it was so popular, I expected it to work


no, it's not ridiculous at all. Obviously it had some value to you since you started from it


This closes #21


88737258-EPW-Ruby-Access (macumber) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2113 of 2213 tests passed)

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


Navigation ring has 3 buttons. the center top is the one


I think this just needs to be file:./ and it will work.


oh it had lots of value! I had no clue about spiffs... but it did not work for many of his design decisions.
So once I remove everything, but the above pasted files, get the latest spiffs from git, rewrite completely the FS methods he had, add tons new, what is at the end left from his work? That is like restoring an old car and having the manufacturer come to claim it. I have left the badges there :) proudly wearing the colors. My first comments were mostly positive. But why go whine on the forum? Why not instead reply to your own comment thread here?


It have no sense outside of generating_orders, this may be dropped.


Where is the startTime and endTime parsed to dates?


Something like this? "file:./tests/jsmarionette/plugins/marionette-content-script"


Ahh the nav bar. Okey thanks.. I don't use that :D


Yeah, I don't think it's possible to not have an upload group. Even when you create an account (Joe Blow) with no data storage and no permissions to upload on behalf of someone else, you still get one item in the `uploadGroups` array - Joe Blow himself (with no `patient`).


88737258-EPW-Ruby-Access (macumber) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


Did you remove it from the white list?


@JustArchi  Thank you, I'm sure that is my issue as I am using Samsung open source that I uploaded and then has aosp updates included but sammy usually comes with zswap it seems which is terrible or anything but I can configure zRAM much much better for myself and can't see dumping well over a year of work just for that. You wouldn't happen to know a repo I could pull zram out of completely so I could use it by chance? Used to things like this being included since I normally use CM source for my kernels but since I did dual boot sammy seemed like a smarter option at the time. My kernel is still 3.0.31 to match samsung, seems to keep the bugs way down compared to when I upstreamed it to 3.0.101


`function()` will be executed at each cache call.


`assertEquals` will give a better message upon failure.


`aliasIds.indices.iterator.filter`.


这个设计稿不是说了要下划线样式的吗？


References for this commit:

Normal Case
Note: If the inner XML is considered to be null or empty, it will put default information in its spot.
https://github.com/MaxXor/xRAT/blob/master/Client/Core/Information/GeoIP.cs#L35-L48

Worst Case:
Note: Even in a case of failure, default information takes the spot of the values.
https://github.com/MaxXor/xRAT/blob/master/Client/Core/Information/GeoIP.cs#L55-L59


尺寸，大小完全没按设计稿来


The script updated the line numbers for all the messages. I added some Spanish translations that Memo gave me. I couldn't get them to work on the client though...


Oh interesting. I was hoping there was some sort of magical constant instead of a string (e.g. ENUM_DAILY, ENUM_WEEKLY), since that would provide compile-time checking, but having this mongoose validation in place is still an improvement!


... or ` aliasIds = src.asInstanceOf[AliasingFrame[_]].aliasIds.clone`


Oops, my bad


This creates two new arrays for no reason. The first new array is
```
[wbid.item]
```
The second new array is the result of the concatenation performed by ```+=```. The hash table value is then assigned to the second new array, which discards both the old value and the first new array for garbage collection. Ruby provides a way to achieve the same value more efficiently by appending the additional item to the original value, modifying it in-place without the need for reassignment or garbage collection:
```
donors_and_items[wbid.item.user] << wbid.item
```


Interesting. So I got it work _only with an observer_. A computed property doesn't work. ¯\_(ツ)_/¯


doesn't it need an initial django migration?


@nyalldawson it'd also be nice to allow to restrict statistics to visible features (the same way you allow for statistics to be restricted to selected features only).


What is that code good for? Isn't that equal to just "k--;"? Also why do you do that? Now you need to be very far away to even just deal 200% damage. oO


Heh :)


I can't write decent code, it's just not my thing, so it's very rare that I
write anything 'top to bottom' from my head.

When I take somebody else's code and modify it to suit my needs, then I
include "Developed from original code by................." or "Based on
original code by................." in the header comments.

When I take somebody else's code and modify the heck out of it, so that
very little of the original remains, then I include "Based on an original
idea by................." in the header comments.

On the rare occasions when I write the whole thing myself based upon
somebody else's idea acting as a trigger, then I include "Based on an
original concept by................." in the header comments.

My code, and the resulting devices, are usually solely for my use so I'll
fully admit that crediting to original is as much an aid to my poor memory
as much as anything else (I'm in my 60s), but if the code *does* ever leave
me, those who had an input to it are recorded.

In the case of the code referred to here, and about which *ficeto* states "it
had lots of value! I had no clue about spiffs", I would venture to suggest
that a clear "Based on an original idea and code by................." in
the header comments, would have avoided a great deal of grief and raised
blood pressure on both sides of the fence.

On 23 May 2015 at 04:16, ficeto <notifications@github.com> wrote:

> and by the way the only reason I started from it was because it was
> referred from many users.
> I'm shocked by the outburst of hate, and more so because it was not
> particularly good piece of software.
> Was a good guidance, but would never run it knowing how it works. It had
> close to nothing implemented in terms of functionality. So that kinda
> already makes it so much not the same. Then the mount and unmount methods
> were a bit old school. Starting point is different with each firmware
> change, so no hope to safe that stuff. What was that wasted page/block in
> the beginning I also have no idea. There were some obvious typos as well,
> but that to say is normal. If that is what is inside NodeMCU, I am not at
> all surprised about all of the issues I hear about. People actually think
> that SPIFFS is unstable, while the whole problem is in the implementation.
> Obviously I should be grateful that he wrote such code, so I can get to
> know a new FS, maybe exchange some knowledge with him, after all there
> might be an idea behind his implementation, but do not feel that anymore
> And where is that guy?
> Wish you happy times
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/esp8266/Arduino/commit/37108bbbc7c9d3eada4786d2e4992e4414b8c88f#commitcomment-11337675>
> .
>


Thought so... with it being an offensive player controlled character there was an element of doubt. The offending comment shall be removed at once!


well it depends:
Yes from 1.9 (2.0) onwards, because synced ist marked deprecated.
On the other hand syncdb right now still offers to create a superuser,
which is needed for authentication, to create quotes.
so the requirement is quite the same, because right now syncdb does do something like
migrate and then ask for createsuperuser


It could just not add to the collection. A plugin could iterate over the returned collection to see what usernames failed if they need to know that.


please do the payment now


You can use `/usr/bin/xdg-open` here, so everything keeps working if you decide to start using another browser (which might as well be another variant of Firefox like Aurora or Iceweasel).


This file provides inverse of a matrix, first by looking into cache
otherwise by calculating itself.


please do payout not


What about a short?


@MaxXor, did you remove this as it was redundant code currently, or did you remove this because it was part of a xss vuln?


typo error update


Errors in this phrase. "Аргумент передется в ваш код при нажатии на кнопку Выполнить"
missing "a"


I did a POC indeed.  It is unlikely that it will be developed further in this form.  I have different ideas which aim to be more suitable, and should combine carbonserver and carbonwriter.


because grep always returns true, I use this trick for check.


I'm not sure I like this approach. Why exactly is setting the mask a special case? Could you elaborate that as I may be missing the idea behind it?


Thank you Chris. I thought I had my repo aligned... I'm getting old... correction... I AM old!


@SabreCat FYI


The indentation is off. You want to fix this to 2 spaces only.


This method wasn't used anymore.


You are hardcoding the path here. This is not suggested path in Rails, you have named routes to help with the maintenance of changing this.


You do not have to interpolate here, it is converted to string. `category.title` is good enough. also for named route you should use shorthand.

`%h3= link_to category.title, category`


Please move this back.


+1, nuget nuget nuget nueget nuegetgnuegnuetnuttt!!!


Depend on FSharp.Core nuget?


`video_video_path(video)` this is not correct way really, - just `video` is enough.


It bombs in Options.CreateParser with an Fclp.UnsupportedTypeException while calling TryAddAsSpecialParser where T is an IEnumerable<string> and I'm guessing its me misusing the library.  I'll re-read the docs and pull the source into my solution to debug further.

I'm basically trying to do this:

InvokeError -endpoints http://test.domain/services/service1.svc http://test.domain/services/service2.svc for instance.

I'd also suggest an overload of Validation which allows specifying errors, for instance I might want to say the 2nd item isn't a valid URL (if you library doesn't do this already for collections) such as:

public ICommandLineOptionFluent<T> Validate(Func<T, IList<Exception>, bool> validate)

Just an idea,,,


Ok thank you for clarifying!  I will re-add this shortly in a commit with my current pull-request


Better way to do this is actually to use Rails `image_tag` helper like this:

```haml
= image_tag video.small_cover_url
```


Test


Whoops, my bad!


This patch will fail and it will fail miserably.


Indentation is off everywhere.